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

Use tar with docker cp to fix various file permissions issues. #62

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jaklaassen-affirm
Copy link

Summary

Fixes various file permissions issues by using tar with docker cp to copy files as the sonar user.

Other Information

I believe this change is needed when using a custom workdir, but I also ran into permissions issues when using the default workdir. In my setup, the docker cp appears to overwrite file permissions with root, even though chown is run in the Dockerfile.

PR Checklist

Please run through before submitting for final review:

  • The PR title is descriptive of what changed.

  • The description sections above are filled out.

  • The CI pipeline is passing

  • The plugin version has been bumped (using SemVer)

    The plugin version should be bumped both in .env and in the README. The plugin version is listed multiple times in the readme. Be sure to do a find/replace to ensure you haven't missed any!

  • Added the ready-for-review label to the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant