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

Nginx Proxy Manager doesn't support ports in domain #3986

Open
Elyviere opened this issue Jan 21, 2025 · 2 comments
Open

Nginx Proxy Manager doesn't support ports in domain #3986

Elyviere opened this issue Jan 21, 2025 · 2 comments
Labels
bug docs This issue is related to documentation

Comments

@Elyviere
Copy link

Elyviere commented Jan 21, 2025

Describe the bug
Nginx Proxy Manager does not appear to support ':' in domain names, but the docs to set up Nginx Proxy Manager to serve Traefik says to include the port in the domain name. The docs therefore seem incorrect, it's not possible to follow the docs without receiving an error. See screenshots below.

Relevant part from the matrix-docker-ansible-deploy docs:
Image

The error in Nginx Proxy Manager:
Image

To Reproduce
Follow the instructions in https://github.com/spantaleev/matrix-docker-ansible-deploy/tree/master/examples/reverse-proxies/nginx-proxy-manager#using-nginx-proxy-manager. When adding the proxy host for "matrix.example.com:8448" and pressing save the following error shows up:
data/domain_names/0 must match pattern "^[^&| @!#%^();:/\\}{=+?<>,~'"]+$"`.

Expected behavior
The docs should describe how to set this up in Nginx Proxy Manager in a way which is supported by Nginx Proxy Manager.

@Elyviere Elyviere added the bug label Jan 21, 2025
@luixxiul luixxiul added the docs This issue is related to documentation label Jan 22, 2025
@Zaken7
Copy link

Zaken7 commented Jan 28, 2025

Bumping this ! I'm facing the same issue.

@spantaleev
Copy link
Owner

It sounds like Nginx Proxy Manager is broken and doesn't support custom ports. Feel free to report this issue upstream and look for a fix there. If it's a guidance issue and our docs are to be updated, feel free to submit a PR or at least tell us what needs to change, so we change it.

Most of us don't care about Nginx Proxy Manager and this is unlikely to be solved by anyone else but you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug docs This issue is related to documentation
Projects
None yet
Development

No branches or pull requests

4 participants