Skip to content
Snippets Groups Projects

RHGitaly: avoid logging all metadata

Merged Georges Racinet requested to merge topic/default/rhgitaly-tracing into branch/default

Logging all invocation metadata is verbose and contains secrets (authentication header). Of these, only the correlation_id is interesting to us at the moment.

To achieve this, we split the service method with an inner method taking the inner request and the correlation id. Only the inner method is now instrumented. This is heavy for the implementation yet good enough for now.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading