Skip to content

PR-F2: Tighten trusted-publish OIDC context and policy docs#1163

Merged
lmeyerov merged 3 commits intomasterfrom
chore/1130-pr-f2-oidc-tightening
Apr 21, 2026
Merged

PR-F2: Tighten trusted-publish OIDC context and policy docs#1163
lmeyerov merged 3 commits intomasterfrom
chore/1130-pr-f2-oidc-tightening

Conversation

@lmeyerov
Copy link
Copy Markdown
Contributor

Summary

Implements PR-F2 (#1130) as a narrow OIDC/trusted-publish hardening slice.

What changed

  • .github/workflows/publish-pypi.yml
    • Added workflow_dispatch confirmation input confirm_ref.
    • Added publish-context assertions before credentialed publish steps:
      • GITHUB_REPOSITORY must be graphistry/pygraphistry.
      • GITHUB_WORKFLOW_REF must match .github/workflows/publish-pypi.yml@${GITHUB_REF}.
    • Added release-tag format enforcement for tag-triggered publish.
    • Kept existing trusted-ref checks:
      • tag commits must be in origin/master history.
      • manual dispatch only from refs/heads/master.
  • DEVELOP.md
    • Documented confirm_ref=master for manual publish recovery.
    • Added explicit PyPI Trusted Publisher alignment checklist (repository/workflow/environment/refs).
  • CHANGELOG.md
    • Added infrastructure entry for PR-F2 hardening.

Why

This reduces confused-deputy/OIDC misuse risk by failing closed unless publish runs from the expected repo/workflow/ref context and by making external Trusted Publisher policy requirements explicit in the runbook.

Validation

  • YAML parse check for publish-pypi.yml (YAML_OK).
  • Full CI intentionally left to PR checks (includes workflow-security gates).

Non-goals

  • SBOM/provenance/signing work (planned as PR-F3/F4).
  • SHA pin migration.

@lmeyerov lmeyerov marked this pull request as ready for review April 21, 2026 02:50
@lmeyerov lmeyerov merged commit 5dccbfd into master Apr 21, 2026
53 checks passed
@lmeyerov lmeyerov deleted the chore/1130-pr-f2-oidc-tightening branch April 21, 2026 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant