Skip to content

DAV's exception logger should deal with any Throwable - #20503

Merged
blizzz merged 1 commit into
masterfrom
fix/noid/dav-catch-throwable
Apr 15, 2020
Merged

DAV's exception logger should deal with any Throwable#20503
blizzz merged 1 commit into
masterfrom
fix/noid/dav-catch-throwable

Conversation

@blizzz

Copy link
Copy Markdown
Member

I was having an occasion where a TypeError was passed in via sabre's WildcardEmitterTrait

"Exception": "TypeError", "Message": "Argument 1 passed to OCA\\DAV\\Connector\\Sabre\\ExceptionLoggerPlugin::logException() must be an instance of Exception, instance of TypeError given, called in /srv/http/nextcloud/master/3rdparty/sabre/event/lib/WildcardEmitterTrait.php on line 89",

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

@skjnldsvskjnldsv left a comment

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.

Sure thing!

@blizzz

Copy link
Copy Markdown
MemberAuthor

cs fixer complains about unrelated files, can be ignored in this context

@blizzzblizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 15, 2020
@rullzerrullzer mentioned this pull request Apr 15, 2020
59 tasks
@blizzz
blizzz merged commit 48a8f0a into masterApr 15, 2020
@blizzz
blizzz deleted the fix/noid/dav-catch-throwable branch April 15, 2020 19:48
@rullzerrullzer mentioned this pull request Apr 15, 2020
57 tasks
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 finishbugfeature: dav

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@blizzz@nickvergessen@georgehrke@ChristophWurst@skjnldsv