6.0.0-git
2019-03-20

[#14579] Gollem not compatible with FTP passwords that contain an @
Summary Gollem not compatible with FTP passwords that contain an @
Queue Gollem
Queue Version FRAMEWORK_5_1
Type Bug
State Not Reproducible
Priority 1. Low
Owners
Requester aaronp (at) critd (dot) com
Created 2017-02-14 (764 days ago)
Due
Updated 2017-06-26 (632 days ago)
Assigned
Resolved 2017-06-26 (632 days ago)
Milestone
Patch No

History
2017-06-26 15:00:51 aaronp (at) critd (dot) com Comment #3 Reply to this comment
Cannot reproduce.
Confirmed. This was definitely not working for me before, and is no 
longer an issue for me now. There must have been some sort of update 
in between then and now that has made this no longer a problem. Thanks 
for your consideration!


2017-06-26 12:27:28 Jan Schneider Comment #2
State ⇒ Not Reproducible
Reply to this comment
Cannot reproduce.
2017-02-14 17:23:38 aaronp (at) critd (dot) com Comment #1
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Summary ⇒ Gollem not compatible with FTP passwords that contain an @
Queue ⇒ Gollem
Milestone ⇒
Patch ⇒ No
Reply to this comment
This can be fixed by urlencode()ing the password first. E.g.:

File: /usr/local/lib/php/Horde/Vfs/Ftp.php, line 202:
         $url .= $this->_params['username'] . ':' . 
urlencode($this->_params['password'])

Although I suspect that other fields might also benefit from 
additional urlencode()ing.

I can verify that this fix works for me.

Saved Queries