Inconsistent behaviour between listing and downloading the files.

[SimoneGiusso]

In what version(s) of Spring Integration are you seeing this issue?

6.2.4

Describe the bug

/ at the beginning of the remote dir path is not necessary when listing files although it is necessary to download them

To Reproduce

    @Test
    fun testFileDownload() {
        sftpTemplate.list("remote-dir").forEach { // All the files under remote-dir are listed
            log.info { it.filename }
        }

        log.info { "---- FINISHED TO LIST DIRECTORIES -----\n\n\n\n" }

        sftpTemplate.get("/remote-dir/MyFile.csv") { // remote-dir/MyFile.csv won't work
            inputStream -> inputStream.use {
                localDir.mkdir()
                FileUtils.copyInputStreamToFile(inputStream, File(localDir.absolutePath + "test.csv"))
        }}
    }

Expected behavior

sftpTemplate.get should work also with "remote-dir/MyFile.csv" as input.

See also stackOverflow question.

[artembilan]

For LIST command we do this:

		remoteDir =
				!remoteDir.isEmpty() && remoteDir.charAt(0) == '/'
						? remoteDir
						: this.sftpClient.canonicalPath(remoteDir);

For GET - do not:

	public InputStream readRaw(String source) throws IOException {
		return this.sftpClient.read(source);
	}

So, maybe this is a problem and we can try to normalize path if it does not start with /?

[SimoneGiusso]

Yes, both commands should threat the dirPath in the same way.

Here probably the GET should normalize the path as the LIST command does.

I found unexpected the fact that one command without "/" does not work while it works for the other.

[SimoneGiusso]

Additionally, in the message's header file_remoteDirectory, two '/' are present before the remote repository name

[artembilan]

@SimoneGiusso , do you mean for the SftpOutboundGateway with a Command.GET ?
Or what part of the flow you are talking about which produces the Message for you?