Skip to content

"File chmod" is not working correctly. #507

Closed
@wiinxt

Description

Describe the bug
The "File chmod" option is not working because fs.chmodSync mode parameter needs an octal and not a decimal. The default value 775 in decimal gives 1407 in octal, so chmod sets -r-----rwt instead of -rwxrwxr-x.

The error is here:

fs.chmodSync(fullpath, Config.getInstance().cfg("file_chmod"));

Node.js doc of fs.chmod: https://nodejs.org/api/fs.html#fschmodpath-mode-callback

I suggest this change to fix the error (convert the number to a string, then parse it as octal) :

fs.chmodSync(fullpath, parseInt(Config.getInstance().cfg("file_chmod").toString(), 8)); 

To Reproduce

  1. Go to Advanced options
  2. Click on Set file permissions
  3. The permissions on files are not 775 (-r-----rwt instead of -rwxrwxr-x)

Expected behavior
The permission must be 775 (-rwxrwxr-x)

Server:

  • Docker

Workaround
I set 509 (which is 775 in octal) for now, it works well.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions