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

frontside.com joints the revolution #332

Merged
merged 1 commit into from
Nov 15, 2023
Merged

frontside.com joints the revolution #332

merged 1 commit into from
Nov 15, 2023

Conversation

cowboyd
Copy link
Member

@cowboyd cowboyd commented Nov 15, 2023

Motivation

Consolidate our middleware stack that we're using now on a bunch of sites that has here to for been copy-pasted between them. Our current "router" package also has some not-so-optimal aspects to it:

  • no way to add middleware
  • does not currently propagate ETAGs for proxyies, so content that is cached on the source, does not end up getting cached on the backend.
  • unclear how tailwind integration works.

Approach

Upgrade to "revolution" (an html server) which has extracted into a separate library. This consolidates a lot of code, makes twind more accessible and standard via twing.config.ts (which is how it is documented on their site) and also improves the caching and proxy behavior to load sites faster.

For example, we were not properly forwarding if-none-match headers and so none of our proxy responses were being cached, so this adds middleware to do that.

Preview

https://frontside--revolutionize.deno.dev/

@cowboyd cowboyd requested a review from a team November 15, 2023 16:27
Copy link

netlify bot commented Nov 15, 2023

Deploy Preview for frontside canceled.

Name Link
🔨 Latest commit 8184832
🔍 Latest deploy log https://app.netlify.com/sites/frontside/deploys/65551e8b4c577b00086b12b3

@cowboyd cowboyd marked this pull request as ready for review November 15, 2023 18:19
Upgrade to the middleware framework we've been using for all our sites
which has now been extracted into a separate library. This
consolidates a lot of code, makes twind more accessible and standard
via `twing.config.ts` and also  improves the caching and proxy
behavior to load sites faster.

For example, we were not properly forwarding `if-none-match` headers
and so none of our proxy responses were being cached.
@cowboyd cowboyd merged commit c8701d3 into production Nov 15, 2023
4 checks passed
@cowboyd cowboyd deleted the revolutionize branch November 15, 2023 19:42
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