summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | tor-bytes: Add a deficit field to Error::TruncatedIan Jackson2024-09-109-14/+23
| | | | | | | | | | | | | | | | This will allow us to fix #1592, but it doesn't do so yet.
| * | | Introduce and use tor_bytes::Error::new_truncated_for_test (fmt)Ian Jackson2024-09-101-1/+5
| | | |
| * | | Introduce and use tor_bytes::Error::new_truncated_for_testIan Jackson2024-09-104-21/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is going to want to do something more complicated (as described in the docs). In this commit we change all the tests that are expecting Truncated errors. That reduces noise in the next commit.
| * | | tor-bytes: Document Readable::take_from correctness propertiesIan Jackson2024-09-101-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | This refers to the `deficit` field in Error::Truncated, which is going to appear in a later commit. It seems kinder to my reviewer to add this doc now early in the branch.
| * | | tor-linkspec: reading tests: Use assert_eqIan Jackson2024-09-101-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Error::Truncated is going to become more complicated, and anyway it would be nice to print the values if the test fails. Error is PartialEq now. (Maybe it wasn't when this was written?)
| * | | tor-bytes: advance: Call peek for the error checkIan Jackson2024-09-101-3/+1
|/ / / | | | | | | | | | | | | This removes one construction site of Error::Truncated. We are about to make constructing one of those more fiddly.
* | | Merge branch 'relay-err' into 'main'David Goulet2024-09-096-5/+127
|\ \ \ | | | | | | | | | | | | | | | | arti-relay: add and use `Error`/`ErrorDetail` See merge request tpo/core/arti!2392
| * | | arti-relay: add and use `Error`/`ErrorDetail`Steven Engler2024-09-055-4/+126
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most of this is copied from `arti_client::err`, but with a lot of stuff removed to simplify the error types (for example no `error_detail` feature handling). I tried to keep things generally consistent with arti-client so that error handling will be similar in both crates. `TorRelayBuilder::create()` will likely need to be fallible in the future (for example if spawning a task for the OR port listener fails), so it now returns a `Result<TorRelay, crate::err::Error>` instead of just a `TorRelay`.
| * | | arti-client: minor documentation grammar fixSteven Engler2024-09-051-1/+1
| | | |
* | | | Merge branch 'pr/arti/keymgr/use-keypairs-if-available' into 'main'gabi-2502024-09-0914-24/+129
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | eliminate all but one use of `KeyMgr::get::<HsIdKeypair>()` Closes #1194 See merge request tpo/core/arti!2393
| * | | rename get_keypair_specifier() to keypair_specifier()Adam Joseph F0B74D717CDE8412A3E0D4D5F29AC8080DA8E1E02024-09-094-8/+8
| | | | | | | | | | | | | | | | https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2393#note_3073480
| * | | tor_hsservice use get::<HsIdKey> rather than get::<HsIdKeypair>Adam Joseph F0B74D717CDE8412A3E0D4D5F29AC8080DA8E1E02024-09-094-22/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are three places where we query the KeyMgr for an `HsIdKeypair` but all we really need is the public part. This commit changes those three callsites to instead use `get::<HsIdKey>`. This relies on the previous commit, which makes sure that a request for an `HsIdKey` will always succeed if the keystore has a `HsIdKeypair` with the same service nickname.
| * | | tor_keymgr: teach the KeyStore how to satisfy public key requests using a ↵Adam Joseph F0B74D717CDE8412A3E0D4D5F29AC8080DA8E1E02024-09-091-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | keypair Now that: - KeySpecifier::get_keypair_specifier() can be used to convert the KeySpecifier for a public key into the KeySpecifier for its secret key - ToEncodableKey<Key=PublicKey> has a "type level pointer" to ToEncodableKey<Key=KeyPair> We can use these two features together to automatically satisfy any request to get a public key using the corresponding secret key (if available).
| * | | tor_key_forge::traits::ToEncodableKey: add KeyPair associated typeAdam Joseph F0B74D717CDE8412A3E0D4D5F29AC8080DA8E1E02024-09-093-1/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This comment adds a second associated type `KeyPair` to ToEncodableKey. For a `ToEncodableKey` which represents a (secret) KeyPair, this type is Self. For a `ToEncodableKey` which represents a public key, this is the `ToEncodableKey` whose `Key` is the pair of which this is the public part. This is essentially a "type level pointer" from the ToEncodableKey for a public key to the ToEncodableKey for its secret key.
| * | | tor_keymgr: add get_keypair_specifier() to KeySpecifier, and derive itAdam Joseph F0B74D717CDE8412A3E0D4D5F29AC8080DA8E1E02024-09-096-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds a new method `get_keypair_specifier()` to `KeySpecifier`. This method is used to indicate when one KeySpecifier (e.g. `KP_hs_id`) is the public part of another keypair (e.g. `KS_hs_id`). It will return the containing keypair in this case, and `None` otherwise.
| * | | tor_hsservice: add `impl From<&FooPublicKeySpecifier> for ↵Adam Joseph F0B74D717CDE8412A3E0D4D5F29AC8080DA8E1E02024-09-094-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | FooKeypairSpecifier` instances This adds the following trivial `From` instances: - tor_hsservice: impl From<&HsIdPublicKeySpecifier> for HsIdKeypairSpecifier - tor_hsservice: impl From<&BlindIdPublicKeySpecifier> for BlindIdKeypairSpecifier - tor_hscrypto::pk: impl From<HsBlindIdKeypair> for HsBlindIdKey - tor_llcrypto::pk::ed25519: impl From<ExpandedKeypair> for PublicKey - tor_keymgr::mgr: impl From<TestKey> for TestPublicKey - tor::hscrypto::pk: impl From<HsIdKeypair> for HsIdKey
* | | | Merge branch 'rpc-connect' into 'main'Nick Mathewson2024-09-0920-22/+1097
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | rpclib: Support opening a data stream. Closes #1524 See merge request tpo/core/arti!2373
| * | | Add a TODO RPC call about revising/moving documentationNick Mathewson2024-09-091-0/+6
| | | |
| * | | Re-run cbindben for new warnings and status codesNick Mathewson2024-09-092-43/+16
| | | |
| * | | Add TODO for renaming connect.rs to streams.rsNick Mathewson2024-09-091-0/+4
| | | |
| * | | socksproto: Add a const for suggested buffer length.Nick Mathewson2024-09-094-5/+12
| | | | | | | | | | | | | | | | | | | | Ticket #1509 will probably get rid of this constant, but for now we may as well put it in one place.
| * | | rpc: More specificity surrounding SOCKS-rpc integrationNick Mathewson2024-09-093-5/+57
| | | |
| * | | Suggestion about describing non-RPC behavior of SOCKS protocol from @diziet.Nick Mathewson2024-09-091-1/+3
| | | |
| * | | rpclib: testing for future-proof proxyinfo formatNick Mathewson2024-09-091-9/+65
| | | | | | | | | | | | | | | | | | | | Here we make sure that we can actually skip over other proxy formats in the future.
| * | | arti/rpclib: Tweak proxy_info format.Nick Mathewson2024-09-093-8/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Renamed address to tcp_address, and made it optional, so that later we can have a unix_path, etc. On deser side, add support for unrecognized listener types.
| * | | arti/rpclib: Lowercase our socks5 listener variant.Nick Mathewson2024-09-092-0/+2
| | | |
| * | | rpclib: New error type for unauthenticated connectionsNick Mathewson2024-09-092-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | This is done so that we can make "not authenticated" a non-internal error, under the theory that someday unauthenticated connections might be exposed.
| * | | arti: Add a comment explaining how RPC and SOCKS interactNick Mathewson2024-09-091-0/+181
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This belongs in a spec, but adding things to a spec is slow and fraught. Instead we'll put it here for now and move it later. There are some XXXXs about "finalizing" the design that we need to resolve before we can merge !2373 and implement stream creation in `arti-rpc-client-core`.
| * | | arti: add and document get_rpc_proxy_info method.Nick Mathewson2024-09-091-2/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently this behaves the same as get_proxy_info, but this may change in the future, if we introduce RPC-unsuitable proxy ports. (Also rename the get_proxy_info method to avoid x_ prefix.)
| * | | rpclib: FFI for opening data streams.Nick Mathewson2024-09-0912-7/+409
| | | | | | | | | | | | | | | | | | | | There's a blocking TODO here about exposing socks error codes that I still need to solve.
| * | | rpclib: Support opening a data stream.Nick Mathewson2024-09-097-5/+342
|/ / / | | | | | | | | | | | | | | | Requires #1523. Implements #1524.
* | | Merge branch 'futures-lite' into 'main'gabi-2502024-09-092-1/+6
|\ \ \ | | | | | | | | | | | | | | | | maint/downgrade-dependencies: Update exceptions. See merge request tpo/core/arti!2398
| * | | maint/cargo_audit: Add exception for RUSTSEC-2024-0370.Gabriela Moldovan2024-09-091-0/+5
| | | | | | | | | | | | | | | | Adding an exception for now, as per #1608
| * | | maint/downgrade-dependencies: Update exceptions.Gabriela Moldovan2024-09-091-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the `minimal-versions` failure: ``` error: package ID specification `[email protected]` did not match any packages Did you mean one of these? [email protected] ``` (we don't actually need to un-downgrade `futures-lite` anymore) But we *do* need to un-downgrade `event-listener` (`async-global-executor` needs `event_listener::EventListener` to be `UnwindSafe`, and in `event-listener 3.0.0`, they are not)
* | | Merge branch 'watch-test' into 'main'gabi-2502024-09-091-0/+1
|\ \ \ | |/ / |/| | | | | | | | arti: Temporarily disable a flaky test. See merge request tpo/core/arti!2391
| * | arti: Temporarily disable a flaky test.Ian Jackson2024-09-051-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | This effectively reverts 71e3d52f5aeb34ca7bca80053079a8c7505b99ac, which was itself a revert of disabling this test the first time. Sadly it still doesn't seem to be reliable. We have agreed on IRC to disable it for now.
* | | Merge branch 'deb-via-b' into 'main'Ian Jackson2024-09-051-9/+3
|\ \ \ | |/ / |/| | | | | | | | Use dpkg-buildpackage -uc -b in CI See merge request tpo/core/arti!2368
| * | Remove the `needs: []` from the deb-source jobIan Jackson2024-09-051-1/+0
| | | | | | | | | | | | | | | | | | | | | The effect of this is just to make the job run sooner. But it's actually quite unlikely to fail, and nothing now depends on it, so we should run it later along with the rest of the jobs in the `test` stage.
| * | Run `git-clean -xdff` in deb-source jobIan Jackson2024-09-051-0/+1
| | | | | | | | | | | | | | | This avoids any inherited data (eg from previous jobs) influencing the output.
| * | When building .debs, build only the .debs and don't rebuild the .dscIan Jackson2024-09-051-8/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | * Pass -b to dpkg-buildpackage, meaning "build binaries only". * Don't `need` the source package job * Don't try to import source package artifacts This means we're not building the .debs from the .dsc, but we weren't doing that anyway. (.dscs are IMPO obsolete...)
| * | Pass -uc to dpkg-buildpackage when building .debsIan Jackson2024-09-051-2/+2
|/ / | | | | | | | | | | | | | | | | | | | | By default dpkg-buildpackage signs the output if the d/changelog isn't UNRELEASED. If we ever change the d/changelog, we don't want it to fail because it tries to sign things. In the future, if and when we want to sign things, that probably won't be done directly in gitlab CI when the package is built.
* | Merge branch 'slow-test' into 'main'Ian Jackson2024-09-051-2/+15
|\ \ | | | | | | | | | | | | | | | | | | arti: Make the config watcher debounce interval configurable. Closes #1589 See merge request tpo/core/arti!2387
| * | arti: Make the config watcher debounce interval configurable (fmt).Gabriela Moldovan2024-09-041-1/+9
| | |
| * | arti: Make the config watcher debounce interval configurable.Gabriela Moldovan2024-09-041-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the `reload_cfg::test::watch_multiple` test would take about 3s to run. This test modifies 3 files and waits for the corresponding `reconfigure()` events to fire. Because of the `sleep(DEBOUNCE_INTERVAL)` in `run_watcher`, it would wind up waiting for about 1s for each of them. This makes the event debouncing optional, and disables it in the tests. Closes #1589
* | | Merge branch 'ticket1137_01' into 'main'gabi-2502024-09-0524-668/+1010
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-keys: New crate for Tor key declaration Closes #1137 See merge request tpo/core/arti!2356
| * | | Rename tor-keys crate to tor-key-forgeDavid Goulet2024-09-0422-41/+43
| | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | maint: Add paste tor the tor-keys crateDavid Goulet2024-09-041-1/+2
| | | |
| * | | tor-keys: Use CryptoRng instead of the Core oneDavid Goulet2024-09-042-5/+5
| | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | doc: Fix unresovled links in the commentsDavid Goulet2024-09-042-7/+7
| | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | tor-keymgr: Use tor-keys crate and remove dead codeDavid Goulet2024-09-0418-846/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Everything copied in the previous commits to tor-keys is now removed and tor-keys crate is used accross the code. Minor changes to tor-keys to accomodate this change. Part of #1137 Signed-off-by: David Goulet <[email protected]>
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 19:40:44 +0000