Skip to content

fix(dev): make OIDC smoke target install its declared JWT dependency - #1118

Draft
seonghobae wants to merge 13 commits into
fix/contextual-orchestrator-owner-boundaryfrom
fix/oidc-smoke-declared-jwt-dependency
Draft

seonghobae wants to merge 13 commits into
fix/contextual-orchestrator-owner-boundaryfrom
fix/oidc-smoke-declared-jwt-dependency

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Current authority — 2026-09-22 KST

Findings owned here

The existing JWT/smoke RED→repair lineage remains authoritative: declared PyJWT dev dependency, removal of direct-interpreter bypass, token/JWKS compatibility evidence, PyJWT security floor, and RS256-only owned verifier paths. #899's latest movement is source-neutral (c10b6545..., no file delta) and is not accepted as repair/evidence.

Ordinary two-parent/non-force convergence 0dd0fd9a... adopts current #899 while preserving the seven-file JWT/smoke delta. Fresh compare from current #899 has exact merge-base and behind_by=0.

#1120, #1117 and #1124 were immediately converged down the stack; each has behind_by=0 against its new current parent and retains only its prior child-owned delta. No force push, destructive rebase, or parent source copy was used.

Predecessor hosted receipts do not transfer. Keep Draft while #899 is unmerged and until one unchanged exact head has applicable repository/security/static-analysis evidence and qualifying independent review.

No self-approval, gate weakening, blind rerun, new source-neutral wake commit, protected-main merge, or release is claimed.

@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Contributor Author

Dependency-security traceability refresh (2026-09-20): the current uv.lock resolves pyjwt 2.13.0, but both backend/dev metadata still permit pyjwt[crypto]>=2.8.0. GitHub Reviewed advisory GHSA-xgmm-8j9v-c9wx / CVE-2026-48526 marks all PyJWT <2.13.0 affected by the mixed-family public-JWK/HMAC confusion issue, and GHSA-w7vc-732c-9m39 / CVE-2026-48525 marks >=2.8.0, <=2.12.1 affected by the detached-JWS unbounded decode DoS; both are patched in 2.13.0. PyPI now publishes 2.14.0 (2026-09-11). Therefore the resolved lock is currently above the known 2.13.0 security boundary, but the declared floor remains security-stale and permits vulnerable future resolution. Do not fix this with a metadata-only floor edit: move declared floor, regenerated lock, and an executable dependency/security regression together, then attach exact-head install/security receipts. References: GHSA-xgmm-8j9v-c9wx ; GHSA-w7vc-732c-9m39 ; https://pypi.org/project/PyJWT/

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant