fix: harden stable promotion and MCP gateway failures - #68
Merged
Merged
Conversation
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.
Closes #67
Summary
Requirement-to-evidence matrix
test-validate-stable-promotion.pyexercises no-checkout fixture operation, source/release/asset/checksum rejection, pagination/monotonicity, image allowlisting and missing platform rejection before a plan is produced. The workflow resolves a peeled tag, verifies main ancestry, validates all immutable digests before any mutable write, then reads eachlatestreference back.McpHttpHostTestscovers upstream 401/403/429/500/503, bounded retry-after, malformed and oversized replies, header/body deadline stalls, reset/refused connections, caller cancellation and no fallback.test-deployment-archive-compose-config.shrenders the actual packaged standalone gateway, combined gateway and explicit-Authentik Compose recipes.McpGatewayRealAccessPipelineTestsretains real SQLite/PostgreSQL persistence and adds simultaneous credential scope isolation plus wrong-purpose rejection; host tests exercise real delegation/current-identity and credential revocation. Existing direct API/stdio parity coverage remains.Validation run locally
dotnet test Helpdesk.sln --configuration Release --no-restore --verbosity minimal— 1,198 passed, 6 existing skips.Deliberate non-actions / follow-up evidence
No release, deployment, tag, image publication, or stable-channel promotion was performed. External configured-Authentik validation remains a separately recorded manual acceptance exercise; this PR does not use operator credentials or claim that evidence.