GH-11121: Fix SftpSession.listNames() for root directory wildcard paths

[yangadam]

Summary

SftpSession.listNames("/*") throws SftpException: SSH_FX_NO_SUCH_FILE because the path parsing in SftpSession.list() uses lastIndex > 0 to detect the / separator. For root-level paths like /*, lastIndexOf('/') returns 0, which is excluded by the > 0 check. This causes the entire path /* to be treated as a literal directory name instead of being split into directory / and file pattern *.

Changes

• Changed lastIndex > 0 to lastIndex >= 0 in SftpSession.list() so root directory paths are correctly parsed
• Added test lsRootWildcard() that reproduces the issue and verifies the fix

Fixes: gh-11121

Signed-off-by: Adam Yang yangadam@users.noreply.github.com

[artembilan]

Well, looks like you have not checked the fix locally before opening PR:

SftpOutboundTests > testFtpOutboundGatewayInsideChain() FAILED
    org.springframework.messaging.MessageHandlingException at SftpOutboundTests.java:192
        Caused by: org.springframework.messaging.MessagingException at SftpOutboundTests.java:192
            Caused by: java.lang.NullPointerException at SftpOutboundTests.java:192

Please, ensure you run ./gradlew "spring-integration-sftp:check locally before pushing to PR.

Thanks

[yangadam]

Well, looks like you have not checked the fix locally before opening PR:

SftpOutboundTests > testFtpOutboundGatewayInsideChain() FAILED
    org.springframework.messaging.MessageHandlingException at SftpOutboundTests.java:192
        Caused by: org.springframework.messaging.MessagingException at SftpOutboundTests.java:192
            Caused by: java.lang.NullPointerException at SftpOutboundTests.java:192

Please, ensure you run ./gradlew "spring-integration-sftp:check locally before pushing to PR.

Thanks

Thanks for running the build. After debugging, I found that the failure was caused by the lack of mocking for the stat method, which previously was not called when the path was /remote-test-dir/.

I can add the mocking, but I need to confirm whether it is acceptable to call stat whenever the path is /remote-test-dir/ or /remote-test-dir/remote-sub-dir/.

@artembilan