httpProxy

[sjorsdonkers]

Adding a http proxy with similar logging for http requests to /json/list /json/version etc.

[krichprollsch]

I would try to avoid using else.
And since I consider the function ws main usage is to proxify a web socket. I would rewrite this way:

if !websocket.IsWebSocketUpgrade(r) {
	// If not a WebSocket upgrade request, handle it as an HTTP request
	// We may expect requests like /json/list or /json/version
	if err := httpProxy(cdpurl, w, r); err != nil {
		slog.Error("httpProxy", slog.Any("err", err))
	}
	return
}

ws, err := upgrader.Upgrade(w, r, nil)
// ...

[krichprollsch]

The goal of the PR is to connect to chrome w/o having to copy the unique ws url?

[krichprollsch]

Suggested change

	req, err := http.NewRequest(r.Method, targetURL, tee_body)
	req, err :=http.NewRequestWithContext(r.Context(), r.Method, targetURL, tee_body)

This way if the caller cancel the original request, you will abort the proxy one.

[sjorsdonkers]

The goal of the PR is to connect to chrome w/o having to copy the unique ws url?

The goal is to see whether we can get Chrome DevTools working using remote-debugging.

With this change Lightpanda shows up in chrome, but the inspect button does not work yet, we do see in this proxy that chrome is sending unimplemented cdp commands.