Skip to content
This repository has been archived by the owner on Jun 4, 2021. It is now read-only.

Add a simple /healthz path on the Nginx frontend #1557

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

alexmbird
Copy link

This adds a simple, unauthenticated /healthz path which can be used to monitor the availability of a Streisand box using a service like uptime robot. It returns a HTTP 200 message with body ok. The body is intentionally nondescript so as not to expose the fact that the host runs Streisand to unauthenticated visitors.

…d. This can be used for remotely monitoring the uptime of a Streisand box.
@alexmbird alexmbird changed the title Add a simple /healthz path to the frontend without basic auth enforced. This can be used for remotely monitoring the uptime of a Streisand box. Add a simple /healthz path on the Nginx frontend Apr 7, 2019
@nopdotcom
Copy link
Member

I think this would have to be optional. A lot of people are paranoid.

Have you considered using https://streisand.example/robots.txt, delivering this file?

User-agent: *
Disallow: /

@nopdotcom nopdotcom added the status/revisions-needed For items with outstanding requested changes label Aug 7, 2019
@alexmbird
Copy link
Author

@nopdotcom I've spruced up this PR so that:

  1. It serves a robots.txt (which is a great idea, it's more standard than a /healthz and just as useful)
  2. Makes that optional with a default of no. I've rolled out a couple of Streisand instances to test this works as expected.

Let me know if you'd like to see any other changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status/revisions-needed For items with outstanding requested changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants