-
Notifications
You must be signed in to change notification settings - Fork 322
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: remove payload logging #5447
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5447 +/- ##
==========================================
+ Coverage 74.76% 75.08% +0.32%
==========================================
Files 440 462 +22
Lines 61705 63800 +2095
==========================================
+ Hits 46132 47905 +1773
- Misses 13022 13231 +209
- Partials 2551 2664 +113 ☔ View full report in Codecov by Sentry. |
This PR is considered to be stale. It has been open 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rudder-server/gateway/handle.go
Line 723 in e675d47
gw.logger.Debugn("response", |
we can remove from here also
Description
Remove logs that can expose event payloads.
Linear Ticket
Resolves PIPE-1825
Security