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

["Michael Rambichler (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CAMEL-19615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17744565#comment-17744565 ] 

Michael Rambichler edited comment on CAMEL-19615 at 7/19/23 1:16 PM:
---------------------------------------------------------------------

PR created: [https://github.com/apache/camel/pull/10730]

 


was (Author: rambichler):
Pullrequest created: [https://github.com/apache/camel/pull/10727]

> camel-ftp: chmodDirectory option try for each junk to change the directory permission and fails
> -----------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-19615
>                 URL: https://issues.apache.org/jira/browse/CAMEL-19615
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-sftp
>    Affects Versions: 3.18.2
>            Reporter: Michael Rambichler
>            Priority: Major
>
> 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 i{color:#FF0000}t will try for each (already existing junk) to change the directory permission and fails.{color}
>  
> *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]
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)