-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
try removing the nginx reverse-proxy for BwdServer #3630
Comments
If we stick with this we'll need:
|
|
Need a test with what happens with incorrect content-length:
[edit: tests added] |
|
There was a bug I couldn't figure out when developing this. It turns out Kestrel really does not like you to have the wrong content-length. It manifested as a full 400 error being returned by Kestrel AFTER a response had already happened (just jammed right on the end of the HTTP output stream, and in some cases in the middle of it). I speculate that nginx really didn't care that much which is why we didn't notice the tests were wrong. |
|
|
Yes, no port exists. (in the url field either) |
Note for when I come back to this:
|
Being handled in #3658 |
There's two customers with a canvas that redirects, but it seems neither of them are terribly active, and there aren't any requests to the non-www version of their page anyway. We'll see if they complain. |
Closing this - the remaining work in covered in #3664. |
We have no real use for the nginx proxy for bwdserver, and it just provides another thing to configure and maintain. We need nginx for the apiserver and we used to have the apiserver and the bwdserver in the same process.
Let's try without it.
The text was updated successfully, but these errors were encountered: