Samin061 commented on issue #69562:
URL: https://github.com/apache/airflow/issues/69562#issuecomment-4922632848

   Tested #68984 on `apache-airflow-providers-amazon==9.32.0rc1` (installed 
from the PyPI RC, confirmed the module resolves to site-packages at 
`9.32.0rc1`, not local sources).
   
   The path-traversal guard ships in the RC: both `S3ToSFTPOperator` and 
`S3ToFTPOperator` call `validate_destination_path` before writing. 
Attacker-controllable S3 key suffixes with `..` segments (e.g. 
`/srv/sftp/incoming/../../../../etc/cron.d/evil`, 
`/var/ftp/out/../../etc/passwd`) now raise `ValueError` instead of escaping the 
configured `sftp_path`/`ftp_path`, while benign nested suffixes like 
`sub/dir/report.csv` are still allowed. Working as expected.


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