search

NREL / api-umbrella

Open source API management platform
MIT License
2.03k stars 325 forks source link

Make connection header clearing conditional on whether or not request is upgrade #219

Open brylie opened 8 years ago

brylie commented 8 years ago

In issue #198 - Proxy Real-time APIs? - @GUI stated:

we clear the Connection header when proxying requests. This is done so we can support backend keepalive connections (see nginx's keepalive docs).

[to support websockets], we probably need to make clearing that header conditional on whether it's an upgrade request or not. That should be fairly simple, but there may be other small tweaks like this necessary in the stack to get websockets fully working.

This issue is opened to track the progress of any work to make connection header clearing optional.

sarbjit-longia commented 6 years ago

Did we end up implementing this feature or not. I am looking to use this for websockets backend. I was wondering if this product is tested using websockets or not.