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

Fix #44313 bitnami/suitecrm:7 fails to come up due to group ownership set to root #44315

Merged
merged 2 commits into from
Aug 16, 2023

Conversation

chris001
Copy link
Contributor

@chris001 chris001 commented Aug 11, 2023

Description of the change

Fix the group ownership of all files and folders in the web app bitnami/suitecrm:7

Benefits

Allows the web app to run. Without my change, the app failed and exited.

Possible drawbacks

None.

Applicable issues

Additional information

…ownership set to `root`.

Signed-off-by: Chris Coleman <colemanchris@hotmail.com>
@github-actions github-actions bot added the triage Triage is needed label Aug 11, 2023
@carrodher carrodher changed the title Fix #44313 bitnami/suitecrm:7 fails to come up due to group ownersh… Fix #44313 bitnami/suitecrm:7 fails to come up due to group ownership set to root Aug 12, 2023
@carrodher carrodher added the verify Execute verification workflow for these changes label Aug 12, 2023
@github-actions github-actions bot added in-progress and removed triage Triage is needed labels Aug 12, 2023
@bitnami-bot bitnami-bot removed the request for review from carrodher August 12, 2023 16:44
Copy link
Contributor

@jotamartos jotamartos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution. Could you please take a look at my comment?

…rm/postunpack.sh


Use `WEB_SERVER_DAEMON_GROUP` for group ownership, instead of `WEB_SERVER_DAEMON_USER`, and instead of `root`.

Co-authored-by: Juan José Martos <jotamartos@gmail.com>
Signed-off-by: Chris Coleman <colemanchris@hotmail.com>
@jotamartos jotamartos merged commit 4ec0d0a into bitnami:main Aug 16, 2023
7 checks passed
@jotamartos
Copy link
Contributor

Version 7.13.4-debian-11-r31 includes your changes. Thanks for your contribution.

@chris001 chris001 deleted the chris001-patch-1 branch August 16, 2023 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved suitecrm verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SuiteCRM v7 Fails to work due to incorrect group ownership in postunpack.sh
3 participants