Fixes#10115
* Adds php-handler configuration tips
* Also some attempt at tightening up the overview
Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
- Follow the .htaccess and exclude .well-known redirects for ACME challenges and PKI validation, which are not handled by Nextcloud, and handle requests for `/remote`
- Preserve the query string when redirecting .well-known URIs.
- Separate the example config files from nginx.rst
Signed-off-by: MichaIng <micha@dietpi.com>
- Changed `try_files` for `/.well-known` to allow URIs that we don't handle and not pass them to our front-end controller
- Changed rule to match `/nextcloud/*` from regex rule to equivalent prefix rule
You probably are correct on NGINX optimizing groups with unused captures into non-capturing groups, but unless we know for sure/see documentation I think it is best we are explicit.
I think we mostly agree on these, grouping them for easy merging.
Creating a new branch is giving me an error for some reason, so I will commit directly. If necessary we can always revert.
- Removed `index.htm` from `index` directive
- Rewrote `DavClnt` user agent handler using `location =` and `return`, rather than `rewrite`
- Removed block which prevents access to hidden files; incorporated rule to prevent access to root-level hidden folders into existing regex rule, as in `.htaccess`
- Removed `$uri/` from `try_files` directives for static files
Signed-off-by: Jivan Pal <jivan.pal@gmail.com>
After inspecting the Nginx configs currently seen at [ https://docs.nextcloud.com/server/19/admin_manual/installation/nginx.html ], I have notice that they are very redundant in some areas, and handle specific paths in special ways which are not needed, and which run the risk of needing to become more specific down the road if NextCloud adds new features, as well as not handling all static filetypes. I have reworked the configs from scratch by directly adapting the `.htaccess` files, and this is the result. Comments are included in the file to give rationale/explanation for why things are written in this new way, and if the maintainers decide to accept this pull request, I will leave it up to them to decide whether to include these expository comments upstream.
Signed-off-by: Jivan Pal <jivan.pal@gmail.com>
+ Ref: https://github.com/nextcloud/documentation/pull/1520
+ The master merge to resolve conflicts has reverted the headers changes for the subdir config. This commit redoes the intended changes, to match Nginx webroot config and .htaccess.
Signed-off-by: Micha Felle <micha@dietpi.com>
