Skip to content

Some fixes#3

Open
patrick7 wants to merge 3 commits intobartosjiri:masterfrom
patrick7:fixes
Open

Some fixes#3
patrick7 wants to merge 3 commits intobartosjiri:masterfrom
patrick7:fixes

Conversation

@patrick7
Copy link
Copy Markdown

@patrick7 patrick7 commented Sep 11, 2022

  • Load assets from /assets to display error pages correctly when loading from subfolders
  • Always permit assets, which will make it working e.g. with allow/deny ip addresses
  • Move error pages to /.nginx-errors to avoid collisions with the asset folder of some web apps

- Load assets from /assets to display error pages correctly when loading from subfolders
- Always permit assets, which will make it working e.g. with allow/deny ip addresses
@patrick7 patrick7 changed the title Some fixes: Some fixes Sep 11, 2022
@bartosjiri
Copy link
Copy Markdown
Owner

Sorry for the massive delay.

I am happy to merge it if some other community members confirm the functionality and described benefits - as I personally no longer use nginx.

@markusgut
Copy link
Copy Markdown

And the config file is so much simpler, too! 👍 Thanks @patrick7

@bartosjiri I can confirm, please merge.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants