Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing SENDME support on circuits #95

Open
dgoulet-tor opened this issue Apr 15, 2021 · 1 comment
Open

Missing SENDME support on circuits #95

dgoulet-tor opened this issue Apr 15, 2021 · 1 comment

Comments

@dgoulet-tor
Copy link

It appears that it is not possible to download a consensus over the ORPort of a relay due to Stem not sending SENDMEs (flow control cell) which means that after 500 cells on the stream (roughly 25k), the transfer will stop and a consensus or all descriptors are bigger than that.

Furthermore, Tor design complicates things because Stem would need to use SENDMEs on a stream and circuit as well which both have different windows.

On a stream, the starting window on a connection is 500 and on a circuit starting window is 1000. Then, tor-spec.txt specifies when/how to send a SENDME at the stream and circuit level.

@atagar
Copy link
Contributor

atagar commented Apr 15, 2021

Interesting! Great catch David, I'd be delighted to merge a patch to address this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants