Skip to content

Cache images on browser #27835

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.

Sign up for GitHub

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

Jump to bottom
Merged
LukasReschke merged 1 commit into from
Jul 7, 2021
Merged

Cache images on browser #27835

LukasReschke merged 1 commit into from
Jul 7, 2021

Conversation

@solracsf
Copy link
Member

@solracsf solracsf commented Jul 6, 2021

Fix #26851

@solracsf solracsf added the 3. to review Waiting for reviews label Jul 6, 2021
@szaimen szaimen added this to the Nextcloud 23 milestone Jul 6, 2021
@juliusknorr
Copy link
Member

juliusknorr commented Jul 7, 2021

Also needs an update on the nginx docs then:
https://github.com/nextcloud/documentation/blob/master/admin_manual/installation/nginx-root.conf.sample#L135-L139

@szaimen szaimen added pending documentation This pull request needs an associated documentation update 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 7, 2021
solracsf added a commit to nextcloud/documentation that referenced this pull request Jul 7, 2021
@solracsf
Cache images on browser
01737b5 
Following nextcloud/server#27835
@solracsf solracsf mentioned this pull request Jul 7, 2021
Merged
@solracsf
Copy link
Member Author

solracsf commented Jul 7, 2021

Done at nextcloud/documentation#6953

@solracsf
Copy link
Member Author

solracsf commented Jul 7, 2021

/backport to stable22

@solracsf
Copy link
Member Author

solracsf commented Jul 7, 2021

/backport to stable21

@solracsf
Copy link
Member Author

solracsf commented Jul 7, 2021

/backport to stable20

@szaimen szaimen removed the pending documentation This pull request needs an associated documentation update label Jul 7, 2021
@LukasReschke LukasReschke merged commit 11739ce into master Jul 7, 2021
@LukasReschke LukasReschke deleted the htaccess-cache branch July 7, 2021 17:09
This was referenced Jul 7, 2021
Merged
Merged
Merged
MichaIng added a commit to nextcloud/documentation that referenced this pull request Jul 9, 2021
@MichaIng
Align Nginx configuration with Apache2 .htaccess
b5ede86 
As of: nextcloud/server#27835

Signed-off-by: MichaIng <[email protected]>
@MichaIng MichaIng mentioned this pull request Jul 9, 2021
Merged
MichaIng added a commit to nextcloud/documentation that referenced this pull request Jul 9, 2021
@MichaIng
Align Nginx configuration with Apache2 .htaccess
68d8d75 
As of: nextcloud/server#27835

Signed-off-by: MichaIng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LukasReschke LukasReschke LukasReschke approved these changes

@juliusknorr juliusknorr juliusknorr approved these changes

@szaimen szaimen szaimen approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

Assignees

No one assigned

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Milestone

Nextcloud 23

Development

Successfully merging this pull request may close these issues.

Missing cache control for (some) static resources

5 participants

@solracsf @juliusknorr @LukasReschke @szaimen