Rewire upstream_peer correctly with forked pingora

memorysafety / river

This repository is the home of the River reverse proxy application, based on the pingora library from Cloudflare.

https://www.memorysafety.org/initiative/reverse-proxy/

Apache License 2.0

1.72k stars 101 forks source link

Rewire upstream_peer correctly with forked pingora #46

Closed jamesmunns closed 3 months ago

jamesmunns commented 3 months ago

This requires https://github.com/cloudflare/pingora/pull/304

In the current main branch, the TLS_SNI stuff is just kind of... wrong. It was a placeholder that I didn't realize I left in when merging #45.

This wires up things correctly, but would require the PR mentioned above.

jamesmunns commented 3 months ago

There is still some ongoing discussion in https://github.com/cloudflare/pingora/pull/304, but I'll go ahead and merge this to unblock further progress, and refactor to take in the decision made upstream.