-
Notifications
You must be signed in to change notification settings - Fork 26
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
Use sdconfig instead of manually editing the site-specific file in the OSSEC guide #560
Conversation
This commit minizes the use of manually editing the site-specific file, instead preferring the use of `securedrop-admin sdconfig` where applicable. As a result, this commit also restructures the flow of the OSSEC Guide, particularly during the sections related to installing SecureDrop.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks great, left a couple of nits :)
Thanks for the review, @eloquence! I believe I was able to address all the suggestions, but please let me know if I need to tweak anything, or if you spot anything else. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Status
Ready for review
Description of Changes
This PR minimizes the use of manually editing the
site-specific
file, instead preferring the use ofsecuredrop-admin sdconfig
where applicable.As a result, this PR also restructures the flow of the OSSEC Guide, particularly during the sections related to installing SecureDrop.
site-specific
manually #556Testing
Checklist (Optional)
make docs-lint
) passed locallymake docs-linkcheck
) passedmake docs
) docs at http://localhost:8000