Skip to content

Bug 1849943: Add Apache and Nginx reverse proxy instructions - #190

Open
topunix wants to merge 3 commits into
bugzilla:mainfrom
topunix:bug-1849943
Open

topunix wants to merge 3 commits into
bugzilla:mainfrom
topunix:bug-1849943

Conversation

@topunix

@topunix topunix commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Part 1 of Bug 1849943 (docs still based on BMO instead of Harmony).

What

  • installing/apache.rst: replaces the TODO stub with reverse proxy instructions. Covers enabling mod_proxy/mod_proxy_http/mod_headers, a virtual host example, Debian/Ubuntu and Fedora/Red Hat paths, and the X-Forwarded-Proto header for TLS termination.
  • installing/nginx.rst: new file. web_server.rst already listed nginx in its toctree but the page didn't exist, which caused a Sphinx build warning.

Both pages match the existing Caddy page: proxy to 127.0.0.1:3001, and run the app with MOJO_REVERSE_PROXY=1.

Notes

  • Port 3001 comes from bugzilla.pl (MOJO_LISTEN default). Both pages mention the PORT override.
  • The Apache page uses a site/conf.d file and never touches the main httpd.conf.
  • The Nginx page calls out client_max_body_size. Nginx defaults to 1 MB, which would silently block larger attachment uploads.

Testing

Built locally with the pinned versions in docs/requirements.txt (full rebuild, sphinx-build -E). Both pages render. The nginx toctree warning is gone. Remaining warnings are pre-existing: the extension glob patterns (staged by makedocs.pl, not by plain sphinx-build) and two in integrating/auth-delegation.rst, which the next PR fixes.

Still to come on 1849943

A separate PR removing BMO-specific references across the docs (BMO test server links, example hostnames, dead mozilla.org links) and the remaining mod_perl instructions in the platform install pages.

Comment thread docs/en/rst/installing/apache.rst Outdated
Comment thread docs/en/rst/installing/nginx.rst

This branch has not been deployed

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants