Skip to content

Allow read/write to user root and only read to group git on documenta… #15041

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.

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

Merged
merged 2 commits into from
Mar 19, 2021
Merged

Allow read/write to user root and only read to group git on documenta… #15041

merged 2 commits into from
Mar 19, 2021

Conversation

bitmand
Copy link
Contributor

@bitmand bitmand commented Mar 19, 2021

The Installation from binary documentation has this section:

If you don’t want the web installer to be able to write the config file at all, it is also possible to make the config file read-only for the gitea user (owner/group root:root, mode 0660), and set INSTALL_LOCK = true...

But setting app.ini to root:root and 0660 will only allow root user to read it and gitea can't read app.ini and fails to start.

This MR changes the documentation to recommending:

(owner/group root:git, mode 0640), and set INSTALL_LOCK = true

Allow root user to read/write and users in git group read-only access.

@6543 6543 added the type/docs This PR mainly updates/creates documentation label Mar 19, 2021
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 19, 2021
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 19, 2021
@6543
Copy link
Member

6543 commented Mar 19, 2021

🚀

@6543 6543 merged commit 0fe3282 into go-gitea:master Mar 19, 2021
@go-gitea go-gitea locked and limited conversation to collaborators May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants