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.

Sign up for GitHub

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

Jump to bottom

YKI:AKR:OTR:VKT(Frontend & Backend): OPHKIOS-21 Cache headereiden korjaus #647

Merged
merged 3 commits into from
Mar 5, 2024
Merged

YKI:AKR:OTR:VKT(Frontend & Backend): OPHKIOS-21 Cache headereiden korjaus #647

merged 3 commits into from
Mar 5, 2024

Conversation

jrkkp
Copy link
Contributor

@jrkkp jrkkp commented Mar 4, 2024 •
edited
Loading

Yhteenveto

Cache-control headereiden korjaus. Cachen aika 30 päivää.

Testattavissa: https://vkt.testiopintopolku.fi/vkt/etusivu

En päivittäny muualle kun pitäisi olla identtinen muissa palveluissa.

@jrkkp jrkkp changed the title VKT(Frontend & Backend): Asset cache experiment [deploy] VKT(Frontend & Backend): Cache headereiden korjaus Mar 4, 2024
@jrkkp jrkkp changed the title VKT(Frontend & Backend): Cache headereiden korjaus YKI:AKR:OTR:VKT(Frontend & Backend): OPHKIOS-21 Cache headereiden korjaus Mar 5, 2024
@jrkkp jrkkp marked this pull request as ready for review March 5, 2024 12:50
pkoivisto
pkoivisto approved these changes Mar 5, 2024
View reviewed changes
Copy link
Contributor

@pkoivisto pkoivisto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muutokset näyttää hyviltä. Jostain syystä mä saan itse testaillessa epäkonsistentetteja tuloksia: max-age pomppii jotenkin 1 ja 30 vuorokauden välillä hieman satunnaisen oloisesti. Tämä siis myös useamman empty cache & hard reloadin jälkeenkin.
Liittyy varmaan jotenkin selaimen aiemmin havaitsemaan 1 vrk:n max-ageen. Ei mielestäni estä kuitenkaan mergaamista.

Screenshot 2024-03-05 at 15 14 56

@pkoivisto
Copy link
Contributor

Muutokset näyttää hyviltä. Jostain syystä mä saan itse testaillessa epäkonsistentetteja tuloksia: max-age pomppii jotenkin 1 ja 30 vuorokauden välillä hieman satunnaisen oloisesti. Tämä siis myös useamman empty cache & hard reloadin jälkeenkin. Liittyy varmaan jotenkin selaimen aiemmin havaitsemaan 1 vrk:n max-ageen. Ei mielestäni estä kuitenkaan mergaamista.

Screenshot 2024-03-05 at 15 14 56

Tai niin, tässähän tietty varmaan on taas useampi välimuistikerros välissä, eli hämäävät vastaukset tulevat varmaan cloudfrontilta tms.

@jrkkp
Copy link
Contributor Author

jrkkp commented Mar 5, 2024

Tai niin, tässähän tietty varmaan on taas useampi välimuistikerros välissä, eli hämäävät vastaukset tulevat varmaan cloudfrontilta tms.

Juu, varmaankin johonkin cloudfront instanssiin päivittynyt tuo 30pv ja toiseen jäänyt se aiemmin ollut 1pv. Tai sitten niin, että välimuisti on konttikohtainen ja jonkun kontin kohdalla on se 1pv cache ja toisella uudempi 30pv.

@jrkkp jrkkp merged commit c841415 into dev Mar 5, 2024
12 checks passed
@jrkkp jrkkp deleted the feature/OPHKIOS-21 branch March 5, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pkoivisto pkoivisto pkoivisto approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jrkkp @pkoivisto