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

Change install log location and restrict permissions #74

Merged
merged 21 commits into from
Nov 22, 2022

Conversation

Dean-Coakley
Copy link
Contributor

@Dean-Coakley Dean-Coakley commented Oct 13, 2022

Proposed changes

  • Change install log location from /tmp/ to /var/log/nginx-agent/
  • Restrict log file permissions for unprivileged user
  • Add install/uninstall CI stage Removed. I recommend adding it back post-open sourcing so it doesn't depend on keys
  • Postinstall script output is now appended to the install log
  • Suppress excessive stdout output.
  • FreeBSD now include preinstall step
  • FreeBSD package now creates /var/log/nginx-agent on install instead of on runtime
  • FreeBSD no longer errors on uninstall

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • If applicable, I have checked that any relevant tests pass after adding my changes
  • I have updated any relevant documentation (README.md)

@Dean-Coakley Dean-Coakley merged commit d2133a2 into main Nov 22, 2022
@Dean-Coakley Dean-Coakley deleted the fix-logfile-installlog branch November 22, 2022 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants