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.

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

Allowing opening shell as root / sudo user #112

Closed
4 tasks done
joehoyle opened this issue Jan 10, 2020 · 1 comment · Fixed by #652
Closed
4 tasks done

Allowing opening shell as root / sudo user #112

joehoyle opened this issue Jan 10, 2020 · 1 comment · Fixed by #652
Assignees
Labels
could have Could be done, or nice to have, low priority for now developer advocacy Developer Advocacy related tasks

Comments

@joehoyle
Copy link
Member

joehoyle commented Jan 10, 2020

To allow people to debug / install packages, they need a root permissions level user in the shell command. Passing -u root to the command would suffice.

Acceptance criteria:

  • Add an option flag to the shell command called --root, optional, alias as -r
  • When flag is set add -u root to the docker exec call
  • Update the commands documentation with this argument
  • Add a note to the command docs on why you would use the root user:
    • To test modifications to the PHP container eg. installing PHP libraries or other packages
    • To carry out any other tasks that require sudo privileges such as making directories outside of the app directory
@joehoyle joehoyle added the good first issue Good for newcomers label Jan 10, 2020
@joehoyle joehoyle added this to the 4.0 milestone Jan 10, 2020
@roborourke roborourke added enhancement and removed good first issue Good for newcomers labels Jan 29, 2020
@rmccue rmccue modified the milestones: 4.0, 5.0 Jun 15, 2020
@rmccue rmccue removed this from the 5.0 milestone Sep 2, 2020
@rmccue rmccue added the could have Could be done, or nice to have, low priority for now label Feb 9, 2021
@missjwo missjwo added the developer advocacy Developer Advocacy related tasks label Dec 16, 2024
@mikelittle mikelittle self-assigned this Jan 28, 2025
@mikelittle
Copy link
Contributor

Note: I don't think there is any point adding sudo to the container as suggested in the PR as the www-data user doesn't have a password.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
could have Could be done, or nice to have, low priority for now developer advocacy Developer Advocacy related tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants