mirror of
https://github.com/pocket-id/pocket-id.git
synced 2026-07-25 06:04:34 +03:00
🚀 Feature: PKCE Support #60
Reference in New Issue
Block a user
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?
Originally created by @stonith404 on GitHub (Nov 15, 2024).
Feature description
Implement PKCE for OIDC.
Pitch
When public clients (e.g., native and single-page applications) request access tokens, some additional security concerns are posed that are not mitigated by the Authorization Code Flow alone. Because of that PKCE should be supported by Pocket ID.
@stonith404 commented on GitHub (Nov 15, 2024):
@cdemi Would you mind to test the
stonith404/pocket-id:developmentimage? PKCE should work in this image.@cdemi commented on GitHub (Nov 15, 2024):
I have spun up the
developmentimage and testing it out against https://oidcdebugger.com/ and I still get a CORS error:Access to XMLHttpRequest at 'https://mypocketid.domain/api/oidc/token' from origin 'https://oidcdebugger.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resourceServer logs and browser show
403on/api/oidc/token:pocketid | [GIN] 2024/11/15 - 15:54:14 | 403 | 57.157µs | xxx.xxx.xxx.xxx | POST "/api/oidc/token"@stonith404 commented on GitHub (Nov 16, 2024):
Thanks, I made some changes and now it should work. Could you try the new
developmentimage again?You now have to set your OIDC client to public in the OIDC client settings, to enable PKCE:

I made some changes to the migration, so please run the following command after you've updated the image:
@stonith404 commented on GitHub (Nov 21, 2024):
Added in
v0.15.0.