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

Logrotate fix #2263

Closed
wants to merge 8 commits into from
Closed

Logrotate fix #2263

wants to merge 8 commits into from

Conversation

syspect-sec
Copy link

What

This will add proper permissions to the logrotate configuration file, which is placed into the /etc/logrotate.d/gvmd during the make build process.

Why

It was pointed out from this forum post that the logs do not have the correct permission configuration for logrotate.

References

Forum post with discussion

Checklist

  • [ *] Tests - Tested and works on Ubuntu 22.04

syspect-tech and others added 8 commits June 15, 2023 07:34
Fixed issue in .deb credential script that throws an error when running script twice for a username to replace a credential
Fixing issues with tar by correcting paths required to create the data archive before compiling the .deb
Fixing a bug in the EXE credential creator resulting from syntax error in the ExecWait commands.
Setting  "_topdir" in the rpmbuild config is required to prevent the rpmbuild process from attempting to access the gvm user's home directory, (which does not exist) resulting in a 0-byte user credential creator .rpm package.
@syspect-sec syspect-sec requested a review from a team as a code owner July 18, 2024 10:53
Copy link

Conventional Commits Report

😢 No conventional commits found.

👉 Learn more about the conventional commits usage at Greenbone.

@cfi-gb
Copy link
Member

cfi-gb commented Jul 23, 2024

Seems there is a similar PR already open since quite some time: #2051

@mattmundell
Copy link
Contributor

Moved to a single commit in /pull/2269.

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.

4 participants