Skip to content
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

docs: clarify limitation around reverse_proxy #457

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dbaynard
Copy link
Contributor

@dbaynard dbaynard commented Mar 4, 2025

Simple change, would have saved me multiple hours of work.
I assume Reverse_proxy with custom http payload - Help - Caddy Community still holds.

That[every proxy request modifying the payload]’s out of scope of reverse_proxy.

You should write your own module to do this.

Edit: I've added a further, related commit. Again, hours of work.

The previous documentation was clear on handling http->https proxying,
but not the other way round. They did not cover the case of proxying to
a unix socket, either. There are special considerations when proxying to
caddy in either of these cases, that should be mentioned.
@dbaynard dbaynard force-pushed the push-ynpwtuyomzuk branch from 9aed03f to 1e0cd2e Compare March 10, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant