[PR #1113] [MERGED] V2 Breaking Changes #1062

New Issue

OVERLORD · 2026-02-04T21:14:03+03:00

OVERLORD commented

2026-02-04 21:14:03 +03:00

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/1113
Author: @stonith404
Created: 11/26/2025
Status: ✅ Merged
Merged: 12/30/2025
Merged by: @stonith404

Base: main ← Head: breaking/v2

📝 Commits (6)

caf0235 fix!: rename LDAP_ATTRIBUTE_ADMIN_GROUP env variable to LDAP_ADMIN_GROUP_NAME (#1089)
d558206 feat!: drop support for storing JWK on the filesystem (#1088)
73e9bef feat: add CLI command for importing and exporting Pocket ID data (#998)
b90f507 feat: remove DbProvider env variable and calculate it dynamically (#1114)
eeaf185 fix!: make wildcard matching in callback URLs more stricter (#1161)
5f2b0cc chore: preparation for merge into main branch

📊 Changes

77 files changed (+4464 additions, -2051 deletions)

View changed files

📝 .github/workflows/e2e-tests.yml (+15 -2)
📝 .gitignore (+1 -0)
📝 backend/cmd/main.go (+7 -0)
📝 backend/go.mod (+1 -1)
📝 backend/internal/bootstrap/bootstrap.go (+69 -38)
📝 backend/internal/bootstrap/db_bootstrap.go (+3 -99)
📝 backend/internal/bootstrap/e2etest_router_bootstrap.go (+1 -1)
📝 backend/internal/bootstrap/router_bootstrap.go (+1 -10)
📝 backend/internal/bootstrap/services_bootstrap.go (+2 -0)
➕ backend/internal/cmds/export.go (+70 -0)
➕ backend/internal/cmds/import.go (+191 -0)
📝 backend/internal/cmds/key_rotate.go (+2 -1)
📝 backend/internal/cmds/key_rotate_test.go (+1 -67)
📝 backend/internal/cmds/root.go (+4 -3)
📝 backend/internal/common/env_config.go (+50 -71)
📝 backend/internal/common/env_config_test.go (+31 -105)
📝 backend/internal/controller/e2etest_controller.go (+5 -2)
📝 backend/internal/dto/app_config_dto.go (+1 -1)
📝 backend/internal/model/app_config.go (+1 -1)
📝 backend/internal/model/types/date_time.go (+9 -0)

...and 57 more files

📄 Description

Migrations need to be renamed before merging.
Should be rebased instead of squash merged into main.

_{🔄 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/1113 **Author:** [@stonith404](https://github.com/stonith404) **Created:** 11/26/2025 **Status:** ✅ Merged **Merged:** 12/30/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `breaking/v2` --- ### 📝 Commits (6) - [`caf0235`](https://github.com/pocket-id/pocket-id/commit/caf0235e2ac701edb306ec1febaa55a5cecdf8ee) fix!: rename `LDAP_ATTRIBUTE_ADMIN_GROUP` env variable to `LDAP_ADMIN_GROUP_NAME` (#1089) - [`d558206`](https://github.com/pocket-id/pocket-id/commit/d5582066f8e11b9f8958695c2ea9bc1cb8f4161d) feat!: drop support for storing JWK on the filesystem (#1088) - [`73e9bef`](https://github.com/pocket-id/pocket-id/commit/73e9bef78cef1767c6c1ee2b662264aeefb7d829) feat: add CLI command for importing and exporting Pocket ID data (#998) - [`b90f507`](https://github.com/pocket-id/pocket-id/commit/b90f5075fdfbfb7d59c9e371cd8b6643ea64fd75) feat: remove DbProvider env variable and calculate it dynamically (#1114) - [`eeaf185`](https://github.com/pocket-id/pocket-id/commit/eeaf185e4f5fbd0fa334ac0042e296640c2f01ec) fix!: make wildcard matching in callback URLs more stricter (#1161) - [`5f2b0cc`](https://github.com/pocket-id/pocket-id/commit/5f2b0ccdb1bc8da2f2c9b7f9241d77218743f441) chore: preparation for merge into main branch ### 📊 Changes **77 files changed** (+4464 additions, -2051 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/e2e-tests.yml` (+15 -2) 📝 `.gitignore` (+1 -0) 📝 `backend/cmd/main.go` (+7 -0) 📝 `backend/go.mod` (+1 -1) 📝 `backend/internal/bootstrap/bootstrap.go` (+69 -38) 📝 `backend/internal/bootstrap/db_bootstrap.go` (+3 -99) 📝 `backend/internal/bootstrap/e2etest_router_bootstrap.go` (+1 -1) 📝 `backend/internal/bootstrap/router_bootstrap.go` (+1 -10) 📝 `backend/internal/bootstrap/services_bootstrap.go` (+2 -0) ➕ `backend/internal/cmds/export.go` (+70 -0) ➕ `backend/internal/cmds/import.go` (+191 -0) 📝 `backend/internal/cmds/key_rotate.go` (+2 -1) 📝 `backend/internal/cmds/key_rotate_test.go` (+1 -67) 📝 `backend/internal/cmds/root.go` (+4 -3) 📝 `backend/internal/common/env_config.go` (+50 -71) 📝 `backend/internal/common/env_config_test.go` (+31 -105) 📝 `backend/internal/controller/e2etest_controller.go` (+5 -2) 📝 `backend/internal/dto/app_config_dto.go` (+1 -1) 📝 `backend/internal/model/app_config.go` (+1 -1) 📝 `backend/internal/model/types/date_time.go` (+9 -0) _...and 57 more files_ </details> ### 📄 Description - Migrations need to be renamed before merging. - Should be rebased instead of squash merged into main. --- <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 2026-02-04 21:14:03 +03:00

OVERLORD closed this issue

2026-02-04 21:14:06 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pocket-id#1062