Skip to content

fix(tests): e2e auth headers for /events + /bundles/export (post #167) - #188

Merged
HongmingWang-Rabbit merged 1 commit into
mainfrom
fix/e2e-auth-headers-post-167
Apr 15, 2026
Merged

fix(tests): e2e auth headers for /events + /bundles/export (post #167)#188
HongmingWang-Rabbit merged 1 commit into
mainfrom
fix/e2e-auth-headers-post-167

Conversation

@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor

E2E broke when #186's self-hosted runner first exercised it. #167 gated these routes; adding bearer headers. 3 curl calls updated.

#167)

PR #167 gated /events and /bundles/export/:id behind AdminAuth. The e2e
script's 3 calls to these routes were unauthenticated and broke when the
runner picked them up for the first time on PR #186 (self-hosted runner
migration). Same admin-gate contract, same fix pattern as the #99/#110
e2e hotfixes.

POST /bundles/import is left unauthenticated because by that point in
the script both workspaces have been deleted and #110 revoked their
tokens, so HasAnyLiveTokenGlobal=0 and AdminAuth fails-open.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@HongmingWang-Rabbit
HongmingWang-Rabbit merged commit cdb45a3 into main Apr 15, 2026
@HongmingWang-Rabbit
HongmingWang-Rabbit deleted the fix/e2e-auth-headers-post-167 branch April 16, 2026 12:30
molecule-ai Bot pushed a commit that referenced this pull request Apr 21, 2026
fix(tests): e2e auth headers for /events + /bundles/export (post #167)
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