This is an automated email from the ASF dual-hosted git repository.

pcongiusti pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new 2a13f3097352 chore(components): file comment to clarify permissions 
intent
2a13f3097352 is described below

commit 2a13f3097352f291e516bddfe50899b92ae55bfd
Author: Pasquale Congiusti <[email protected]>
AuthorDate: Fri Dec 5 11:54:01 2025 +0100

    chore(components): file comment to clarify permissions intent
---
 .../java/org/apache/camel/component/file/FileEndpoint.java | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git 
a/components/camel-file/src/main/java/org/apache/camel/component/file/FileEndpoint.java
 
b/components/camel-file/src/main/java/org/apache/camel/component/file/FileEndpoint.java
index 0769ca715f2d..10223271a330 100644
--- 
a/components/camel-file/src/main/java/org/apache/camel/component/file/FileEndpoint.java
+++ 
b/components/camel-file/src/main/java/org/apache/camel/component/file/FileEndpoint.java
@@ -447,14 +447,15 @@ public class FileEndpoint extends 
GenericFileEndpoint<File> {
             permissions.add(PosixFilePermission.GROUP_EXECUTE);
         }
 
+        // NOTE: the user is explicitly setting the permissions
         if ((othersValue & CHMOD_WRITE_MASK) > 0) {
-            permissions.add(PosixFilePermission.OTHERS_WRITE);
+            permissions.add(PosixFilePermission.OTHERS_WRITE); // NOSONAR
         }
         if ((othersValue & CHMOD_READ_MASK) > 0) {
-            permissions.add(PosixFilePermission.OTHERS_READ);
+            permissions.add(PosixFilePermission.OTHERS_READ); // NOSONAR
         }
         if ((othersValue & CHMOD_EXECUTE_MASK) > 0) {
-            permissions.add(PosixFilePermission.OTHERS_EXECUTE);
+            permissions.add(PosixFilePermission.OTHERS_EXECUTE); // NOSONAR
         }
 
         return permissions;
@@ -509,14 +510,15 @@ public class FileEndpoint extends 
GenericFileEndpoint<File> {
             permissions.add(PosixFilePermission.GROUP_EXECUTE);
         }
 
+        // NOTE: the user is explicitly setting the permissions
         if ((othersValue & CHMOD_WRITE_MASK) > 0) {
-            permissions.add(PosixFilePermission.OTHERS_WRITE);
+            permissions.add(PosixFilePermission.OTHERS_WRITE); // NOSONAR
         }
         if ((othersValue & CHMOD_READ_MASK) > 0) {
-            permissions.add(PosixFilePermission.OTHERS_READ);
+            permissions.add(PosixFilePermission.OTHERS_READ); // NOSONAR
         }
         if ((othersValue & CHMOD_EXECUTE_MASK) > 0) {
-            permissions.add(PosixFilePermission.OTHERS_EXECUTE);
+            permissions.add(PosixFilePermission.OTHERS_EXECUTE); // NOSONAR
         }
 
         return permissions;

Reply via email to