Skip to content
This repository has been archived by the owner on Jan 8, 2022. It is now read-only.

simple maintenance page for SDR #487

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

simple maintenance page for SDR #487

wants to merge 2 commits into from

Conversation

peetucket
Copy link
Contributor

@peetucket peetucket commented Oct 15, 2020

Why was this change made?

We need a simple to implement maintenance page for Hydrus that will prevent users from accessing the app. Deploying this branch just temporarily routes every get request to a static HTML page...any posts will result in a 404.

It does this by adding a new static maintenance page into the public folder and then commenting out every route and replacing with a catchall get route to this static page.

How was this change tested?

Localhost

Which documentation and/or configurations were updated?

@jcoyne
Copy link
Contributor

jcoyne commented Oct 16, 2020

Have you considered https://github.com/capistrano/maintenance ?

@peetucket
Copy link
Contributor Author

This is an other option https://github.com/biola/turnout (but it looks to be a bit stale). The reason for opting for this particular solution is that it requires no action on the part of Ops (who seemed reluctant to jump in) and no other changes to code or config. I'm trying to muck about with Hydrus the absolute minimum amount necessary.

Though it could be worthy of us to consider a standard approach to maintenance pages for our user facing apps (like H2 and ETD for example).

@jcoyne
Copy link
Contributor

jcoyne commented Oct 16, 2020

@peetucket this strikes me as something we should have in every app, and have a consistent way of dealing with it.

@peetucket
Copy link
Contributor Author

Agree completely, along with a way to show a message to people easily even if the app is up (e.g. "Performance is currently degraded and we are working on it..."). But I'm also not seeing the time created to make it happen right now in an elegant way for the migration next weekend, so this is low time cost stop-gap for just Hydrus for next weekend.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants