mirrors/arti.git/crates/tor-proto/src/channel, branch arti-v2.1.0mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v2.1.02026-02-26T17:46:12ZMerge branch 'relay-channel-fixes' into 'main'2026-02-26T17:46:12ZDavid Gouletdgoulet@torproject.org2026-02-26T17:46:12Zurn:sha1:0c32b749546d42853b3539344a1560786382de66
relay: Couple fixes related to channel creation
See merge request tpo/core/arti!3726proto: The AUTHENTICATE cell requires the SHA256 RSA identity digest2026-02-26T16:40:59ZDavid Gouletdgoulet@torproject.org2026-02-26T14:00:48Zurn:sha1:259f29798dd832cf5013ba7a3c5d888eebd290fb
Before this commit, we would use the RsaIdentity which is a SHA1 digest.
We do the same for the peer RSA key.
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Add 'CreateFast/CreatedFast' to relay message filter2026-02-26T14:18:02ZSteven Engleropara@torproject.org2026-02-26T07:19:26Zurn:sha1:74b48cd7c4d1a89b25c972c14c97543add1ff760tor-proto: improve debug tracing message2026-02-24T23:11:42ZSteven Engleropara@torproject.org2026-02-24T23:08:32Zurn:sha1:bfeeda0a524cc17ad3580fa51fed7f5f7558686f
Before:
```text
Completed handshake with Ed25519Identity { ClByyE4zT+p3dudeCVkUfVA1zp2y2sOUe7u/HHLliWU } [(RsaIdentity { $547bb37f9cd8079eaf1cfcb73fff5887d6f58138 }, [9, 111, 131, 62, 212, 144, 149, 88, 211, 87, 175, 114, 18, 111, 189, 36, 205, 21, 66, 205, 217, 176, 139, 130, 70, 211, 40, 130, 79, 29, 144, 179])]
```
After:
```text
Completed handshake with ed25519:wo9NA5RXkPAc7sB2301JBpvdk1RvSOGZEzgwuWP/msA [$feebaba2b6faf53ae1dc8dda8681ce026537dfa4]
```
tor-proto: split field into two fields2026-02-24T23:05:14ZSteven Engleropara@torproject.org2026-02-24T22:58:22Zurn:sha1:bdd4e0506cd792a144968ef625990375fc71687a
These were previously in a single `Option`, but now that the `Option`
was removed, I think it's nicer to make these separate fields.
tor-proto: remove unnecessary `Option`s2026-02-24T22:53:37ZSteven Engleropara@torproject.org2026-02-24T22:46:18Zurn:sha1:719093fdb1a74e862166784c2f3e94125b1df360
I don't think that the `Option`s are needed anymore, since
unauthenticated channels no longer transition through the
`VerifiedChannel` state.
proto: Filter the IP address in the channel's OwnedChanTarget2026-02-19T20:57:29ZDavid Gouletdgoulet@torproject.org2026-02-19T20:45:00Zurn:sha1:278ce607162c75bf359e24ff120322a83e3e3e1d
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Reuse the RelayIds builder during the handshake2026-02-19T20:57:29ZDavid Gouletdgoulet@torproject.org2026-02-19T20:21:56Zurn:sha1:7abdd4467281715a4e7b29d4f092cbdb41303817
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Pass PeerAddr at the channel handshake finish for initiators2026-02-19T20:57:29ZDavid Gouletdgoulet@torproject.org2026-02-19T15:11:44Zurn:sha1:5bc019e82fdc7c8eac940c1c9d7cbd6227fe6ea6
Responder relay handshake requires the peer address at the very start as
it sends its NETINFO right away.
For initiators, we only need it during the finalization process which is
when the NETINFO is sent and the Channel is created.
Signed-off-by: David Goulet <dgoulet@torproject.org>
proto: Use constructor for PeerInfo instead of Builder2026-02-19T20:57:29ZDavid Gouletdgoulet@torproject.org2026-02-19T14:36:56Zurn:sha1:50084d13d3a81923f7f2319cd23d1d0b649e30fc
Signed-off-by: David Goulet <dgoulet@torproject.org>