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

feat: ajout d'une page de maintenance se déployant via un flag en DB #386

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

LucasDetre
Copy link
Collaborator

No description provided.

@gBusato
Copy link
Collaborator

gBusato commented Jul 26, 2024

Après avoir vu vite fait ce qui est fait, pour une page de maintenance il faut en faite faire la requête non pas dans le layout mais dans le middleware pour savoir s'il y a une maintenance en cours. Si c'est le cas, rediriger vers une page de maintenance à proprement parler avec un code 303

Un truc du style

export function middleware(req) {
  /// faire la requête pour savoir si on est en mode maintenance 
  ///  ....
  if (isMaintenanceMode) {
    return NextResponse.redirect('/maintenance');
  }
  return NextResponse.next();
}

export const config = {
  matcher: '/((?!maintenance).*)', // Exclure la page de maintenance de cette requête
};

@gBusato gBusato force-pushed the feat/page-maintenance-v0 branch 7 times, most recently from 2b8266a to ec34e10 Compare July 31, 2024 15:38
@gBusato
Copy link
Collaborator

gBusato commented Jul 31, 2024

Voici les évolutions du mode maintenance :

  • Redirection vers la page /maintenance depuis le middleware si le flag est définie coté DB

Coté reverse proxy :

  • Redirection vers la page maintenance.html dans le cas ou il y a un fichier maintenance.enable de présent.

Pour cela, j'ai rajouté les scripts pour rajouter le monde maintenance dans le readme.md.

Ce qu'il reste à faire :

  • Autoriser le chargement des fichiers qui sont dans /html/assets quand le mode maintenance est activé coté reverse proxy

@FaXaq FaXaq marked this pull request as draft September 23, 2024 08:50
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