Skip to content

README: add Caddy proxy example #687

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
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

philips
Copy link
Contributor

@philips philips commented Apr 3, 2025

What's new?

README: It took me a minute to figure out that Pinchflat needed me to "strip_prefix" so maybe this will help others.

What's changed?

Add docs on Caddy configuration

Any other comments?

It took me a minute to figure all of this out so I thought it might help others. No worries if you don't want to merge.

  • I am the original author of this code and I am giving it freely to the community and Pinchflat project maintainers

@pyrox0
Copy link

pyrox0 commented Apr 6, 2025

I don't need this in my setup. I just have a plain reverse proxy from my caddy machine -> pinchflat, and it just works. This would only be needed if you don't want to run it on a subdomain, but as a subfolder of an existing domain.

@kieraneglin
Copy link
Owner

After thinking on it, I'm probably going to add the reverse proxy stuff to a dedicated page in the docs rather than the README. I'll leave this PR open as a reference/reminder until I do so!

@philips
Copy link
Contributor Author

philips commented Apr 10, 2025

@kieraneglin Yeah, a separate doc makes sense. Thanks

It took me a minute to figure out that Pinchflat needed me to
"strip_prefix" so maybe this will help others.
@philips
Copy link
Contributor Author

philips commented May 28, 2025

@climba03003 thank you for the review. merged your suggestion and rebased.

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.

4 participants