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

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

Summary

Removes reqRepoReader auth requirement from GET /repos/{owner}/{repo}/metadata and /manifest endpoints, making them publicly accessible for public repos without authentication.

Changes

  • routers/api/v1/api.go: Split Combo routes into separate Get/Put routes
  • GET: no auth required (public repos accessible anonymously)
  • PUT: still requires reqToken() + reqAdmin()

Why

  • CI workflows need platform type without a token
  • Package registries need version info
  • Metadata is non-sensitive (version, platform, element name)
  • Matches existing pattern (badges are already public)

Closes #676

## Summary Removes `reqRepoReader` auth requirement from GET `/repos/{owner}/{repo}/metadata` and `/manifest` endpoints, making them publicly accessible for public repos without authentication. ## Changes - `routers/api/v1/api.go`: Split `Combo` routes into separate `Get`/`Put` routes - GET: no auth required (public repos accessible anonymously) - PUT: still requires `reqToken()` + `reqAdmin()` ## Why - CI workflows need platform type without a token - Package registries need version info - Metadata is non-sensitive (version, platform, element name) - Matches existing pattern (badges are already public) Closes #676
jmiller added 1 commit 2026-06-21 15:18:49 +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) Successful in 2s
Generic: Repo Health / Site Health (pull_request) Has been skipped
Generic: Repo Health / Access control (pull_request) Successful in 1s
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 11s
PR RC Release / Build RC Release (pull_request) Failing after 54s
Universal: PR Check / Secret Scan (pull_request) Successful in 54s
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Successful in 42s
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 5s
add7c0da4d
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:19:11 +00:00
jmiller deleted branch chore/676-public-metadata 2026-06-21 15:19:11 +00:00

Pull request closed

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