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

persistent cache #30

Open
Tracked by #991
pi0 opened this issue May 15, 2021 · 10 comments
Open
Tracked by #991

persistent cache #30

pi0 opened this issue May 15, 2021 · 10 comments
Assignees

Comments

@pi0
Copy link
Member

pi0 commented May 15, 2021

Support abstract persistent cache for both input source and converted images. (implies supporting ttl)

Related: #46

@btkostner
Copy link

For this issue, I would assume the best place to put the cache would be at this function https://github.com/unjs/ipx/blob/main/src/ipx.ts#L72.

If nobody is working on this, I might be able to take a stab at it.

@ohbob
Copy link

ohbob commented Apr 17, 2022

were you able to do the cache?

@btkostner
Copy link

@ohbob I did not get to this, but it looks like @ascorbic did. Look at the attached draft PR for progress.

@multipliedtwice
Copy link

Hello, thank you for an awesome library. Do you plan to release the feature?

@guillaumeprevost
Copy link

Hi ! Any news on this feature ?

@AbdallahAlhaddad
Copy link

Hi @pi0
Any update on this feature?

@pi0 pi0 mentioned this issue Sep 14, 2023
9 tasks
@pi0 pi0 self-assigned this Oct 18, 2023
@bernhardberger
Copy link

Is there any timeframe on this? If it's something that will happen within Q1/2024 we could possibly hold out switching away from IPX in our projects but it would be good to know for planning how far the roadmap this is.

@pi0
Copy link
Member Author

pi0 commented Dec 23, 2023

@bernhardberger It is on my plan. BTW curious how is your setup is it on top of nuxt/niro/h3?

@bernhardberger
Copy link

bernhardberger commented Dec 23, 2023

tbh, no clue - the node process is running supervised by our hosting partner and the nitro server more or less directly exposed to the outside world. There's no reverse proxy involved (that we could have control over) that would do the caching. Our clients and also the higher ups internally want to avoid cloud based solutions at all cost, also because it has a lot of implications regarding compliance etc in the EU with our larger corporate customers.

We're trying to move away from monolithic solutions as it would open up more possibilities to share frontend code between different projects and lessen the need to have frontend devs that have backend specific framework knowledge (Laravel, TYPO3, WordPress etc.). Our projects are mostly brand/marketing stuff (MPA, Content driven websites and portals) so going headless is mostly a decision driven by workflow and less by having to have the ability to scale (in which case we'd do SSG). But estimating about 20-30 projects yearly IPX in it's current form is a deal breaker when we had like 10-20 projects per server and now we'd be stuck at like 2 before resources run out 🙈

//Edit: I misread. FE stack is Nuxt3 + some Nitro middlewares for routing+ h3-compression to make Lighthouse stop complaining. SEO stuff unfortunately is the highest priority.

PPS: IPX doesn't work properly with SWR in Nuxt3. Delivers Bytestream json instead of images. ISR works.

@andrevandal
Copy link

hey, any update?

maybe persist images like aws-samples/image-optimization could be a good improvement.

I'm already using Cloudflare cache to prevent my cpu meltdown, but it's not the ideal. :S

if we could store the generated images an use them instead of creating a new one, it should be amazing.

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

No branches or pull requests

8 participants