Parse Sabre Exception in OC.Files.Client and file-upload

[PVince81]

Description

In case of error, instead of a generic error message, an upload will
display whichever message is returned in the Sabre Exception, if
applicable.

Related Issue

Part of #24304.
Should fix owncloud/files_antivirus#163

Motivation and Context

See #24304

How Has This Been Tested?

I manually tested this by triggering Sabre exception on upload from the server side (hard-coded exception). Tested with chunked upload error and regular upload error. The message from the server is displayed in the client.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[PVince81]

Yes, I did have to parse the sabre exception in two places... This is really messy because we have multiple ways of doing queries, and the one for the PUT upload is managed by jquery.fileupload, so it needed its own code path... I hope one day we can find a way to consolidate all.

@VicDeo please review and test with owncloud/files_antivirus#163

[jvillafanez]

I don't think there is a good way out for this...
👍 as long as it works.

[VicDeo]

Master

This branch

so yes, fixes owncloud/files_antivirus#163

[PVince81]

Thanks, guys!

[PVince81]

stable10: #28635

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.