Skip to content
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

Update Maintenance Mode code to 504 #8771

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

morganestes
Copy link
Contributor

When a site is in Maintenance Mode on the server side (either through actual maintenance or because it's been taken offline for various reasons), the server response is

HTTP/1.1 504 Target in maintenance.

This change updates that code and move the section down to the other 504 examples.

Summary

5xx Level Errors - Corrects the server response code for Maintenance Mode.

Effect

May break links since the anchor will change.

Release

  • When ready

Post Launch

Do not remove - To be completed by the docs team upon merge:

  • Redirect /old-path/ => /new-path/ (if applicable)
  • Include/exclude pages ^ respectively within docs search service provider (if applicable)
  • For Heroes - add a props post to the discussion board.
  • Remove from the project board

When a site is in Maintenance Mode on the server side (either through actual maintenance or because it's been taken offline for various reasons), the server response is
> HTTP/1.1 504 Target in maintenance.

This change updates that code and move the section down to the other 504 examples.
@morganestes morganestes requested a review from a team as a code owner November 6, 2023 17:32
Copy link
Member

@rachelwhitton rachelwhitton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@rachelwhitton rachelwhitton merged commit 37d4a4d into pantheon-systems:main Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants