zhangshenghang opened a new pull request, #11078:
URL: https://github.com/apache/seatunnel/pull/11078

   ### Purpose
   
   Fixes #5516.
   
   The SFTP filesystem layer already supports `fs.sftp.keyfile`, but the SFTP 
connector did not expose a connector-level option for it. As a result, users 
could not configure public key authentication through normal `SftpFile` source 
or sink options.
   
   ### Changes
   
   - Add a `keyfile` option to the SFTP connector options.
   - Pass `keyfile` through `SftpConf` to the existing 
`SFTPFileSystem.FS_SFTP_KEYFILE` setting.
   - Keep password authentication supported while allowing password to be 
omitted when keyfile authentication is used.
   - Add unit coverage for option exposure and Hadoop config forwarding.
   - Update English and Chinese SFTP source/sink documentation.
   
   ### Tests
   
   - `JAVA_HOME=/Library/Java/JavaVirtualMachines/temurin-17.jdk/Contents/Home 
./mvnw spotless:apply`
   - `JAVA_HOME=/Library/Java/JavaVirtualMachines/temurin-17.jdk/Contents/Home 
./mvnw -q -DskipTests verify`
   - `JAVA_HOME=/Library/Java/JavaVirtualMachines/temurin-17.jdk/Contents/Home 
./mvnw -pl seatunnel-connectors-v2/connector-file/connector-file-sftp -DskipIT 
-Dskip.spotless=true -Dtest=SftpFileFactoryTest test`


-- 
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]

Reply via email to