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

Package installation creates database.yml as root and thus breaks the installation until next update #3834

Closed
MrGeneration opened this issue Nov 4, 2021 · 0 comments

Comments

@MrGeneration
Copy link
Member

Infos:

Expected behavior:

Zammads installation processes set correct file permissions to ensure everything works.

Actual behavior:

During installation, since issue #3828 and the chmod 600 for the database.yml fresh installations break.
The reason for this is the postinstall.sh script to not run a chown on the file in question.

Here's proof that it's always been this way but wasn't discovered "thanks to" too open permissions:

root@debian-verify:~# cat /opt/zammad/VERSION
4.1.x
root@debian-verify:~# ls -la /opt/zammad/config/database.yml
-rw-r--r-- 1 root root 139 Nov  4 11:12 /opt/zammad/config/database.yml

Technically any update or re-installation over the existing installation would solve the issue as our packages enforce a chown -R zammad.zammad upon every installation.

After upgrade:

root@debian-verify:~# cat /opt/zammad/VERSION
5.0.x
root@debian-verify:~# ls -la /opt/zammad/config/database.yml
-rw------- 1 zammad zammad 139 Nov  4 11:12 /opt/zammad/config/database.yml

Steps to reproduce the behavior:

  • Install Zammad 5.0.2 as of 3rd or 4th November before 12pm german time

Yes I'm sure this is a bug and no feature request or a general question.

@MrGeneration MrGeneration added this to the 5.0.3 milestone Nov 4, 2021
@MrGeneration MrGeneration self-assigned this Nov 4, 2021
@MrGeneration MrGeneration added this to In progress in OLD Workflow Nov 4, 2021
@MrGeneration MrGeneration moved this from In progress to Review in OLD Workflow Nov 4, 2021
@MrGeneration MrGeneration moved this from Review to Backporting to stable in OLD Workflow Nov 4, 2021
zammad-sync pushed a commit that referenced this issue Nov 4, 2021
…hus breaks the installation until next update
@MrGeneration MrGeneration moved this from Backporting to stable to Done in OLD Workflow Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
OLD Workflow
  
Done
Development

No branches or pull requests

1 participant