-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Caddy 2.6.2 handling of strip_prefix is broken #5312
Comments
Thanks for opening an issue! We'll look into this. It's not immediately clear to me what is going on, so I'll need your help to understand it better. Ideally, we need to be able to reproduce the bug in the most minimal way possible. This allows us to write regression tests to verify the fix is working. If we can't reproduce it, then you'll have to test our changes for us until it's fixed -- and then we can't add test cases, either. In particular with your case, it would be helpful to remove I've attached a template below that will help make this easier and faster! This will require some effort on your part -- please understand that we will be dedicating time to fix the bug you are reporting if you can just help us understand it and reproduce it easily. This template will ask for some information you've already provided; that's OK, just fill it out the best you can. 👍 I've also included some helpful tips below the template. Feel free to let me know if you have any questions! Thank you again for your report, we look forward to resolving it! Template
Instructions -- please heed otherwise we cannot help you (help us help you!)
Example of a tutorial: Create a config file: |
That rewrite is normal. If you don't have a file that exists at Keep in mind that |
Thanks Francis |
1. Environment1a. Operating system and versionDocker container built using:
1b. Caddy version (run
|
Well, what also matters is what exists on disk. Do you have a |
I only have |
copying content from Caddy logs still show: |
Is there any workaround I could use? |
Sorry, I don't have time right now to look deeper into this, but I will as soon as I can. I'll need to play around with this to trace what's going on with the rewrites and paths, before I can give a more informed answer. |
I'm still fighting with it. EDIT: |
I was able to work around this issue by creating my own variable outside of
|
Well, But I'll take a closer look soon, when I'm less busy. |
Sorry for the delay on this. Forgot to come back to this. I think I'll close this. There's no bug here, really. Like I said earlier, Your workaround using
Using |
Hi,
today I had to update to Caddy 2.6.2 and a config which worked just fine with 2.5.2, stopped.
I've looked at #5073 and #4801 and even though they are closed, it seems they didn't fix it or my use case is different, however I don't see how.
The config in question:
When calling URI
/v1/auth/access_token
the PHP server is called with/index.php
.I've looked through the docs and it seems there was no change between version 2.5.2 and 2.6.2 regarding
strip_prefix
. Did something else change?The text was updated successfully, but these errors were encountered: