gnodet-bot commented on code in PR #29855: URL: https://github.com/apache/camel/pull/29855#discussion_r4564965599
########## components/camel-file/src/main/docs/file-component.adoc: ########## @@ -719,6 +719,10 @@ done and ready to be consumed. With `jailStartingDirectory` enabled (the default), symbolic links are resolved when checking that a file stays within the starting directory. The consumer skips a file, or with `recursive=true` a directory, that is a link resolving outside the starting directory (logged at WARN the first time), and the producer fails to write through such a link. +The consumer also checks the target of `move`, `preMove` and `moveFailed`, including the default `.camel` folder: when +that target is inside the starting directory but resolves outside of it through a link, the move fails like any other +failed move and the file stays where it is. A target configured outside the starting directory (such as Review Comment: 📝 **Doc nit:** This sentence runs 42 words (the project targets ~25-word sentences for readability). Consider splitting: ```suggestion The consumer also checks the target of `move`, `preMove` and `moveFailed`, including the default `.camel` folder. When that target is inside the starting directory but resolves outside of it through a link, the move fails like any other failed move and the file stays where it is. A target configured outside the starting directory (such as ``` -- 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]
