agent-substrate / agent-substrate/substrate

Unify `atenet -> atunnel` traffic path

Đang mở
#1,208 3 bình luận 0 reaction 0 người được giao Xem trên GitHub
area/network kind/cleanup
Ngôn ngữ chính
Go
Star
1.8k
Fork
316
Merge trung bình
2 ngày 43 phút
Pull request đã merge (30 ngày)
287

Mô tả

Right now the ingress traffic paths in Substrate are a bit all over the place. Currently `atenet` can call into the `atunnel` in one of 2 ways:
1. :443 HTTP reverse proxy: parses and authorizes every HTTP request then proxies it to the actor
2. :444 CONNECT proxy: authorizes the CONNECT request once, opens actorIP:port, then ferries raw bytes bidirectionally.

It's not clear to me that we need the first path at all.

Parsing L7 on every request leads to worse performance overall, it is equivalent to an L7 sidecar vs the ambient model.
1. It leads to changes such as the [following](https://github.com/agent-substrate/substrate/pull/1183) which make atunnel ever more complicated.
2. There is no current or planned L7 per request behavior we want to put in atunnel, and ideally we'd keep it that way.

Edit: One additional datapoint is that there is a design initiative right now to pack multiple actors into a single worker, see [comment](https://github.com/agent-substrate/substrate/issues/853#issuecomment-5358052607). The `tunnel` performance will become even more critical in that world.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.