Uploaded image for project: 'Commons VFS'
  1. Commons VFS
  2. VFS-563

[sftp][tests] test sshd does not implement setstats and test does not verify it

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.1
    • Fix Version/s: None
    • Labels:
    • Environment:

      Maven 3.2.5, Java 7u76, Windows 7 (x64, german)

      Description

      While trying some test cases with external servers I noticed that LastModfifiedTest is calling setLastModified and getLastModified and then compares the delta. However since the default cache mode is on resolve and there is no resolve or refresh in between, it only compares the stale in memory representation.

      So I went on and added refresh() to the actual checks. This made the sftp file system fail. After some checking I noticed, that the Apache Mina SSHD (0.8) which we are using is actually not implementing it.

      org.apache.sshd.server.sftp.SftpSubsystem

      {code }

      case SSH_FXP_SETSTAT:
      case SSH_FXP_FSETSTAT:

      { // This is required for WinSCP / Cyberduck to upload properly // Blindly reply "OK" // TODO implement it sendStatus(id, SSH_FX_OK, ""); break; }
      
      

      I think newer mina versions do support this function, so we need to migrate the internal test server to this version before we can actually enable the refresh().

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              b.eckenfels Bernd Eckenfels
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: