Skip to content

fix: [stable-8.0] Remove the unmarshalable request body field from graph log messages - #12918

Open
2403905 wants to merge 1 commit into
stable-8.0from
backport/8.0/graph-drop-body-log-field
Open

fix: [stable-8.0] Remove the unmarshalable request body field from graph log messages#12918
2403905 wants to merge 1 commit into
stable-8.0from
backport/8.0/graph-drop-body-log-field

Conversation

@2403905

@2403905 2403905 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Backport of #12915 to stable-8.0.

Summary

  • We've removed the body field from the log messages of the graph service's HTTP handlers. The field was set from http.Request.Body, an io.ReadCloser, which can never be serialized into a useful log value.

@kw-security

kw-security commented Sep 9, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@2403905
2403905 enabled auto-merge (squash) September 9, 2026 17:16
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.

3 participants