[PR #529] [MERGED] refactor!: remove old DB env variables, and jwk migrations logic #761

Closed
opened 2025-10-08 00:16:43 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/529
Author: @kmendell
Created: 5/13/2025
Status: Merged
Merged: 5/14/2025
Merged by: @stonith404

Base: mainHead: refactor/remove-migration-code


📝 Commits (1)

  • a91f226 refactor: remove old DB env variables, and jwk migrations logic

📊 Changes

5 files changed (+28 additions, -396 deletions)

View changed files

📝 backend/internal/bootstrap/bootstrap.go (+0 -4)
backend/internal/bootstrap/config_migration.go (+0 -34)
backend/internal/bootstrap/jwk_migration.go (+0 -136)
backend/internal/bootstrap/jwk_migration_test.go (+0 -190)
📝 backend/internal/common/env_config.go (+28 -32)

📄 Description

Moved these changes from PR #520 to this seperate one on a branch off of main instead.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pocket-id/pocket-id/pull/529 **Author:** [@kmendell](https://github.com/kmendell) **Created:** 5/13/2025 **Status:** ✅ Merged **Merged:** 5/14/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `refactor/remove-migration-code` --- ### 📝 Commits (1) - [`a91f226`](https://github.com/pocket-id/pocket-id/commit/a91f2268897fa5a9dfe83d0c7900f71825822c2e) refactor: remove old DB env variables, and jwk migrations logic ### 📊 Changes **5 files changed** (+28 additions, -396 deletions) <details> <summary>View changed files</summary> 📝 `backend/internal/bootstrap/bootstrap.go` (+0 -4) ➖ `backend/internal/bootstrap/config_migration.go` (+0 -34) ➖ `backend/internal/bootstrap/jwk_migration.go` (+0 -136) ➖ `backend/internal/bootstrap/jwk_migration_test.go` (+0 -190) 📝 `backend/internal/common/env_config.go` (+28 -32) </details> ### 📄 Description Moved these changes from PR #520 to this seperate one on a branch off of `main` instead. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2025-10-08 00:16:43 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#761