feat: make metadata/manifest GET endpoint publicly accessible (#676) #677

Closed
jmiller wants to merge 1 commits from fix/676-public-metadata into main
Owner

Summary\n\nRemoves reqRepoReader auth requirement from GET /repos/{owner}/{repo}/metadata and /manifest endpoints, making them publicly accessible for public repos without authentication.\n\n## Changes\n\n- routers/api/v1/api.go: Split Combo routes into separate Get/Put routes\n- GET: no auth required (public repos accessible anonymously)\n- PUT: still requires reqToken() + reqAdmin()\n\n## Why\n\n- CI workflows need platform type without a token\n- Package registries need version info\n- Metadata is non-sensitive (version, platform, element name)\n- Matches existing pattern (badges are already public)\n\nCloses #676

## Summary\n\nRemoves `reqRepoReader` auth requirement from GET `/repos/{owner}/{repo}/metadata` and `/manifest` endpoints, making them publicly accessible for public repos without authentication.\n\n## Changes\n\n- `routers/api/v1/api.go`: Split `Combo` routes into separate `Get`/`Put` routes\n- GET: no auth required (public repos accessible anonymously)\n- PUT: still requires `reqToken()` + `reqAdmin()`\n\n## Why\n\n- CI workflows need platform type without a token\n- Package registries need version info\n- Metadata is non-sensitive (version, platform, element name)\n- Matches existing pattern (badges are already public)\n\nCloses #676
jmiller added 1 commit 2026-06-21 15:07:59 +00:00
feat: make metadata/manifest GET endpoint publicly accessible (#676)
Universal: PR Check / Build RC Package (pull_request) Blocked by required conditions
Universal: PR Check / Report Issues (pull_request) Blocked by required conditions
Generic: Repo Health / Scripts governance (pull_request) Blocked by required conditions
Generic: Repo Health / Repository health (pull_request) Blocked by required conditions
Generic: Repo Health / Report Issues (pull_request) Blocked by required conditions
Universal: PR Check / Branch Policy (pull_request) Failing after 1s
Generic: Repo Health / Site Health (pull_request) Has been skipped
Generic: Repo Health / Access control (pull_request) Successful in 2s
Universal: PR Check / Validate PR (pull_request) Failing after 7s
Branch Cleanup / Delete merged branch (pull_request) Has been skipped
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Universal: Workflow Sync Trigger / Sync workflows to live repos (pull_request) Has been skipped
Universal: Auto Version Bump / Version Bump (push) Successful in 10s
PR RC Release / Build RC Release (pull_request) Failing after 1m2s
Universal: PR Check / Secret Scan (pull_request) Successful in 1m3s
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Successful in 35s
Universal: Build & Release / Promote to RC (pull_request) Has been skipped
Universal: Build & Release / Build & Release Pipeline (pull_request) Has been skipped
Publish to Composer / Publish Package (release) Failing after 22s
f570e3e488
Remove reqRepoReader auth requirement from GET /repos/{owner}/{repo}/metadata
and /manifest endpoints. PUT (update) still requires token + admin.
jmiller closed this pull request 2026-06-21 15:08:18 +00:00
jmiller deleted branch fix/676-public-metadata 2026-06-21 15:08:18 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.