• Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.2
    • Component/s: FTP
    • Labels:
    • Environment:



      Extended Passive Mode (ESPV) is not supported yet in the current library.

      Adding support for it will enable this library to use it to connect ot servers behind routers that are incorrectly configured to use PASV.

      I dont know if I should try to place a patch as I am on doubt on what the right approach should be:

      a) Create a new EXTENDED_PASSIVE_LOCAL_DATA_CONNECTION_MODE connection mode
      This would probably fulfill the actual requirement, but would also require multiple changes across to support the EPSV command and its result as well.

      b) create a public property "ignorePassiveHost" that can be used in FTPClient# __parsePassiveModeReply() as follows

      { __passiveHost = getRemoteAddress().getHostAddress(); }


      { __passiveHost = parts[0] + '.' + parts[1] + '.' + parts[2] + '.' + parts[3]; }

      c) externalize (as a protected method) the call to create socket in FTPClient#openDataConnection to a protected method "createSocket)
      This would allow user code to override the default connection behavior to accomplish a similar functionality.

      d) make __parsePassiveModeReply a protected method and also add a protected method for setting the value on __passiveHost
      This would allow user code to override the default behavior to accomplish a similar functionality.

      Thanks for any comments / work on this.

      I can submit a patch on any of these options if you need me to.


        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        182d 23h 31m 1 Sebb 13/Mar/11 00:26
        Resolved Resolved Closed Closed
        83d 18h 58m 1 Sebb 04/Jun/11 19:25
        Sebb made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Sebb made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.2 [ 12315361 ]
        Resolution Fixed [ 1 ]
        Sebb added a comment -

        Included by NET-288 and NET-313

        Sebb added a comment - Included by NET-288 and NET-313
        Sebb made changes -
        Field Original Value New Value
        Affects Version/s 2.0 [ 12312040 ]
        Affects Version/s 2.1 [ 12313001 ]
        Carlos Silva A. created issue -


          • Assignee:
            Carlos Silva A.
          • Votes:
            2 Vote for this issue
            0 Start watching this issue


            • Created: