supabase / auth / 27304287137

71%

master: 71%

Build Type

Pull #2572

github

Committed by Chris Stockton

Commit Message

chore(conf): do not halt startup for non-essential config fields

This wraps some non-essential fields that would previously halt
the startup of the auth server when Validate() failed with a
cachedValue. This cached value holds the original error for
testability but maintains the existing API's around it. When err
is non-nil we log a warning.

I also removed some unrelated dead code in internal/mailer I
spotted while auditing uses of the newly wrapped config values.

Pull Request Pull Request #2572: chore(conf): do not halt startup for non-essential config fields

Coverage Stats

63 of 63 new or added lines in 1 file covered. (100.0%)

17696 of 24862 relevant lines covered (71.18%)

651.41 hits per line