starred/pocket-id-pocket-id-1
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-09 14:53:00 +03:00
Code Issues 122 Packages Projects Releases 10 Wiki Activity

[PR #901] [MERGED] feat: initial client_credentials flow support #561

New Issue
Closed
opened 2025-10-08 00:12:57 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2025-10-08 00:12:57 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/901
Author: @savely-krasovsky
Created: 8/31/2025
Status: ✅ Merged
Merged: 9/3/2025
Merged by: @kmendell

Base: main ← Head: client-credentials

📝 Commits (10+)

  • 110533b feat: initial client_credentials flow support
  • 576e210 fix: add prefix as discussed internally
  • 19ce5ad feat: add tests
  • 0cc9de3 Merge branch 'main' into client-credentials
  • 99b53e0 fix: additional test and typo fixes
  • f6f724f style: few improvements
  • 51beb80 Merge branch 'main' into client-credentials
  • 3a47525 fix: add client_credentials flow in grant_types_supported
  • 609d909 Merge branch 'main' into client-credentials
  • 4e26597 style: small fixes

📊 Changes

4 files changed (+164 additions, -6 deletions)

View changed files

📝 backend/internal/controller/well_known_controller.go (+1 -1)
📝 backend/internal/dto/oidc_dto.go (+1 -0)
📝 backend/internal/service/oidc_service.go (+35 -3)
📝 backend/internal/service/oidc_service_test.go (+127 -2)

📄 Description

resource parameter could be used by every flow according to RFC 9068 (https://datatracker.ietf.org/doc/html/rfc9068#name-requesting-a-jwt-access-tok), but it's out of the scope of that PR.

Closes #890

🔄 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/901 **Author:** [@savely-krasovsky](https://github.com/savely-krasovsky) **Created:** 8/31/2025 **Status:** ✅ Merged **Merged:** 9/3/2025 **Merged by:** [@kmendell](https://github.com/kmendell) **Base:** `main` ← **Head:** `client-credentials` --- ### 📝 Commits (10+) - [`110533b`](https://github.com/pocket-id/pocket-id/commit/110533bf25f4e3e0286435333c48ac0d5ca046c9) feat: initial client_credentials flow support - [`576e210`](https://github.com/pocket-id/pocket-id/commit/576e210044dfb1f187205f2cd4df1a5f587c6df5) fix: add prefix as discussed internally - [`19ce5ad`](https://github.com/pocket-id/pocket-id/commit/19ce5adf21e66e93f9f5573bf48fe13f8c9ff334) feat: add tests - [`0cc9de3`](https://github.com/pocket-id/pocket-id/commit/0cc9de3cbe781b15bd68db1b64e1255d1439efc4) Merge branch 'main' into client-credentials - [`99b53e0`](https://github.com/pocket-id/pocket-id/commit/99b53e0245ff9e94ed36c227578f76b664a48f28) fix: additional test and typo fixes - [`f6f724f`](https://github.com/pocket-id/pocket-id/commit/f6f724fe4583b5d9bfa20da2e84ae3c9c213d797) style: few improvements - [`51beb80`](https://github.com/pocket-id/pocket-id/commit/51beb804373259edb498b99eb866c27590f5cbdb) Merge branch 'main' into client-credentials - [`3a47525`](https://github.com/pocket-id/pocket-id/commit/3a475252e531d307c16c6eae9b4a6435579f9df6) fix: add client_credentials flow in grant_types_supported - [`609d909`](https://github.com/pocket-id/pocket-id/commit/609d90966cff44cd7392fe44168cb1b034be90bb) Merge branch 'main' into client-credentials - [`4e26597`](https://github.com/pocket-id/pocket-id/commit/4e265970680a328807f4e9e0f571229745438414) style: small fixes ### 📊 Changes **4 files changed** (+164 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `backend/internal/controller/well_known_controller.go` (+1 -1) 📝 `backend/internal/dto/oidc_dto.go` (+1 -0) 📝 `backend/internal/service/oidc_service.go` (+35 -3) 📝 `backend/internal/service/oidc_service_test.go` (+127 -2) </details> ### 📄 Description `resource` parameter could be used by every flow according to RFC 9068 (https://datatracker.ietf.org/doc/html/rfc9068#name-requesting-a-jwt-access-tok), but it's out of the scope of that PR. Closes #890 --- <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:12:57 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#561
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?