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

Allow to read logs by root #41

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Allow to read logs by root #41

merged 1 commit into from
Feb 6, 2024

Conversation

Mehdi-Bendriss
Copy link
Contributor

This PR:

  • allows the root user to read logs from ${SNAP_COMMON}/var/log/
    • this is done through the mongod parameter processUmask=037 set on start
  • ensures that every new file created in ${SNAP_COMMON}/var/log/ are created with the root group.
    • this is done through chgrp + chmod g+s

MiaAltieri
MiaAltieri reviewed Feb 6, 2024
View reviewed changes
snap/hooks/install Show resolved Hide resolved
snap/local/start-mongod.sh Show resolved Hide resolved
@Mehdi-Bendriss Mehdi-Bendriss merged commit 81ab09c into 6/edge Feb 6, 2024
3 checks passed
@Mehdi-Bendriss Mehdi-Bendriss deleted the allow-read-logs-with-root branch February 6, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MiaAltieri MiaAltieri MiaAltieri approved these changes

@delgod delgod delgod approved these changes

@dmitry-ratushnyy dmitry-ratushnyy Awaiting requested review from dmitry-ratushnyy

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Mehdi-Bendriss @delgod @MiaAltieri