FtpServer
  1. FtpServer
  2. FTPSERVER-247

Add DataConnectionConfiguration.isImplicitSsl

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.0-RC1
    • Component/s: Core, Server
    • Labels:
      None

      Description

      It would be extremely helpful to be able to force the data channel to use SSL when in "implicit SSL" mode.

      When using implicit SSL, some FTP clients do not send a PROT command to change the data channel protection level yet they assume that the data channel will be encrypted.

      FtpServer assumes that the data channel will be un-encrypted. It would be nice to have an option to force the data channel to be encrypted to accommodate these clients.

        Activity

        Kevin Conaway created issue -
        Hide
        Kevin Conaway added a comment -

        Attaching patch

        Show
        Kevin Conaway added a comment - Attaching patch
        Kevin Conaway made changes -
        Field Original Value New Value
        Attachment ftpserver-237.patch [ 12396610 ]
        Kevin Conaway made changes -
        Summary Add DataConnectionConfiguration.isSslRequired Add DataConnectionConfiguration.isImplicitSsl
        Kevin Conaway made changes -
        Attachment ftpserver-237.patch [ 12396610 ]
        Hide
        Kevin Conaway added a comment -

        Attaching updated patch to add the "implicitSsl" property to the DataConnectionConfiguration along with updates to the XML configuration and associated test cases.

        Show
        Kevin Conaway added a comment - Attaching updated patch to add the "implicitSsl" property to the DataConnectionConfiguration along with updates to the XML configuration and associated test cases.
        Kevin Conaway made changes -
        Attachment ftpserver-247.patch [ 12396694 ]
        Hide
        Niklas Gustavsson added a comment -

        Modified patch applied in rev 729474. Thanks for the great work!. My modifications includes some additional tests.

        Show
        Niklas Gustavsson added a comment - Modified patch applied in rev 729474. Thanks for the great work!. My modifications includes some additional tests.
        Niklas Gustavsson made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 1.0.0-RC1 [ 12313542 ]
        Assignee Niklas Gustavsson [ niklas ]
        Status Open [ 1 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        3d 17h 1 Niklas Gustavsson 26/Dec/08 10:09

          People

          • Assignee:
            Niklas Gustavsson
            Reporter:
            Kevin Conaway
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development