Cyberduck Mountain Duck CLI

#2812 closed defect (fixed)

opening new connection with URL that contains trailing white space fails

Reported by: gnaegi Owned by: dkocher
Priority: normal Milestone: 3.1
Component: webdav Version: 3.0.3
Severity: normal Keywords:
Cc: gnaegi@… Architecture:
Platform:

Description

In our system the generate the WebDAV URL so that users can copy/paste it to their WebDAV client. When doing a dripple click to select the entire URL at once some whitespace after the URL with line breaks is also marked. I did not realize this, copied it to the Cyberduck new connection dialog and got the following error while connecting:

PROPFIND /myurl/webdav%0A HTTP/1.1[\r][\n]
Authorization: Basic Z25hZWdpOmhpaGkx[\r][\n]
Content-Type: text/xml; charset=utf-8[\r][\n]
User-Agent: Cyberduck/3.0.3 (4205)[\r][\n]
Host: www.myhost.com[\r][\n]
Content-Length: 207[\r][\n]
Depth: 0[\r][\n]
[\r][\n]
HTTP/1.1 501 Not Implemented[\r][\n]
HTTP/1.1 501 Not Implemented[\r][\n]
Date: Tue, 16 Dec 2008 16:47:18 GMT[\r][\n]
Server: Apache/2.0.53 (Linux/SUSE)[\r][\n]
Content-Length: 1238[\r][\n]
Connection: close[\r][\n]
Content-Type: text/html;charset=utf-8[\r][\n]
[\r][\n]

The %0A at the end of the URL triggered the problem. I guess all you need to do is to call url.trim() to make the new connection dialog a bit more robust against user errors.

Thanks! Florian Gnägi http://www.frentix.com

Change History (1)

comment:1 Changed on Dec 16, 2008 at 5:22:52 PM by dkocher

  • Milestone set to 3.1
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r4373.

Note: See TracTickets for help on using tickets.
swiss made software