After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 695888 - Owncloud configuration expects a trailing slash
Owncloud configuration expects a trailing slash
Status: RESOLVED FIXED
Product: gnome-online-accounts
Classification: Core
Component: general
3.7.x
Other All
: Normal minor
: ---
Assigned To: GNOME Online Accounts maintainer(s)
GNOME Online Accounts maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-03-14 23:14 UTC by Adam Williamson
Modified: 2013-03-19 16:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
owncloud: Make sure Uri always ends in a trailing slash (2.07 KB, patch)
2013-03-15 13:19 UTC, Debarshi Ray
committed Details | Review

Description Adam Williamson 2013-03-14 23:14:37 UTC
If I try and set up an Owncloud account thus:

http://www.myserver.com/owncloud

it fails with a 404. Checking my server logs, I see:

192.168.1.5 - - [14/Mar/2013:16:11:48 -0700] "GET /owncloudremote.php/webdav/ HTTP/1.1" 404 26094

It really ought to be smart enough to append the trailing slash itself.
Comment 1 Debarshi Ray 2013-03-15 13:19:14 UTC
Created attachment 238978 [details] [review]
owncloud: Make sure Uri always ends in a trailing slash
Comment 2 Debarshi Ray 2013-03-15 13:23:10 UTC
Adam, could you please try this patch?

My test instance has an URL like https://owncloud.server.com/, which used to work when you typed "owncloud.server.com", but it was broken for cases where there is an actual path in the URL.
Comment 3 Debarshi Ray 2013-03-19 15:37:43 UTC
Comment on attachment 238978 [details] [review]
owncloud: Make sure Uri always ends in a trailing slash

Lets get this in for 3.7.92.

Reading the patch, I expect it to work and it does not break anything that used to work before. Lets try to have as many fixes in before the hard code freeze.
Comment 4 Debarshi Ray 2013-03-19 15:38:40 UTC
Thanks for the bug report.
Comment 5 Adam Williamson 2013-03-19 16:30:31 UTC
Sorry I didn't get to testing it :( I did a build with the patch all ready to test, then got sucked into an Alpha breaking bug vortex...