Update email confirmation route(s)/journeys to be confirmed via POST #3296

Closed
opened 2026-02-05 06:17:00 +03:00 by OVERLORD · 0 comments
Owner

Originally created by @ssddanbrown on GitHub (Oct 22, 2022).

Originally assigned to: @ssddanbrown on GitHub.

Currently ConfirmEmailController@confirm is done via GET, which could be coincidentally triggered by systems generally expecting GET endpoints to be idempotent, such as email link scanning systems.

Flow & handling should be updated so confirmation action is done via user confirmation in non-GET request.

Originally created by @ssddanbrown on GitHub (Oct 22, 2022). Originally assigned to: @ssddanbrown on GitHub. Currently `ConfirmEmailController@confirm` is done via GET, which could be coincidentally triggered by systems generally expecting GET endpoints to be idempotent, such as email link scanning systems. Flow & handling should be updated so confirmation action is done via user confirmation in non-GET request.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#3296