Uh oh!
There was an error while loading. Please reload this page.
fix: document the flags the server actually requires - #19
Merged
Conversation
The setup line repeated in 11 READMEs was v1-era and missing FOUR flags the 2.4.0 server refuses to start without. As written it fails on the first one before ever reaching the others: $ ./authorizer --database-type sqlite --database-url authorizer.db \ --admin-secret secret Error: --encryption-key is required: ... Replaced with a command verified to boot against a build of authorizer main: adds --url (authorizerdev/authorizer#764), --encryption-key, --jwt-type/--jwt-secret and --client-id/--client-secret. Also adds --url to the two org-SSO compose files and the k8s-tokenreview manifest; all three map 8080:8080, so the container's own address and the address the demo dials agree.
--url alone now suffices: authorizerdev/authorizer#768 makes `authorizer mcp` honour it. Verified with `node mcp-agent.mjs --verify` against a live server — all six assertions pass. The example passed --url and --mcp-authorizer-url with the same value. Removing the latter WITHOUT the server fix failed with `rpc error: code = Unauthenticated`, which is how the underlying bug was found.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The setup line repeated verbatim in 11 READMEs predates two releases' worth of required flags. It does not merely lack
--url— it fails before reaching it:Adding
--encryption-keythen surfacesmissing jwt type, thenclient ID missing in rootArgs, then--url is required. Four required flags missing in total.What
Replaced with a command verified to boot against a build of
authorizermain:Also adds
--urltowith-org-samlandwith-org-sso-oidccompose files and thewith-k8s-tokenreviewmanifest. All three map8080:8080, so the container's own address and the address the demo dials agree. All three validated as YAML.Not changed:
with-agent-permissions/run-server.shandwith-auth-recipes/run-server.shalready pass--url.Related
--urlmandatory; #766 records it, #767 fixes the flag help--urlas required docs#88 — the same sweep across the docs site