fix: consistent ClientID assignment, close #470 by localnerve · Pull Request #472 · authorizerdev/authorizer
Navigation Menu
{{ message }}
authorizerdev / authorizer Public
- Notifications You must be signed in to change notification settings
- Fork 204
Merged
lakhansamani merged 1 commit intoauthorizerdev:mainfrom
Dec 30, 2025Merged
fix: consistent ClientID assignment, close #470#472
lakhansamani merged 1 commit intoauthorizerdev:mainfrom
fix: consistent ClientID assignment, close #470#472
lakhansamani merged 1 commit intoauthorizerdev:mainfrom
Conversation
Copy link Copy Markdown
Contributor
localnerve
commented
Oct 3, 2025
localnerve
commented
What does this PR do?
- Fixes CLIENT_ID env variable not set on first start #470, make ClientID assignment consistent. Keeps the old policy, but if osClientID is there and different, use and update.
- Fixes the testSuite to execute without crashing after prior API updates that result in correct err and nil responses (see test suite for updates, pretty minor, but necessary to keep test suite functional).
Which issue(s) does this PR fix?
If this PR affects any API reference documentation, please share the updated endpoint references
No change
lakhansamani approved these changes Dec 30, 2025
lakhansamani
merged commit
35c6083
into
authorizerdev:main
Copy link Copy Markdown
Contributor
lakhansamani
commented
Dec 30, 2025
lakhansamani commented
Dec 30, 2025@localnerve thank you for fixing it and sorry for the delay
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants