-
Notifications
You must be signed in to change notification settings - Fork 92
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
feat: v1.5.0 #237
feat: v1.5.0 #237
Conversation
ticoombs
commented
Apr 13, 2024
•
edited
Loading
edited
- postgres v16 (fixes: On next release, include upgrade instructions for postgres 15 -> 16 #227)
- pictrs 0.5.x
- nginx cache (fixes: Consider enabling nginx caching #195)
- autoexplain & stats (fixes: Recommended/default config should include pg_stat_statements and auto_explain #211)
- lemmy_web_port (fixes: Better way of avoiding port conflicts #32)
2105427
to
af44a74
Compare
Also ansible format is finding some mis-indent. |
@@ -60,6 +61,13 @@ server { | |||
proxy_set_header X-Real-IP $remote_addr; | |||
proxy_set_header Host $host; | |||
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for; | |||
|
|||
proxy_cache lemmy_cache_{{domain|replace(".","_")|replace("-","_")}}; |
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.
this probably needs to ensure that at least the accept header is included in the proxy_cache_key
to avoid serving activity+json
or html to the wrong client because it's already cached
We're shooting for the |
@dessalines we're good to go |
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.
Sweet, thx!