Repository navigation
Conversation
dannycolin
reviewed
Sep 23, 2026
dannycolin
reviewed
Sep 23, 2026
justdave
approved these changes
Oct 12, 2026
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 theX-Forwarded-Protoheader for TLS termination.installing/nginx.rst: new file.web_server.rstalready listednginxin 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 withMOJO_REVERSE_PROXY=1.Notes
bugzilla.pl(MOJO_LISTENdefault). Both pages mention thePORToverride.httpd.conf.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. Thenginxtoctree warning is gone. Remaining warnings are pre-existing: the extension glob patterns (staged bymakedocs.pl, not by plain sphinx-build) and two inintegrating/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.