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

Add mapfile serving for nginx conf examples #8095

Merged
merged 2 commits into from
Feb 24, 2022
Merged

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Feb 24, 2022

It got lost by #2197 🙄

#1504

@juliushaertl juliushaertl merged commit 3b33f6a into master Feb 24, 2022
@juliushaertl juliushaertl deleted the fix/mapfiles-nginx branch February 24, 2022 08:14
@skjnldsv
Copy link
Member Author

/backport to stable23

@skjnldsv
Copy link
Member Author

/backport to stable22

@skjnldsv
Copy link
Member Author

/backport to stable21

@backportbot-nextcloud

This comment was marked as resolved.

@backportbot-nextcloud

This comment was marked as resolved.

@szaimen
Copy link
Contributor

szaimen commented Feb 25, 2022

@skjnldsv do you mind adding this also to the .htaccess file?
https://github.com/nextcloud/server/blob/master/.htaccess

@szaimen
Copy link
Contributor

szaimen commented Feb 25, 2022

Otherwise I fear this will remain broken for native apache instances...

@skjnldsv
Copy link
Member Author

@skjnldsv do you mind adding this also to the .htaccess file? https://github.com/nextcloud/server/blob/master/.htaccess

I thought we had that already 😮‍💨

@skjnldsv
Copy link
Member Author

@skjnldsv
Copy link
Member Author

Yeah, it seems done already then.
If you have htaccess.RewriteBase properly configured (by default), then it's fine

@skjnldsv
Copy link
Member Author

@skjnldsv do you mind adding this also to the .htaccess file? https://github.com/nextcloud/server/blob/master/.htaccess

Also, those lines onlyadd caching, which we probably don't want for map files 😉

@szaimen
Copy link
Contributor

szaimen commented Feb 25, 2022

Right, so I guess with apache it should already work?

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

Successfully merging this pull request may close these issues.

None yet

3 participants