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

Fallback til client-side rendering #373

Closed
wants to merge 15 commits into from
Closed

Fallback til client-side rendering #373

wants to merge 15 commits into from

Conversation

anders-nom
Copy link
Contributor

@anders-nom anders-nom commented Jul 4, 2024

Dersom klienten kjører en annet versjon av bygget enn serveren, kjører vi nå en full re-render av dekoratøren. Skal hindre diverse feil som kan oppstå når client/server er ute av sync.

Endringer:

  • Setter client env var BUILD_ID til commit-sha for bygget
  • Rename /auth endepunktet til /client-state. Åpen for bedre forslag!
  • /client-state responsen inkluderer build-id'en til serveren
  • (Mulig at det er en bedre approach å ha build-id sjekken på et separat kall til et eget endepunkt for raskest mulig response.)
  • Sjekker build id fra serveren mot build id i client bundle'en. Dersom de er ulike, fetches assets for re-render fra serveren.
  • Kopierer csr-bundle js til filnavnene for de siste prod-deployene, slik at apper som har cachet disse versjonene tvinger til CSR.
  • Setter window event listeners i main init, etter sjekken på build-id, slik at vi ikke får duplikate listeners dersom CSR-funksjonen trigges.

Det slo meg også at vi egentlig ikke gjør client-side rendering her, vi får jo ferdige html-fragmenter fra serveren. Må ta en runde på terminologien! 😄

}
});

window.addEventListener("message", (e) => {

Check failure

Code scanning / SonarCloud

Origins should be verified during cross-origin communications

<!--SONAR_ISSUE_KEY:AZB-l5-xaynw1uoeukhj-->Verify the origin of the received message. <p>See more on <a href="https://sonarcloud.io/project/issues?id=navikt_decorator-next&issues=AZB-l5-xaynw1uoeukhj&open=AZB-l5-xaynw1uoeukhj&pullRequest=373">SonarCloud</a></p>
Copy link

sonarqubecloud bot commented Jul 4, 2024

@anders-nom
Copy link
Contributor Author

Endte opp med andre løsninger for dette problemet 😄

@anders-nom anders-nom closed this Jul 8, 2024
@anders-nom anders-nom deleted the csr-fallback branch July 9, 2024 16:37
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.

1 participant