-
Notifications
You must be signed in to change notification settings - Fork 169
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
docs: emphasize that port 80 needs to be reachable for lets encrypt #5
Conversation
docs/server/tutorials/tlsssl.md
Outdated
- Your `appURL` site configuration option begins with `https://...`. | ||
- The host is reachable on port `80` and port `443`. | ||
|
||
- 🚨 [Lets Encrypt requires that port `80` be reachable in order prove that you own your domain](https://letsencrypt.readthedocs.io/en/latest/challenges.html#http-01-challenge). If port `80` is unreachable, HTTPS will fail with errors such as the following: |
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.
I think it would be better to put this warning + the errors at the end of the section here. Otherwise, it is hard to read the bullet points here about if the following conditions are met:
without losing your train of thought and/or not realizing more bullets come after.
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.
.
@slimsag PTAL, updated |
add clarity to professional development stipend and educational travel clause #5
* Update travel.md add clarity to professional development stipend and educational travel clause #5 * squash! Prettier Co-authored-by: cassiemelani <cassiemelani@users.noreply.github.com>
Fixes: sourcegraph/sourcegraph#252