Skip to content

Prevent quoting debug reports#50781

Merged
rosstimothy merged 1 commit intomasterfrom
tross/log_quoting
Jan 7, 2025
Merged

Prevent quoting debug reports#50781
rosstimothy merged 1 commit intomasterfrom
tross/log_quoting

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

The refactoring done in #50451 caused the trace.Error debug report to be quoted in log output. This updates the quoting bypass used for log level to also consider trace.Errors in order to restore the previous output.

Changelog: Prevent quoting errors in log messages.

The refactoring done in #50451 caused the trace.Error debug report
to be quoted in log output. This updates the quoting bypass used
for log level to also consider trace.Errors in order to restore
the previous output.
@rosstimothy rosstimothy enabled auto-merge January 6, 2025 20:52
@rosstimothy rosstimothy added this pull request to the merge queue Jan 7, 2025
Merged via the queue into master with commit 9e34cc9 Jan 7, 2025
@rosstimothy rosstimothy deleted the tross/log_quoting branch January 7, 2025 16:08
@public-teleport-github-review-bot
Copy link
Copy Markdown

@rosstimothy See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR
branch/v17 Create PR

carloscastrojumo pushed a commit to carloscastrojumo/teleport that referenced this pull request Feb 19, 2025
The refactoring done in gravitational#50451 caused the trace.Error debug report
to be quoted in log output. This updates the quoting bypass used
for log level to also consider trace.Errors in order to restore
the previous output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants