Repository navigation
Align Python workflows with supported endpoints and retired APIs - #191
Merged
Merged
Conversation
maxlastglance
force-pushed
the
fix/python-endpoint-contracts
branch
from
November 7, 2026 09:39
82d03e9 to
f63ae02
Compare
This was referenced Nov 7, 2026
maxlastglance
force-pushed
the
fix/python-endpoint-contracts
branch
from
November 7, 2026 09:48
f63ae02 to
6edd6f4
Compare
Contributor
Author
|
The Metadata suite exposed missing 403/404 OpenAPI response declarations on the new current-access endpoint (1074 tests, one assertion failure). The declarations are now included in the isolated #177 commit. The existing response-contract check and the four access tests pass locally after the correction; CI is running again. The PR still contains four commits. The live-tested Python code and endpoint business logic are unchanged. The current review head is |
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.
The Python client still calls removed catalog and CSV analysis APIs, expects the old statistics response, and has no working route for current-user access lookup.
Four separate commits address the live findings:
Closes #175. Closes #176. Closes #177. Closes #178. Closes #147.
Validation: 341 SDK unit tests passed locally. Four standalone Metadata endpoint tests passed for owner/member/missing-grant/anonymous cases. Schema tests cover upload and authenticated analysis, explicit/default/sole-image selection, candidate scores, rejected unsupported switches and routing precedence in both gateway configurations. The live user profile against a separate candidate Metadata instance and internal gateway on s93 reports 104 passed/11 failed, versus 100/15 before this batch. All four newly supported cases pass and no previously passing case fails. Three remaining raw failures are the explicitly retired catalog calls; eight belong to other open findings. Candidate containers and their temporary credentials were removed afterward. All PR checks now pass on the final review head; image publishing is intentionally skipped for pull requests. The earlier missing OpenAPI 403/404 declarations were corrected within the isolated #177 commit, and the complete suite passed on rerun.
The live check used a candidate JAR on the existing runtime image and a candidate gateway config; all 577 bundled core classes match a build from the candidate source. The persistent s93 deployment remains on its existing version. Removed catalog functions must be reported as unsupported, never counted as successful catalog reads. Hold merge until the maintainer has reviewed the four commits.