summaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | tor-netdoc: Make Nickname have its own parsing error typeIan Jackson2026-04-206-17/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will let us use it in more places, more sensibly. Sadly it means a lot of syntactic vinegar at many of the call sites in the old parser, which has very janky ideas about how errors should work.
* | | | | tor-netdoc: semver.md for InternetHost and HostnameIan Jackson2026-04-201-0/+1
| | | | |
* | | | | tor-netdoc: Tests for InternetHost and HostnameIan Jackson2026-04-201-0/+65
| | | | |
* | | | | tor-netdoc: Introduce InternetHost newtypeIan Jackson2026-04-202-1/+52
| | | | | | | | | | | | | | | | | | | | Tests will come in a moment.
* | | | | tor-netdoc: Introduce Hostname newtypeIan Jackson2026-04-203-0/+82
| | | | | | | | | | | | | | | | | | | | Tests will come in a moment.
* | | | | Merge branch 'stream-close' into 'main'opara2026-04-201-6/+9
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-proto: Small comment and variable name changes See merge request tpo/core/arti!3865
| * | | | | tor-proto: small comment and variable name changesSteven Engler2026-04-071-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | I think this makes the code a little easier to follow.
* | | | | | Merge branch 'rm-var-suffix' into 'main'David Goulet2026-04-201-4/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | proto: Remove dummy suffixes from variable name See merge request tpo/core/arti!3903
| * | | | | | proto: Remove dummy suffixes from variable nameGabriela Moldovan2026-04-201-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I used these `_foo` suffixes to make sense of things during development: at the time, I had multiple "forward" and "backward" types, and I needed to distinguish them without spending too much time thinking of a provisory name. I meant to remove them before upstreaming my branch, but I forgot...
* | | | | | | proto: Remove unnecessary clippy allowGabriela Moldovan2026-04-201-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm removing this so that we don't accidentally suppress new instances of this warning.
* | | | | | | Merge branch 'p2-misc-exactly-1' into 'main'Clara Engler2026-04-202-3/+6
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | tor-netdoc: derives: Three small improvements See merge request tpo/core/arti!3887
| * | | | | | tor-netdoc: derives: Expect an intro itemIan Jackson2026-04-141-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This can't trigger, but when messing about with some new features, I messed up and got a syntax error here. Let's try this feature out, here.
| * | | | | | tor-netdoc: derives: Reformat a long lineIan Jackson2026-04-141-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | No functional change.
| * | | | | | tor-netdoc: derives: Remove an otiose testIan Jackson2026-04-141-1/+1
| | |_|/ / / | |/| | | | | | | | | | | | | | | | T_SIGNATURES is false, just above.
* | | | | | arti-relay: add cgo for tor-protoSteven Engler2026-04-161-1/+1
| | | | | |
* | | | | | arti-relay: add 'flowctl-cc' for tor-circmgrSteven Engler2026-04-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we build client circuits through the circuit manager, we should build them with congestion control support.
* | | | | | Merge branch 'create-fast' into 'main'gabi-2502026-04-166-5/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-proto: Give our RSA identity to the channel reactor See merge request tpo/core/arti!3897
| * | | | | | tor-proto: give our rsa ident to the channel reactorSteven Engler2026-04-166-5/+31
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | This will be needed for ntor handshakes.
* | | | | | Merge branch 'rpc-weak-ids-redux' into 'main'Nick Mathewson2026-04-1611-618/+264
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RPC: implement weak IDs Closes #868 See merge request tpo/core/arti!3883
| * | | | | | rpc: Propagate and use LookupErrors.Nick Mathewson2026-04-133-12/+43
| | | | | | |
| * | | | | | rpc: Adjust invoke_rpc_method API and related lookups.Nick Mathewson2026-04-136-47/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The important thing here is that we need to look up the object _after_ we invoke any non-dispatched methods, since the object might be an expired weak reference. This required some related changes to tests and error types. Additionally, it required us to change the return value of Context::remove, since we can remove objects that are not completely present.
| * | | | | | rpc: new method for upgrading/downgrading/cloning object IDsNick Mathewson2026-04-131-1/+57
| | | | | | |
| * | | | | | rpc: Remove caveats wrt rpc::release.Nick Mathewson2026-04-131-10/+0
| | | | | | |
| * | | | | | rpc: Add context support for adding and removing weak refsNick Mathewson2026-04-135-21/+38
| | | | | | |
| * | | | | | rpc: Restore some weak-reference tests; remove others.Nick Mathewson2026-04-131-199/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The removed tests are ones that no longer make sense given our simplified behavior.
| * | | | | | rpc: allow weak references to be created.Nick Mathewson2026-04-131-9/+39
| | | | | | |
| * | | | | | rpc: rename strong_arena to arena.Nick Mathewson2026-04-131-7/+6
| | | | | | |
| * | | | | | rpc: collapse GenIdx enum to its single variant.Nick Mathewson2026-04-133-37/+12
| | | | | | |
| * | | | | | rpc: Remove notion of separate table/idx space for weak referencesNick Mathewson2026-04-134-325/+38
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We no longer plan to have all weak-refs automatically de-duplicated, for reasons discussed at #868. This lets us discard a lot of code, including code to check for Arc/weak identity. I've disabled some unit tests here, but I'll add them back or rewrite them (or discard them as irrelevant) by the end of the branch.
* | | | | | keymgr: Remove outdated references to derive-adhocGabriela Moldovan2026-04-161-3/+3
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | This updates some outdated references from back when `derive-deftly` was called `derive-adhoc`.
* | | | | keymgr: Use CAPS for metasyntactic variables in macroGabriela Moldovan2026-04-161-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3891#note_3395849
* | | | | hsclient, hsservice: Use unquoted paths in ctor_path attrGabriela Moldovan2026-04-162-3/+3
| | | | |
* | | | | keymgr: Don't require ctor_path attr to be quotedGabriela Moldovan2026-04-162-2/+2
| | | | | | | | | | | | | | | | | | | | This updates a doc and the corresponding test.
* | | | | keymgr: Use unquoted keypair_specifier in testsGabriela Moldovan2026-04-161-1/+1
| | | | |
* | | | | hsservice: Use unquoted type in keypair_specifier attrGabriela Moldovan2026-04-161-2/+2
| | | | |
* | | | | arti-relay: Use unquoted type in keypair_specifier attrGabriela Moldovan2026-04-161-2/+2
| | | | |
* | | | | keymgr: Make KeySpecifier take an unquoted keypair_specifierGabriela Moldovan2026-04-161-2/+2
| | | | | | | | | | | | | | | | | | | | This also makes the macro `beta_deftly`.
* | | | | keymgr: Require keypair_specifier to be a typeGabriela Moldovan2026-04-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I want to make all uses of `keypair_specifier` unquoted, so this can't be a `token_stream` (and in fact, `keypair_specifier` was always meant to be a type).
* | | | | keymgr: Update outdated KeySpecifier docGabriela Moldovan2026-04-161-1/+1
| |/ / / |/| | | | | | | | | | | This attribute is called `keypair_specifier`, not `key_specifier`.
* | | | Merge branch 'chan-unit-tests' into 'main'David Goulet2026-04-1410-284/+661
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | proto: Add relay channel handshake unit tests See merge request tpo/core/arti!3853
| * | | | proto: Remove unused imports in unit testsDavid Goulet2026-04-141-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | proto: Fix relay unit tests with new CreateRequestHandlerDavid Goulet2026-04-141-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The CreateRequestHandler was introduced prior to the relay handshake unit tests so fix them now that we have a way to build one. Signed-off-by: David Goulet <[email protected]>
| * | | | proto: Add a test helper function for building a CircNetParametersDavid Goulet2026-04-143-0/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can't access `NetParameters` in this crate to build from consensus default value so instead add a `defaults_for_tests()` gated function for each parameters object which puts in the defaults from the spec. We'll need CircNetParameters in order to build create request handler for unit tests hence why it is in src/circuit.rs.
| * | | | proto: Move DummyChan/DummyChanProvider to relay channel.rsDavid Goulet2026-04-142-64/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From the reactor.rs to channel.rs, makes more sense and we'll need it in the handshake tests. No behavior change, just code movement and a function rename. Signed-off-by: David Goulet <[email protected]>
| * | | | proto: Remove re-export of MsgBuf in unit testsDavid Goulet2026-04-143-5/+3
| | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | proto: Add channel relay handshake unit testsDavid Goulet2026-04-145-182/+431
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of copying the client unit tests into the channel module, just make both current unit tests run on a client and relay handshake. This required a bit of trickery with type HandshakeConnectFn but works out in the end. It also adds the RelayMsgBuf that wraps a MsgBuf in order to implement CertifiedConn which is very relay only. Signed-off-by: David Goulet <[email protected]>
| * | | | proto: Move test certs module at the topDavid Goulet2026-04-141-31/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just for mental sanity to have all const at the same place and easily readable. No behavior change. Signed-off-by: David Goulet <[email protected]>
| * | | | proto: Move channel client handshake tests into client moduleDavid Goulet2026-04-143-218/+237
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No behavior change. Make the const cells public as they will be useful for more upcoming unit tests especially on the relay side. Signed-off-by: David Goulet <[email protected]>
* | | | | Merge branch 'create-fast' into 'main'opara2026-04-143-23/+74
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | arti-relay: Log the public ntor key See merge request tpo/core/arti!3882
| * | | | arti-relay: improve ntor key loggingSteven Engler2026-04-141-5/+8
| | | | | | | | | | | | | | | | | | | | We only need the max, not the entire sorted list.