Skip to content

fix(conf): 忽略无效 SMTP headers 配置#2568

Open
pjpjq wants to merge 1 commit into
supabase:masterfrom
pjpjq:codex/ignore-invalid-smtp-headers
Open

fix(conf): 忽略无效 SMTP headers 配置#2568
pjpjq wants to merge 1 commit into
supabase:masterfrom
pjpjq:codex/ignore-invalid-smtp-headers

Conversation

@pjpjq
Copy link
Copy Markdown

@pjpjq pjpjq commented Jun 6, 2026

Summary

  • treat invalid optional GOTRUE_SMTP_HEADERS JSON as a warning instead of failing global config validation
  • clear stale normalized SMTP headers before each validation pass
  • keep valid SMTP header parsing unchanged and cover invalid env loading with regression tests

Fixes #2533

Tests

  • go test ./internal/conf -run TestValidate -count=1
  • go test ./internal/conf/confload -run 'TestGlobal|TestGlobalIgnoresInvalidSMTPHeaders' -count=1
  • go test ./internal/conf/... -count=1

@pjpjq pjpjq requested a review from a team as a code owner June 6, 2026 12:14
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.

Auth pod crash-loops on invalid GOTRUE_SMTP_HEADERS JSON — fatal Validate() breaks the entire service

1 participant