-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
docs: update caching documentation for Souin #1720
base: 3.1
Are you sure you want to change the base?
Conversation
core/performance.md
Outdated
email [email protected] | ||
hostname domain.com | ||
network your_network | ||
provider fastly # available providers are fastly, varnish, or souin (default) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Akamai and cloudflare too
core/performance.md
Outdated
If you are using the [API Platform distribution](../distribution/index.md), the Caddy server comes with an HTTP Cache module | ||
called [Souin](https://github.com/darkweak/souin/). It enables Caddy to act as its own caching reverse proxy by default. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fact, the official HTTP cache module is the cache-handler
. Souin is the development repository and the cache-handler is the very stable release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought it was the other way around, my bad, i'll make the changes ! We should also build Caddy with the cache-handler
instead of Souin directly I suppose
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose too but maybe we can add something in the documentation for those who want to use the development repository that is updated faster than the cache-handler
one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, I'll add some lines to explain this possibility
a9e94a7
to
8962069
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is a documentation PR aiming to explain the future (see api-platform/api-platform#2383 ) integration of Souin as a HTTP Caching module for Caddy in the distribution