[Tickets #14579] Gollem not compatible with FTP passwords that contain an @

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: https://bugs.horde.org/ticket/14579
------------------------------------------------------------------------------
  Ticket             | 14579
  Created By         | aaronp at critd.com
  Summary            | Gollem not compatible with FTP passwords that contain
                     | an @
  Queue              | Gollem
  Version            | FRAMEWORK_5_1
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


aaronp at critd.com (2017-02-14 17:23) wrote:

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.