-
-
Notifications
You must be signed in to change notification settings - Fork 881
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
Various updates #1250
Various updates #1250
Conversation
The template already allowed setting it to false. This completes it.
eba889b
to
c33d62e
Compare
This ensures there are no empty lines in the file.
This aligns error_log and access_log
This double spacing doesn't align to anything so it doesn't make sense.
This mitigates DSA-3701 / CVE-2016-1247 and matches the default setup on Debian. https://www.debian.org/security/2016/dsa-3701 https://security-tracker.debian.org/tracker/CVE-2016-1247
‘Various Updates’ is going to be too vague for the changelog. Could you update the PR title or split this into separate PRs? |
As you can see the commits are cleanly split with the intention of separating. I just wanted to push my changes so they weren't just on my desktop :) |
@ekohl I figured that was probably the case and you just wanted to show off what you'd been up to. :) |
Something like that, something like a backup :) |
See individual commits: