Uploaded image for project: 'OODT (Retired)'
  1. OODT (Retired)
  2. OODT-659

CommonsNetFtpProtocol.ls() method is causing unnecessary traffic

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 0.6
    • 0.7
    • push pull framework
    • None
    • *nix

    • Committer (Medium) - This is for regular contributors/committers

    Description

      While trying to download files from an FTP server with 6400+ files on it pushpull would crash with a 421 Error from the FTP server.

      After a lot of research and testing we found that the code was calling pwd() for each file when building up the list within the ls() method.

      In the case of trying to scan a directory with a large number of files, this can make the FTP server drop the connection due to the large number of request.

      We moved the call to pwd outside of the loop, and everything works great. We are able to download the files easily now.

      Attachments

        Activity

          People

            cgoodale Cameron Goodale
            cgoodale Cameron Goodale
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: