-
-
Notifications
You must be signed in to change notification settings - Fork 304
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
Improve documentation for the --header parameter #1389
Conversation
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.
Great stuff! Now that I think about it, we could make even improve that. I left a comment.
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.
Thank you for the review. Here are the updates made:
-
Improved Documentation: Enhanced clarity on
--header
usage, particularly regarding how conflicts with existing headers are handled. -
HSTS Example: Added an example in the README for enabling HSTS using
--header
, essential for achieving an A+ rating on SSL Labs. -
Fullchain Certificates Usage: Documented the use of fullchain certificates with the
--tls-cert
option to improve HTTPS security.
These changes aim to clarify usage and enhance security features. Please let me know if additional modifications are necessary.
Thanks! |
Glad to contribute! And it means a lot to me! |
A more detailed explanation of the use of the parameter --header