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

Allow to include custom configuration in the location / of proxy blocks #2816

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

Conversation

jebabin
Copy link

@jebabin jebabin commented Apr 10, 2023

This allow to customise the location / of proxy blocks

@nginxproxymanagerci
Copy link

Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-2816

Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.

@webysther
Copy link

I think the #2899 is a complimentary to this one.

@jebabin
Copy link
Author

jebabin commented Aug 29, 2023

I think the #2899 is a complimentary to this one.

Unless I misunderstand the {{id}} things, It’s different as your patch allow a custom file per proxy entry while mine allow to have a custom config global to all proxy entries.
custom config per proxy can be done via GUI.

My use case is for example to customize the logging with if conditions (which can’t be done at an higher level of config) globally for all proxies

Copy link

PR is now considered stale. If you want to keep it open, please comment 👍

@github-actions github-actions bot added the stale label Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants