Skip to content

Set WEB_BASE_URL explicitly for all deployed API Lambdas - #217

Merged
sdunster merged 1 commit into
mainfrom
pr217
Sep 9, 2026
Merged

Set WEB_BASE_URL explicitly for all deployed API Lambdas#217
sdunster merged 1 commit into
mainfrom
pr217

Conversation

@sdunster

@sdunster sdunster commented Sep 9, 2026

Copy link
Copy Markdown
Member

Member-facing period edit links were built from the first WEBAUTHN_RP_ORIGIN
entry, which in prod is the "new." alias rather than the seslogin.com apex.
Set WEB_BASE_URL directly instead of relying on that fallback:

prod/preprod -> https://seslogin.com
test -> https://test.seslogin.com

preprod points at the canonical site because it shares prod's DB and emails
real members; test keeps its own origin so the edit page can be exercised there.
WEBAUTHN config is untouched, so existing passkeys are unaffected.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

Member-facing period edit links were built from the first WEBAUTHN_RP_ORIGIN
entry, which in prod is the "new." alias rather than the seslogin.com apex.
Set WEB_BASE_URL directly instead of relying on that fallback:

  prod/preprod -> https://seslogin.com
  test         -> https://test.seslogin.com

preprod points at the canonical site because it shares prod's DB and emails
real members; test keeps its own origin so the edit page can be exercised there.
WEBAUTHN config is untouched, so existing passkeys are unaffected.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@sdunster

sdunster commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

terraform change already applied

@sdunster
sdunster merged commit 42ee6c3 into main Sep 9, 2026
2 checks passed
@sdunster
sdunster deleted the pr217 branch September 9, 2026 09:56
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