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

feat: v1.5.0 #237

Merged
merged 32 commits into from
Jun 7, 2024
Merged

feat: v1.5.0 #237

merged 32 commits into from
Jun 7, 2024

Conversation

ticoombs
Copy link
Collaborator

@ticoombs ticoombs commented Apr 13, 2024

README.md Outdated Show resolved Hide resolved
examples/config.hjson Outdated Show resolved Hide resolved
examples/config.hjson Outdated Show resolved Hide resolved
examples/vars.yml Outdated Show resolved Hide resolved
examples/hosts Show resolved Hide resolved
@dessalines
Copy link
Member

Also ansible format is finding some mis-indent.

lemmy.yml Outdated Show resolved Hide resolved
@@ -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("-","_")}};
Copy link
Contributor

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

@dessalines
Copy link
Member

We're shooting for the 0.19.4 release tomorrow, so this will also need to be merged with it, as it contains the upgrade instructions.

@ticoombs ticoombs marked this pull request as ready for review June 7, 2024 06:16
@ticoombs ticoombs self-assigned this Jun 7, 2024
@ticoombs
Copy link
Collaborator Author

ticoombs commented Jun 7, 2024

@dessalines we're good to go

@ticoombs ticoombs added this to the 1.5.0 milestone Jun 7, 2024
@ticoombs ticoombs changed the title WIP: 1.5.0 feat: v1.5.0 Jun 7, 2024
Copy link
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

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

Sweet, thx!

@dessalines dessalines merged commit c872f2d into main Jun 7, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants