mirrors/arti.git/crates/tor-hsservice/src/svc.rs, branch ios-sqlite-hacksmirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=ios-sqlite-hacks2023-09-18T10:53:51Ztor-hsservice: Unify ids as IptLocalId replacing IntroPointId2023-09-18T10:53:51ZIan Jacksonijackson@chiark.greenend.org.uk2023-09-18T10:42:26Zurn:sha1:6a1bd5b2be447e96c33f4b3cb4d877ec085ab205
IntroPointId was RelayIds but that's wrong, because there can be
different IPTs at the same relay - but also because an established IPT
might change its RelayIds.
Use IptLocalId instead, which I think is the type we decided to use
for this, and which is, conveniently, Copy.
And change the variable names to match, everywhere.
Specifically: in places where an intro point is implied (ipt_mgr.rs,
ipt_establish.rs) use the name `lid` everywhere, like in ipt_mgr.rs.
Elsewhere, use `ipt_lid`.
(We could use a longer name, but in that case it should be changed in
ipt_mgr.rs too.)
No actual functional change in this commit.
tor-hsservice: Have Establisher provide more information to Manager2023-08-29T16:53:32ZIan Jacksonijackson@chiark.greenend.org.uk2023-08-29T13:40:38Zurn:sha1:4c3e71ec870bcff0da8458708569b8190099f28c
The establisher has a netdir; the manager generally doesn't. So it
will be convenient for the establisher to provide the linkspecs and so
on.
tor-hsservice: Make wait_for_netdir* more widely available.2023-08-25T16:53:58ZGabriela Moldovangabi@torproject.org2023-08-23T00:05:11Zurn:sha1:2ced596f1c446cc4af2632d4e0c7ff9e6d7b2f09
`wait_for_netdir` is needed by the descriptor publisher (to compute the
initial netdir).
tor-hsservice: Expose the ipt_establish and publish modules2023-08-23T09:04:31ZIan Jacksonijackson@chiark.greenend.org.uk2023-08-15T17:53:21Zurn:sha1:b39b92b856a23951120a93e1161a6494b99495ae
I still think putting these in svc/ module doesn't make much sense.
Anyway, we can leave them there for now, but I need to get at them
from crate::ipt_establisher.
Use "typos-cli" to fix a bunch of typos.2023-08-22T20:23:51ZNick Mathewsonnickm@torproject.org2023-08-22T13:06:44Zurn:sha1:7a37641aeb03623a707bd7f3910231c997d66bc9HSS: Refactor RendRequest so we can return a stream of it.2023-08-22T14:50:43ZNick Mathewsonnickm@torproject.org2023-08-18T17:06:42Zurn:sha1:ec6721ec9412fa08a47855965c6a1e0ceb6e63fe
We need a type that holds a rend_handshake::IntroRequest object
internally, but where we don't materialize that object from the
Introduce2 message inside the MsgHandler, since that's more crypto
than we want to put in that task.
Merge branch 'hss-err' into 'main'2023-08-18T13:39:20Zgabi-250gabi@torproject.org2023-08-18T13:39:20Zurn:sha1:15f9da4d0ea60092aa1e231816e7dd69df10ba0f
tor-hsservice errors: Introduce more error types
See merge request tpo/core/arti!1515tor-hsservice errors: Distinguish operational errors by context2023-08-17T12:28:44ZIan Jacksonijackson@chiark.greenend.org.uk2023-08-17T12:04:53Zurn:sha1:360c90a9d8ec14c713560bc089f9706527056d31
At the very least, I need FatalError to be distinct:
IptEstablisher::new ought not to fail unless everything is terrible.
Add a the Spawn variant to FatalError (that we'll need soon) and the
Bug variant (which it seems likely we might need).
This also gets rid of the crate-level Result alias.
tor-hsservice errors: Return ReconfigureError from reconfigure2023-08-17T12:28:44ZIan Jacksonijackson@chiark.greenend.org.uk2023-08-17T12:01:00Zurn:sha1:e1405cdb1191de87bd771c7644233e11c1a668a8
This is what we do elsewhere.
tor-hsservice errors: Use Bug for methods that oughtn't to fail2023-08-17T12:19:21ZIan Jacksonijackson@chiark.greenend.org.uk2023-08-17T12:02:34Zurn:sha1:5e047da9cc4017433b7282858a33543587a976ee
If the service encouters operational errors, surfacing them here is
not helpful. So these methods ought to work, if they weren't called
erroneously.