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

Exclude home page from SSR to fix connected flash #398

Merged
merged 1 commit into from
Aug 31, 2022

Conversation

florimondmanca
Copy link
Collaborator

@florimondmanca florimondmanca commented Aug 31, 2022

La page / est "semi-publique": elle a un contenu différent selon qu'on est connecté ou pas.

Actuellement elle souffre d'un flash. En effet, le SSR est activé sur cette page, et comme le serveur n'a aucune connaissance de l'état d'authentification, quand on est connecté on voit apparaître la page non-connecté pendant un court instant (elle provient du HTML généré par le serveur), puis quand le SPA prend le relais et calcule $user, on bascule sur la vue connectée.

Cette PR corrige ce flash en excluant la page / du SSR, puisque celle-ci dépend en réalité d'un état d'authentification que le serveur ne peut pas connaître.

Elle renomme donc PUBLIC_PAGES en STATIC_PAGES pour qu'il soit plus clair ce qui peut être servi en SSR ou non à ce stade.

Alternatives envisagées

Il est théoriquement possible de stocker les infos d'authentification sur le serveur SvelteKit et de passer par un cookie, de cette façon on pourrait servir à peu près n'importe quelle page en SSR, mais c'est bien plus complexe.

@florimondmanca florimondmanca merged commit 50354cd into master Aug 31, 2022
@florimondmanca florimondmanca deleted the fm/home-no-ssr branch August 31, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants