-
-
Notifications
You must be signed in to change notification settings - Fork 643
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
fixes issue #6283 #6300
base: main
Are you sure you want to change the base?
fixes issue #6283 #6300
Conversation
@viveak910 you need to sign the Plone Contributor Agreement to merge this pull request. Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement |
✅ Deploy Preview for plone-components canceled.
|
I have already signed the Plone contributor agreement. but I am getting an error about that. please kindly check |
i already signed the aggrement |
@viveak910 You need to wait to be added to the plone organization. |
@viveak910 you're writing to a bot. Please read and follow First-time contributors, especially |
@viveak910 thanks for the fix. if it's not too much touble it would be nice to add in these headers too. These are used in logging to ensure the original IP address is recorded or perhaps for routing purposes.
|
will do @djay |
Hey, I fixed the issue with the Host header! The problem was that the proxy was sending requests with the wrong Host, so I updated the onProxyReq function to set it correctly to the API backend (api.myapp). Now, requests are going to the right place, and the “connection refused” error should be gone.
Let me know if it works for you!