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

Improve ngnix config (.well-known) #11112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

martin-rueegg
Copy link
Contributor

@martin-rueegg martin-rueegg commented Sep 12, 2023

  • avoid rule for /.well-known to apply also to e.g. /.well-known.xml
  • deny access to /.well-known and /.well-known/
  • provide "more understandable" explanation for "regex exception"
  • make sure, /.well-known/acme-challenge and /.well-known/pki-validation are matched as a directory, not the beginning of some other service name
  • use temporary redirect if nextcloud is not installed in webroot

☑️ Resolves

  • Fix #…

🖼️ Screenshots

  • /installation/nginx.html
    image

@martin-rueegg
Copy link
Contributor Author

Happy to backport, once accepted.

@susnux
Copy link
Contributor

susnux commented Nov 13, 2024

Could you rebase this onto current master branch? Thank you!

- provide subsections for the different services and configurations
- provide command line `curl` examples to debug the redirects
- provide exmaple config for apache .htaccess to redirect any well known service
- consistently use `NGINX` in headers, and `nginx` in text
- fix some spelling/grammar

Signed-off-by: Martin Rüegg <[email protected]>
@martin-rueegg martin-rueegg force-pushed the enh/improve-nginx-well-known branch from a6deeb9 to 6a70911 Compare November 21, 2024 09:06
@martin-rueegg
Copy link
Contributor Author

Could you rebase this onto current master branch? Thank you!

Done, @susnux. Sorry for my delay.

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.

7 participants