Repository navigation
feat(authup): track authup 1.0.0-beta.66 - #35
Merged
Merged
Conversation
No image entrypoint, CLI role, port or environment-variable contract change between beta.65 and beta.66: Dockerfile and entrypoint.sh are unchanged and packages/server-config/src gained no field. The new CLI commands (login, logout, whoami, api <entity>) sit beside an unchanged start tree. The device authorization grant adds /device_authorization and the hosted /device page. Both sit outside /console and ride the server catch-all, and the /device redirect target /console/auth/device rides the auth console prefix, so no routing value changes. Assert that catch-all in the contract script for both Ingress and HTTPRoute, since a new hosted page now depends on it. A console bundle substituted through AUTH_CONSOLE_PATH must be rebuilt: the auth console render contract is version 5, and an older bundle is refused at boot. Also render the README's topology sentence from appVersion; it had been pinned at beta.64 since the 0.4.0 release.
Merged
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
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.
Adopts Authup v1.0.0-beta.66.
What changed upstream
Verified by diffing the
authupmonorepo atv1.0.0-beta.65..v1.0.0-beta.66:Dockerfileandentrypoint.share unchanged, so the image entrypoint contract holds.packages/server-config/srcgained no field: no new environment variables and no newserver.configlong-tail options.apps/authup/src/module.tsonly adds operator-side commands (login,logout,whoami,api <entity>, a CLI that signs itself in through the device grant) beside an unchangedstarttree. Role args and ports are identical.Three deployment-facing facts, none of which needs a chart value:
/device_authorizationand the hosted/devicepage sit outside/console, so they ride the server catch-all, and the/deviceredirect target/console/auth/devicerides the auth console's existing/console/authprefix. Confirmed against rendered split manifests, not just the upstream note. The grant is opt-in per client through the provisioning fieldgrantTypes.AUTH_CONSOLE_PATHand built against an older contract is refused at boot and must be rebuilt. Called out inartifacthub.io/changes.permission_checkpermission are API and provisioning behavior. They change which grants an operator hands out, not what the chart renders.Changes
charts/authup/Chart.yamlappVersionto1.0.0-beta.66; rewroteartifacthub.io/changes.agents/references/authup.md/consolebelong to the API set through the catch-all.agents/architecture.mdscripts/check-beta64-contract.pycharts/authup/README.md.gotmplappVersioncharts/authup/README.mdDrive-by fix
The README prose read "It runs Authup v1.0.0-beta.64's role-based CLI topology" with the version hardcoded, so it was already stale at beta.65. It now renders
{{ template "chart.appVersion" . }}.Contract assertions
check_routingnow asserts, in split mode, that:path: /,pathType: Prefix, backed by the API Servicematches(an empty match is the catch-all) and points at the API ServiceBoth were mutation-tested: breaking the catch-all path in each branch of
_ingress.tplfires the matching assertion, and_ingress.tplis unmodified in this PR.Verification
make testpasses (lint, the sevenci/*-values.yamlrenders, 476-path values coverage, the full contract script)make docs schemaproduces no drift beyond the regenerated README committed here