Skip to content

Log debug messages at the console debug level - #2489

Merged
kmcginnes merged 1 commit into
mainfrom
t3/fix-logger-debug-level
Oct 28, 2026
Merged

kmcginnes merged 1 commit into
mainfrom
t3/fix-logger-debug-level

Conversation

@kmcginnes

Copy link
Copy Markdown
Collaborator

Summary

ServerLoggerConnector.debug wrote to the browser console through the info path instead of debug, so debug messages could not be filtered in the console like the other levels.

User-facing: Debug messages now appear under the browser console's Debug level instead of Info.

 public debug(message: string) {
-  this.#clientLogger.info(message);
+  this.#clientLogger.debug(message);
   return this.#sendLog("debug", message);
 }

The request to POST /logger already carried level: "debug" after Send client logs to /logger as a JSON body (#2468), and the proxy already drops it at the default LOG_LEVEL=info, so the server side needs no change.

Evidence

Before: a test for debug expecting logger.debug failed, because the call landed on logger.log.
After: a five-row test.each pins both the console method and the /logger level for error, warn, info, debug and trace, and passes.

Verified live: ran the app against the proxy; debug printed at the console Debug level and was dropped by the server at LOG_LEVEL=info, while info still printed.

Merge Danger

Door: two-way

Impact: negligible

Related


🤖 Written by an AI agent on my behalf.

@kmcginnes kmcginnes left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@kmcginnes
kmcginnes merged commit da9d908 into main Oct 28, 2026
4 checks passed
@kmcginnes
kmcginnes deleted the t3/fix-logger-debug-level branch October 28, 2026 19:32
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