Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix file upload permissions #111

Closed
kesara opened this issue Jul 8, 2021 · 3 comments
Closed

Fix file upload permissions #111

kesara opened this issue Jul 8, 2021 · 3 comments
Assignees
@kesara
Labels
internal Fix internally

Comments

@kesara
Copy link
Member

kesara commented Jul 8, 2021

  • Set FILE_UPLOAD_PERMISSIONS to 0o664.
  • Passuid: 30, gid: 8 at the build time so gunicorn runs as that. (These uid, gid will have to be created on the docker instance.)
@kesara kesara added the internal Fix internally label Jul 8, 2021
@kesara
Copy link
Member Author

kesara commented Jul 12, 2021

GID:8 is used by mem group in AlpineLinux: https://git.alpinelinux.org/alpine-baselayout/tree/group

@kesara
Copy link
Member Author

kesara commented Jul 12, 2021 •
edited
Loading

UID:30 seems to be not in use: https://git.alpinelinux.org/alpine-baselayout/tree/passwd

@kesara kesara self-assigned this Jul 12, 2021
This was referenced Jul 15, 2021
Merged
Merged
@kesara
Copy link
Member Author

kesara commented Aug 9, 2021

Released with v1.2.0.

@kesara kesara closed this as completed Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kesara kesara

Labels
internal Fix internally
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kesara