mirrors/arti.git/crates/tor-proto/src/circuit, branch arti-v0.0.3 mirror of https://gitlab.torproject.org/tpo/core/arti http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v0.0.3 2021-12-16T21:01:50Z tor-proto: Allow one meta-cell handler at a time. 2021-12-16T21:01:50Z Nick Mathewson nickm@torproject.org 2021-12-16T21:01:50Z urn:sha1:64257a10fae8d062a20c6fcc0c7c7787390e6e13 Previously the code would let us try to install a meta-cell handler before the old one was done, leading to possible confusion. Merge branch 'ct_sendme_tags' into 'main' 2021-12-16T20:29:01Z eta tor@eta.st 2021-12-16T20:29:01Z urn:sha1:6a12f388f288dc7b9ea82d9e9952d98c19be6402 tor-proto: use const-time eq on sendme tags. See merge request tpo/core/arti!201 tor-proto: set HalfStream::connected_ok right. 2021-12-16T16:48:39Z Nick Mathewson nickm@torproject.org 2021-12-16T16:48:39Z urn:sha1:51c4ca73f265558df6ae0c6766a02b228b8404be Previously we'd always set it to true, allowing one CONNECTED per half-closed stream even if the stream had already received a CONNECTED cell. This resolves an XXXX. tor-proto: replace a streammap XXXX with a ticket. 2021-12-16T16:33:50Z Nick Mathewson nickm@torproject.org 2021-12-16T16:33:50Z urn:sha1:a09fbc9f271ce6ec474126db4415f7515bfd2b13 tor-proto: use const-time eq on sendme tags. 2021-12-16T16:06:30Z Nick Mathewson nickm@torproject.org 2021-12-16T16:06:30Z urn:sha1:d794355b077cf9aa7399c54e650afaaf548cdea4 There's no known attack here, but it's best practice to always compare digests using a constant-time comparison operator. This resolves an XXXX comment. Extend trace messages for destroy/truncated reasons. 2021-12-15T16:33:48Z Nick Mathewson nickm@torproject.org 2021-12-15T16:33:48Z urn:sha1:f73840544cdd79ce839b5ae0958d85db45f9b1ce It makes sense to put the method for human-readable strings onto the type itself, so that we can format these whenever they occur. I'm choosing the "human_str" method name here, since caret-generated types already have a to_str. I was thinking about using Display, but caret types already implement that. I've also moved the message from "warn!" to "debug!", since these aren't necessarily a problem condition. Merge remote-tracking branch 'origin/mr/191' 2021-12-15T15:46:58Z Nick Mathewson nickm@torproject.org 2021-12-15T15:46:58Z urn:sha1:1cf0b87eb75d45ba36b8b7c5619c13b7ae97e487 Merge branch 'check_put_return' into 'main' 2021-12-15T15:45:08Z eta tor@eta.st 2021-12-15T15:45:08Z urn:sha1:2e0ff665bb09cc979cd88dcce0eb6c1fc45f0a5e Always check whether stream-level SENDMEs are expected. Closes #261 See merge request tpo/core/arti!192 In reactor, use enums on whether to destroy circuits 2021-12-14T22:36:30Z Neel Chauhan neel@neelc.org 2021-12-14T22:36:30Z urn:sha1:043b97fecfe5d1e1da7bf8d1ba014d26df435023 Methodize the destroy circuit reason 2021-12-14T22:26:45Z Neel Chauhan neel@neelc.org 2021-12-14T22:26:45Z urn:sha1:b601d8b1476810e33550b2f007fe91396ed47ba8