Skip to content

docs(config): clarify forwarded_for_headers configuration - #63542

Open
joshtrichards wants to merge 1 commit into
masterfrom
jtr/docs-fwd-for-headers
Open

docs(config): clarify forwarded_for_headers configuration#63542
joshtrichards wants to merge 1 commit into
masterfrom
jtr/docs-fwd-for-headers

Conversation

@joshtrichards

@joshtrichards joshtrichards commented Aug 24, 2026

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

  • Correct the forwarded_for_headers example to use HTTP_X_FORWARDED_FOR and remove the confusing references to HTTP_X_FORWARDED and HTTP_FORWARDED_FOR
  • Note that the default HTTP_X_FORWARDED_FOR configuration is suitable for many environments and may not need to be changed
  • Document the trusted-proxy security requirements and comma-separated address handling
  • Link to the X-Forwarded-For header reference

Came up due to some confusion on the forum, since a quick scan of the entry could lead one to believe the examples are valid headers/header keys (they aren't): https://help.nextcloud.com/t/administration-settings-not-showing-via-an-https-connection/248505

TODO

  • ...

Checklist

AI (if applicable)

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

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added this to the Nextcloud 35 milestone Aug 24, 2026
@joshtrichards
joshtrichards requested a review from a team as a code owner August 24, 2026 16:35
@joshtrichards joshtrichards added the 3. to review Waiting for reviews label Aug 24, 2026
@joshtrichards
joshtrichards requested review from Altahrim, come-nc, icewind1991 and leftybournes and removed request for a team August 24, 2026 16:35
@joshtrichards

Copy link
Copy Markdown
Member Author

/backport to stable34

@joshtrichards

Copy link
Copy Markdown
Member Author

/backport to stable33

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant