mirrors/arti.git/crates/tor-proto/src/congestion/params.rs, branch arti-v1.4.4mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.4.42025-04-23T18:38:25Ztor-proto: expand docs for `CongestionWindowParams::set_sendme_inc`2025-04-23T18:38:25ZSteven Engleropara@torproject.org2025-04-17T03:19:44Zurn:sha1:b0deeca5b39c53c8523cc6f10bd34bda8862528etests: Add CC ntorv3 negotiation unit test2025-04-23T18:36:28ZDavid Gouletdgoulet@torproject.org2025-01-22T20:12:17Zurn:sha1:e8b72b7400aa7f34cf9465818307f59433f4e8f0
Also add one for the sendme_inc validity function.
Part of #1817
Signed-off-by: David Goulet <dgoulet@torproject.org>
circ: Apply possible subprotocol changes to circ params2025-04-23T18:36:28ZDavid Gouletdgoulet@torproject.org2025-01-22T16:41:17Zurn:sha1:e1064562c3f718325b9930d7d6120be38a445b27
Congestion control can change the circuit parameters if the relay we are
negotiating with doesn't support FlowCtrl=2.
This commit adds a function in the circuit builder that will apply any
changes to the circuit parameters of the hop based on the hop protocol
values. For now, only congestion control applies.
Part of #1817
Signed-off-by: David Goulet <dgoulet@torproject.org>
congestion: Setup a fallback algorithm in the params2025-04-23T18:36:28ZDavid Gouletdgoulet@torproject.org2025-01-22T16:17:21Zurn:sha1:f4a0429dbb20f3087c9591344f8d701cef77a7e7
CircParameters is built before path selection and thus once we start
building the hops, we can't access the consensus values that were used
to build it in the first place.
For congestion control, we require a fallback algorithm in case the hop
doesn't support FlowCtrl=2.
This commit adds a "fallback_alg" to the CC parameters which will be
used for this exact case.
Part of #1817
Signed-off-by: David Goulet <dgoulet@torproject.org>
circ: Set the negotiated CC sendme_inc from handshake2025-04-23T18:36:28ZDavid Gouletdgoulet@torproject.org2025-01-22T15:15:57Zurn:sha1:a10054323c10a3b6f20db6389633faf8d7ebc749
When receiving the congestion control response extension, evaluate our
state and set the sendme_inc if valid in our circuit parameters.
For this, a series of helper functions is needed.
Part of #1817
Signed-off-by: David Goulet <dgoulet@torproject.org>
fix: fix typos2025-01-30T08:39:30ZDimitris Apostoloudimitris.apostolou@icloud.com2025-01-30T08:39:30Zurn:sha1:6371829c976f6a5b77d46bf977f369a915f2ced3congestion: Use getters for all parameters2025-01-16T14:16:01ZDavid Gouletdgoulet@torproject.org2025-01-15T15:29:46Zurn:sha1:2fd9a4d5b814b3a0aa031313d081f0d306016908
Allows us to remove the pub.
Signed-off-by: David Goulet <dgoulet@torproject.org>
congestion: Remove Default from every parameters2025-01-16T14:16:01ZDavid Gouletdgoulet@torproject.org2025-01-15T14:13:00Zurn:sha1:b4378a03332715d34de75617ee43775690fdd9b0
Important to enforce that every field is explicitely set so we avoid
forgetting fields.
Signed-off-by: David Goulet <dgoulet@torproject.org>
congestion: Make the cc_alg values a caret_int for better code semantic2025-01-16T14:16:01ZDavid Gouletdgoulet@torproject.org2025-01-09T15:00:13Zurn:sha1:62f8f85ebd5d7972b8e7c6cd493d84eb8f87ba8f
Signed-off-by: David Goulet <dgoulet@torproject.org>
congestion: Remove FixedWindow start window default value2025-01-16T14:16:01ZDavid Gouletdgoulet@torproject.org2025-01-08T19:53:39Zurn:sha1:14ee43b3ae64d2c64cfa14c2cd054965c5b7c366
In unit tests, we set a 1000 by default but else, we don't set a default
so our implementation doesn't forget to set the right value from the
consensus.
Signed-off-by: David Goulet <dgoulet@torproject.org>