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

Consider adding caching to our reverse proxy #47

Open
PeterJCLaw opened this issue May 14, 2023 · 0 comments
Open

Consider adding caching to our reverse proxy #47

PeterJCLaw opened this issue May 14, 2023 · 0 comments

Comments

@PeterJCLaw
Copy link
Member

PeterJCLaw commented May 14, 2023

Currently we're not caching GitHub Pages (or in fact any of our backend sites). Doing so might reduce the I/O our proxy needs to do in the case where pages haven't changed.

Apparently compression is disabled on the upstream side of the connection due to our use of url rewriting, meaning that we're currently communicating with e.g: GitHub Pages in a non-compressed format. Since we do (I believe) send out compressed data to clients I'm guessing this means we're also compressing everything on the fly all the time, which is also probably not great for workload. (Not that this has actually been an issue thus far).

As reported by @prophile in Slack (https://studentrobotics.slack.com/archives/C02BXUAK33M/p1684074664913729)

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

No branches or pull requests

1 participant