fix(share): use public DAV endpoint v2 for retrieving page content

[mejo-]

On some instances, the old public DAV endpoint (/public.php/webdav/...) returns 404 for some reason. The new public DAV endpoint v2 (/public.php/dav/files/<shareToken/...) works as expected.

The files_sharing app uses the new v2 endpoint, so let's use it as well.

Fixes: #2257

🏁 Checklist

• Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
• Sign-off message is added to all commits
• Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests

[mejo-]

We should also notify people in #1530 after this PR got released, that for those who still experienced the problem, it might be finally solved with the latest release.

[christianlupus]

I tested on my instance and it solved the issue for me. @mejo- tested on my instance with this patch cherry-pick-ed successfully.

No maintainer and mo overview over possible side effects

[max-nextcloud]

Thanks for getting to the root cause of this!

[max-nextcloud]

Cypress failures seem related though.

I suspect the trim is here:

collectives/src/components/Page/TextEditor.vue

Lines 178 to 180 in ac4d3a2

	// or when page is empty
	|| !this.davContent.trim()) {
	this.setTextEdit()

and for some reason the url does not return any responseText in some cases:

collectives/src/mixins/pageContentMixin.js

Lines 39 to 41 in ac4d3a2

	const content = await axios.get(davUrl, axiosConfig)
	// content.data will attempt to parse as json, but we want the raw text.
	return content.request.responseText

[mejo-]

The problem is that this change breaks getting the content of the page in password-protected shares. I'm currently looking into it.

[mejo-]

The problem is that this change breaks getting the content of the page in password-protected shares. I'm currently looking into it.

@max-nextcloud and me found the problem. The PR is ready for review again ☺️