Skip to content

[stable15] remove setup args from logging - #15443

Merged
MorrisJobke merged 3 commits into
stable15from
backport/15308/stable15
May 9, 2019
Merged

[stable15] remove setup args from logging#15443
MorrisJobke merged 3 commits into
stable15from
backport/15308/stable15

Conversation

@backportbot-nextcloud

Copy link
Copy Markdown

backport of #15308

private function filterTrace(array $trace) {
$sensitiveValues = [];
$trace = array_map(function (array $traceLine) use (&$sensitiveValues) {
$className = $traceLine['class'] ?? '';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

parse error

Copy link
Copy Markdown
Member

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?

Copy link
Copy Markdown
Member

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

@rullzerrullzer mentioned this pull request May 9, 2019
@MorrisJobkeMorrisJobke added the 4. to release Ready to be released and/or waiting for tests to finish label May 9, 2019
blizzzand others added 3 commits May 9, 2019 11:25
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>
@blizzz
blizzzforce-pushed the backport/15308/stable15 branch from a0f43eb to eb8c2e9CompareMay 9, 2019 09:26
@blizzz

Copy link
Copy Markdown
Member

rebased to retrigger ci

@MorrisJobke

Copy link
Copy Markdown
Member

rebased to retrigger ci

Why? It was already triggered, but CI is just overloaded.

@blizzz

Copy link
Copy Markdown
Member

Why? It was already triggered, but CI is just overloaded.

It appeared to me it was floating dead in the water

@MorrisJobke
MorrisJobke merged commit 1df96e0 into stable15May 9, 2019
@MorrisJobke
MorrisJobke deleted the backport/15308/stable15 branch May 9, 2019 11:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blizzz@MorrisJobke@nickvergessen@ChristophWurst