Skip to content

Conversation

@backportbot-nextcloud
Copy link

backport of #15308

private function filterTrace(array $trace) {
$sensitiveValues = [];
$trace = array_map(function (array $traceLine) use (&$sensitiveValues) {
$className = $traceLine['class'] ?? '';
Copy link
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
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
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

@rullzer rullzer mentioned this pull request May 9, 2019
@MorrisJobke MorrisJobke added the 4. to release Ready to be released and/or waiting for tests to finish label May 9, 2019
blizzz and 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 blizzz force-pushed the backport/15308/stable15 branch from a0f43eb to eb8c2e9 Compare May 9, 2019 09:26
@blizzz
Copy link
Member

blizzz commented May 9, 2019

rebased to retrigger ci

@MorrisJobke
Copy link
Member

rebased to retrigger ci

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

@blizzz
Copy link
Member

blizzz commented May 9, 2019

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 stable15 May 9, 2019
@MorrisJobke MorrisJobke deleted the backport/15308/stable15 branch May 9, 2019 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants