[CAMEL-19615] camel-ftp: chmodDirectory option try for each junk to change the directory permission and fails

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.18.2
• Fix Version/s: 3.14.10, 3.20.7, 3.21.1, 3.22.0, 4.0-RC2, 4.0.0
• Component/s: camel-sftp
• Estimated Complexity: Unknown
• Assignee: Unassigned
• Reporter: Michael Rambichler
• Created: 19/Jul/23 07:42
• Updated: 20/Jul/23 07:10
• Resolved: 19/Jul/23 16:28

Description

We have a blocking issue with the current chmodDirectory option.

https://camel.apache.org/components/3.20.x/sftp-component.html#_endpoint_query_option_chmodDirectory

 

Steps to reproduce:

SFTP Directory: ../home/user/restricted/dir/subdir (where restricted e.g. does not have change rights for the current user)

Component Call:
"sftp://localhost:21/home/user/restricted/dir/subdir/newDir" +
"?username=admin&password=admin&chmod=777&chmodDirectory=770"
 
Behaviour:
The current implementation steps down the directory and try to create each junk. If the chmodDirectory option is set then it will try for each (already existing junk) to change the directory permission and fails.
 
Expected Behaviour:
It should only try to change the permission if the directory does not yet exists
 
I have already created a fix and added a Testcase for better debugging (but the Testcase does not reproduce the error because it would remove the permission for the current user and afterwards your need root rights to delete the Testdirectory again. Not suitable pipeline builds)
https://github.com/michael-salzburg/camel
 

Issue Links

• links to: GitHub Pull Request #10730