abrt / retrace-server

Application for remote coredump analysis
GNU General Public License v2.0
40 stars 30 forks source link

NEEDINFO: spec: Set ownership and permission for logotate.d #406

Closed mzidek-gh closed 3 years ago

mzidek-gh commented 3 years ago

According to a comment in: https://github.com/abrt/retrace-server/issues/403

We should change the ownsership to our logrotate config file.

This change will result in both retrace and root (instead of just root) to be able to modify the config file.

I am not sure why retrace user needs to be able to do this. Usually configuration is only modofied by a user with root priviledges. Can someone clarify?

packit-as-a-service[bot] commented 3 years ago

Congratulations! One of the builds has completed. :champagne:

:warning: Please note that our current plans include removal of these comments in the near future (at least 2 weeks after including this disclaimer), if you have serious concerns regarding their removal or would like to continue receiving them please reach out to us. :warning:

You can install the built RPMs by following these steps:

Please note that the RPMs should be used only in a testing environment.

mzidek-gh commented 3 years ago

Ok, I completely misunderstood the issue. I will update the patch. Thanks for clarification.

mzidek-gh commented 3 years ago

I have merged this PR with https://github.com/abrt/retrace-server/pull/405

Closing.