Skip to content

fix: add Origin header to E2E test POST requests blocked by CSRF middleware#125

Merged
mahata merged 1 commit intofeat/email-signupfrom
copilot/fix-e2e-workflow-failure
Mar 25, 2026
Merged

fix: add Origin header to E2E test POST requests blocked by CSRF middleware#125
mahata merged 1 commit intofeat/email-signupfrom
copilot/fix-e2e-workflow-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 25, 2026

Playwright's page.request.post() doesn't include an Origin header, causing Hono's globally-applied csrf() middleware to return 403 for /test/login and /test/logout — failing every E2E test at the authentication step.

Changes

  • e2e/auth-helpers.ts — add Origin: http://localhost:3000 to POST /test/login
  • e2e/app.spec.ts — same for POST /test/logout in afterEach
  • e2e/database-persistence.spec.ts — same for POST /test/logout in afterEach
const response = await page.request.post("/test/login", {
  headers: { Origin: "http://localhost:3000" },
});
Original prompt

Fix the failing GitHub Actions workflow e2e
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 68544185741
Job URL: https://github.com/mahata/mlack/actions/runs/23545433970/job/68544185741


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

@mahata mahata marked this pull request as ready for review March 25, 2026 14:19
@mahata mahata merged commit 70f4da8 into feat/email-signup Mar 25, 2026
1 of 2 checks passed
@mahata mahata deleted the copilot/fix-e2e-workflow-failure branch March 25, 2026 14:19
Copilot AI changed the title [WIP] Fix failing GitHub Actions workflow e2e fix: add Origin header to E2E test POST requests blocked by CSRF middleware Mar 25, 2026
Copilot AI requested a review from mahata March 25, 2026 14:20
Copilot stopped work on behalf of mahata due to an error March 25, 2026 14:20
mahata added a commit that referenced this pull request Mar 25, 2026
* feat(auth): implement email authentication with login and registration routes

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* test(emailAuth): add unit tests for success flows and conflict paths (#121)

* Initial plan

* test(emailAuth): add unit tests for success flows and conflict paths

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>
Agent-Logs-Url: https://github.com/mahata/mlack/sessions/e8292715-c961-480d-8c1a-490e2db65f9a

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* fix: CSRF protection for email auth form endpoints (#122)

* Initial plan

* fix: add CSRF protection for email auth form endpoints

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>
Agent-Logs-Url: https://github.com/mahata/mlack/sessions/81a07abc-5f9a-4910-8bdc-71951fdb71bb

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* fix: add missing Origin header in POST tests blocked by CSRF middleware (#124)

* Initial plan

* fix: add Origin header to POST tests blocked by CSRF middleware

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>
Agent-Logs-Url: https://github.com/mahata/mlack/sessions/86a80766-f33f-4962-ba8e-c5907a70e520

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* Initial plan (#125)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* fix: add Origin header to E2E test requests blocked by CSRF middleware (#126)

* Initial plan

* fix: add Origin header to E2E test requests blocked by CSRF middleware

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>
Agent-Logs-Url: https://github.com/mahata/mlack/sessions/edbeed38-429c-4f9f-beb6-cf8d58dc7ec4

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>
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.

2 participants