Skip to content

Reverse proxy docs - Add comment to use Docker DNS in nginx config example - #8604

Open
tvoirand wants to merge 1 commit into
nextcloud:mainfrom
tvoirand:reverse-proxy-docs-nginx-docker-dns-comment
Open

Reverse proxy docs - Add comment to use Docker DNS in nginx config example#8604
tvoirand wants to merge 1 commit into
nextcloud:mainfrom
tvoirand:reverse-proxy-docs-nginx-docker-dns-comment

Conversation

@tvoirand

@tvoirand tvoirand commented Aug 21, 2026

Copy link
Copy Markdown

Hi,

Thanks for your work on this great project!

The reverse proxy docs describe how to adapt the sample web server configuration in a scenario where the reverse proxy is running on the same server in a Docker container. In this scenario there are three options, the first one relying on host networking, and the next two relying on Docker bridge network's DNS name resolution.

I used the Docker bridge network's DNS method with nginx. But without explicitly telling nginx to use the Docker DNS server with a resolver 127.0.0.11 directive, this method fails with the nginx logs giving "no resolver defined to resolve nextcloud-aio-apache".
The resolver directive is not mentioned in the nginx config example. Adding it as an optional/commented line could help.

  • It does add yet another optional/commented line in the config example, so I guess adding this is a compromise between 1/ keeping the example simple and letting the user adapt by themself versus 2/ helping the user by making the example exhaustive at the risk of cluttering with too much information.

  • I'm not an expert of nginx in Docker containers, it's possible that some other aspect of my setup made this resolver 127.0.0.11 directive necessary.

  • Resolves: #

Summary

  • The PR was tested and verified that it works locally
  • Or will be tested after merge on a dedicated test instance (available for maintainers)
  • Sign-off message is added to all commits
  • Tests (playwright if possible) are included
  • Screenshots before/after for front-end changes
  • Documentation has been updated or is not required
  • Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
  • Milestone next added

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Thibaut Voirand <thibaut.voirand@murena.io>
@Zoey2936

Copy link
Copy Markdown
Collaborator

resolver should not be needed since the proxy_pass goes to an ip, not a hostname/domain

@szaimen szaimen added 2. developing Work in progress documentation Improvements or additions to documentation labels Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants