Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent WebDAV endpoints #18096

Closed
J0WI opened this issue Nov 24, 2019 · 4 comments
Closed

Inconsistent WebDAV endpoints #18096

J0WI opened this issue Nov 24, 2019 · 4 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug needs info

Comments

@J0WI
Copy link
Contributor

J0WI commented Nov 24, 2019

The documented url is /remote.php/dav/files/USERNAME/, but remote.php/webdav/ is still exposed to the user interface:

<input id="webdavurl" type="text" readonly="readonly"
value="<?php p(\OCP\Util::linkToRemote('webdav')); ?>"/>

'webdav-root' => $this->config->getSystemValue('webdav-root', 'remote.php/webdav'),

And some hardcoded endpoints are here:
https://github.com/nextcloud/server/tree/1074e9838b508aacaac8e8d49f9506d68402dafa/build/integration/features/bootstrap

@J0WI J0WI added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Nov 24, 2019
@gary-kim
Copy link
Member

gary-kim commented Nov 26, 2019

Also hardcoded into some of my scripts and configs. Gonna have to go through to change that... 🤣

Referencing #18061, it may be nice to keep the old endpoint around for a bit before it is fully deprecated. Though the web interface exposed link should change.

@nursoda
Copy link

nursoda commented Jan 7, 2020

What's the rationale to deprecate a static endpoint that depends on login credentials only? In terms of user documentation, it's far easier to have a static string users can cut'n'paste than /USERNAME/ …

@szaimen
Copy link
Contributor

szaimen commented May 28, 2021

Is this Issue still valid? If not, please close this issue. Thanks! :)

@gary-kim
Copy link
Member

It's been changed on the frontend for files at this point, so I'd say that places still using the old endpoint should be reported as a separate more specific bug.

Let's close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug needs info
Projects
None yet
Development

No branches or pull requests

4 participants