Skip to content

feat(core): serverCache() middleware#2989

Closed
iuioiua wants to merge 9 commits into
freshframework:mainfrom
iuioiua:cache-middleware
Closed

feat(core): serverCache() middleware#2989
iuioiua wants to merge 9 commits into
freshframework:mainfrom
iuioiua:cache-middleware

Conversation

@iuioiua

@iuioiua iuioiua commented May 22, 2025

Copy link
Copy Markdown
Contributor

Early reviews welcome.

Todo:

  • Refine use in www
  • Add test for respecting of Vary headers
  • Add documentation
  • Add to init workflow

Closes #8

Comment thread src/middlewares/cache.ts Outdated
Comment thread src/middlewares/cache.ts Outdated
Comment thread src/middlewares/cache.ts Outdated
Comment thread src/middlewares/cache.ts
Comment thread src/middlewares/cache.ts Outdated

@iuioiua iuioiua left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Great suggestions! Thank you.

Comment thread src/middlewares/cache.ts Outdated
Comment thread src/middlewares/cache.ts Outdated
@iuioiua iuioiua changed the title feat(core): cache() middleware feat(core): serverCache() middleware May 27, 2025
@iuioiua iuioiua closed this Sep 2, 2025
@iuioiua iuioiua deleted the cache-middleware branch September 2, 2025 23:00
@DontMash

DontMash commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

Why was this abandoned?

@iuioiua

iuioiua commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

I no longer have the time to contribute. Feel free to copy any code from here, if you'd like to create your own PR 🙂

@DontMash

DontMash commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

I no longer have the time to contribute. Feel free to copy any code from here, if you'd like to create your own PR 🙂

Based on what you've already done, what is still left to do? Is the Todo-list in the description up-to-date?

@iuioiua

iuioiua commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

It was a little while ago but yes, the todo list is up-to-date, IIUC.

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.

Caching of pages

3 participants