Skip to content

refactor(auth): route challenge debug logs through structured logger (#133) - #147

Open
ZacLou wants to merge 1 commit into
ourdao:mainfrom
ZacLou:refactor/route-challenge-logs-through-logger
Open

refactor(auth): route challenge debug logs through structured logger (#133)#147
ZacLou wants to merge 1 commit into
ourdao:mainfrom
ZacLou:refactor/route-challenge-logs-through-logger

Conversation

@ZacLou

@ZacLou ZacLou commented Sep 1, 2026

Copy link
Copy Markdown

Closes #133

Replaces the console.debug calls in MemoryNonceStore.issue and PostgresNonceStore.issue with structured logging through the Fastify request logger.

  • Adds a LoggerLike interface and an optional log parameter to NonceStore.issue.
  • The /auth/challenge route passes
    eq.log, so the debug output respects LOG_LEVEL (including silent).
  • Keeps the parameter optional so existing callers/tests that don't provide a logger still compile.

Verified with:

pm run typecheck`n-
pm run lint (only pre-existing warnings)

…urdao#133)

Replace console.debug calls in MemoryNonceStore.issue and
PostgresNonceStore.issue with optional structured logging via a
LoggerLike parameter. The /auth/challenge route now passes

eq.log so debug output honours LOG_LEVEL instead of bypassing it.

Closes ourdao#133
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.

Member addresses are debug-logged on every challenge request

1 participant