gnodet opened a new pull request, #21950: URL: https://github.com/apache/camel/pull/21950
## Summary - **Commit 1**: Create `camel-ftp-common` module with 14 shared classes moved from `camel-ftp` (RemoteFile*, FtpConstants, FtpUtils, SftpRemoteFile*, SftpDefaultMoveExistingFileStrategy). Delete 2 duplicate classes from `camel-mina-sftp`. Update dependency graph and BOMs. - **Commit 2**: Extract `AbstractSftpConsumer` into `camel-ftp-common` with shared polling/directory traversal logic. Both `SftpConsumer` and `MinaSftpConsumer` become thin subclasses (~80 lines each vs ~350 before). - **Commit 3**: Extract `BaseSftpConfiguration` into `camel-ftp-common` with ~20 shared SFTP config fields (known hosts, private key, server alive, chmod, compression, etc.). `SftpConfiguration` and `MinaSftpConfiguration` keep only implementation-specific fields. ## Test plan - [x] `camel-ftp-common`, `camel-ftp`, and `camel-mina-sftp` build successfully - [ ] Run `camel-ftp` unit tests: `mvn test -pl components/camel-ftp` - [ ] Run `camel-mina-sftp` unit tests: `mvn test -pl components/camel-mina-sftp` - [ ] Verify no regression in SFTP consumer behavior -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
