-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
[stable15] remove setup args from logging #15443
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
Conversation
| private function filterTrace(array $trace) { | ||
| $sensitiveValues = []; | ||
| $trace = array_map(function (array $traceLine) use (&$sensitiveValues) { | ||
| $className = $traceLine['class'] ?? ''; |
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.
parse error
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.
Why? Nextcloud 15 is PH 7.0+, right?
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.
And it went through in 14
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Co-Authored-By: blizzz <blizzz@arthur-schiwon.de> Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
a0f43eb to
eb8c2e9
Compare
|
rebased to retrigger ci |
Why? It was already triggered, but CI is just overloaded. |
It appeared to me it was floating dead in the water |
backport of #15308