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

Frontend cache-control TTL #112

Open
klausmark opened this issue Jun 4, 2024 · 2 comments
Open

Frontend cache-control TTL #112

klausmark opened this issue Jun 4, 2024 · 2 comments

Comments

@klausmark
Copy link

In some of the labs, you use the NodePort to connect to the frontend. If done through a PROXY the dynamic content is cached. The frontend should set appropriate headers to fix this.

@michaelin
Copy link
Contributor

Thanks @klausmark. Nice catch.

This issue caused problems in the rolling updates exercise. After doing a rolling update, the image version wasn't updated in the UI. When describing the pods, it was correct, though.

It seems like we need to set the cache-control header as suggested. See https://stackoverflow.com/questions/23112316/using-flask-how-do-i-modify-the-cache-control-header-for-all-output

@klausmark
Copy link
Author

Hi @michaelin
After reviewing the source code of quotes-flask, I’m reconsidering my initial assumption. It seems more likely that the issue could be due to the browser’s JavaScript settings or possibly the proxy interfering with JavaScript execution on non-HTTPS, non-standard port websites.

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

2 participants