Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs: restructure and clarify reverse proxy documentation (part one) #7022

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

Open
joshtrichards wants to merge 3 commits into main
base: main
Choose a base branch
Loading
from jtr/docs-rp-i

Conversation

@joshtrichards
Copy link
Member

@joshtrichards joshtrichards commented Oct 22, 2025
edited
Loading

Updated the reverse proxy documentation for Nextcloud AIO, enhancing clarity and structure:

  • Reorganized the top sections.
  • Added sections to provide context on integrated versus external reverse proxies, as well as secure tunnels/proxy platforms.
  • Deduplicated some content.
  • General proofreading and other improvements.

No substantive content was dropped, but some was moved. If you notice a large block missing during review, please look for it elsewhere in the diff.

This was just a first pass, focused mostly on the top portion. Other areas were left unchanged to keep the work (and PR reviews) to a reasonable size.

szaimen reacted with heart emoji szaimen reacted with rocket emoji
Updated the reverse proxy documentation for Nextcloud AIO, enhancing clarity and structure. 
Re-organized top section(s).
Added sections to provide context on integrated versus external reverse proxies versus secure tunnels/proxy platforms.
De-duplicated some content.
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added 3. to review Waiting for reviews documentation Improvements or additions to documentation enhancement New feature or request labels Oct 22, 2025
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards changed the title (削除) docs: Revise reverse proxy docs (first pass) (削除ここまで) (追記) docs: restructure and clarify reverse proxy documentation (part one) (追記ここまで) Oct 23, 2025
@szaimen szaimen removed the enhancement New feature or request label Oct 23, 2025
@szaimen szaimen added this to the next milestone Oct 23, 2025
Copy link
Collaborator

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey 👋

Thanks a lot for working on this! ☺️

In general, I am still wondering if we provide too much and too verbose information in this file but probably your are right and this is needed for people to get an overview first in oder to understand what to do with it. I just hope that people actually read it and not skip it if too much info is provided. But I also don't see a better way currently.

Apart from one point below, the restructuring looks good to me. Thanks again! ☺️

Cloudflare and Tailscale provide Zero Trust Network Access services. For AIO we are primarily concerned with:

## Content
- Cloudflare Tunnel / Cloudflare Proxy
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should point out that Cloudlare Tunel usually still exposes things externally. So I am not completely sure if it fits the category Zero Trust Network Access platform. Also Cloudflare Proxy does not fit in this category IMO

### Adapting the sample web server configurations below
1. Replace `<your-nc-domain>` with the domain on which you want to run Nextcloud.
1. Adjust the port `11000` to match your chosen `APACHE_PORT`.
1. Adjust `localhost` or `127.0.0.1` to point to the Nextcloud server IP or domain depending on where the reverse proxy is running. See the following options.
Copy link
Collaborator

@szaimen szaimen Oct 23, 2025
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw, I always have the feeling that people skip this part and directly head over to the example config. Maybe we should document the different possibilities in the config examples directly as comments? WDYT?

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

Reviewers

@szaimen szaimen szaimen left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

3. to review Waiting for reviews documentation Improvements or additions to documentation

Projects

None yet

Milestone

next

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /