uNetworking / uWebSockets

Simple, secure & standards compliant web server for the most demanding of applications
Apache License 2.0
17.3k stars 1.75k forks source link

CONNECT and HTTP Proxy #1676

Open Napolitain opened 10 months ago

Napolitain commented 10 months ago

I tried to setup a server with .connect("/") endpoint, to support HTTPS proxy. I dont think it works at all though, and I'm not even sure that method in the library is supposed to do that.

I'm building an app that requires capturing CONNECT event, so I'm curious to know if there is an easy enough way (in other words, having an api on HTTP level).

Because of the passthrough nature of the server, it is particularly important to have high performance, so I'm inclined to choose uWebSockets.

Here's my starting point :

   auto app = uWS::App({
        "",
        "",
    }).connect("/", [](auto *res, auto *req) {
        // When a client connects to the HTTP proxy server. It should answer 200 OK.
        LOG(INFO) << "CONNECT";
        res->writeStatus("200 OK")->end();
    }).get("/", [](auto *res, auto *req) {
        // When a client connects to the HTTP proxy server. It should answer 200 OK.
        LOG(INFO) << "GET";
        res->writeStatus("200 OK")->end("Hello World!");
});

Get on localhost:8080 logs GET, get on google with HTTP/HTTPS proxy set to localhost:8080 doesnt log CONNECT (though, it does sends CONNECT according to Wireshark).

Is this feature lacking ? Or is it simply too high level of a library (I know CONNECT is a bit inbetween HTTP and plain TCP connection)

Thanks for clarifying

uNetworkingAB commented 10 months ago

https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/CONNECT

That's probably a lack in the Http router, we probably only support simple URLs, not "www.example.com:443" kind of lines.

We could definitely make the router more generic, but I think you will have better luck with a proper proxy project instead of uWS. Proxying is not really a considered case for uWS. We don't even have client support, at all.

uNetworkingAB commented 10 months ago

I marked this as out of scope, but the Http router fix itself is probably in-scope. So we can keep this issue open. Http router needs a rewrite at some point anyways.

uNetworkingAB commented 8 months ago

Do people expect something like this:

GET https://google.com/test/yes

to be matching with app.get("/test/yes", ..)

and have something like req.getBaseURL() to return "https://google.com"

? Or how does Express do it?

uNetworkingAB commented 8 months ago

probably better to use parameter routes to catch the domain like so:

app.get("/:domain/test/yes", ..)

or

app.get("/:domain/*", ..) and use req.getWildcard() for the local URL