Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.

caching and cookie management #708

Closed
fisx opened this issue Jun 22, 2016 · 6 comments
Closed

caching and cookie management #708

fisx opened this issue Jun 22, 2016 · 6 comments
Milestone

Comments

@fisx
Copy link
Collaborator

fisx commented Jun 22, 2016

  1. i experienced weird behavior that could only be fixed by deleting all cookies, and i turned out to have collected a lot of them.
  2. we have not thought about caching at all so far. there may be some performance to be gained, especially if we use some hashing-naming-scheme for the /static content, but there may also be some hidden issues with stale pages that are delivered from cache, but shouldn't.
@fisx fisx added this to the it.3 milestone Jun 22, 2016
@fisx
Copy link
Collaborator Author

fisx commented Jun 28, 2016

related, or duplicate of: #283.

@fisx
Copy link
Collaborator Author

fisx commented Jun 28, 2016

Link „Profil anzeigen“ in user menu in main nav redirects to idea space overview
https://aula-stage1.liqd.net/user/18/ideas

moritz, can you clear all you aula-related cookies and try to reproduce this?
i suspect it won't work. this is a known bug.
Moritz: Link worked as expected again, when I tested it this morning

@fisx
Copy link
Collaborator Author

fisx commented Jul 19, 2016

this may be resolved by now. not sure, though.

@fisx fisx mentioned this issue Jul 26, 2016
@fisx
Copy link
Collaborator Author

fisx commented Jul 26, 2016

lots of packages on hackage:

nothing that does browser cache busting.

@fisx
Copy link
Collaborator Author

fisx commented Jul 26, 2016

goal:

  • cache everything under /static
  • do not cache anything else
  • every reference to a cached resource should end with [?|&]last_changed=<epoch_time>

we could write a more sophisticated (and much slower) middleware that parses all of html, css, js, and whatever else i forgot, but...

@fisx
Copy link
Collaborator Author

fisx commented Jul 27, 2016

This has been resolved in #735.

(I opened #921 to keep the last two comments in scope.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant