summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | tor-hsservice: Remove outdated IPT manager TODOs.Gabriela Moldovan2024-09-091-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | These TODOs were addressed a while ago (when we introduced `IptManager::ipt_errors`).
| * | | | tor-hsservice: Remove dead_code allows.Gabriela Moldovan2024-09-091-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | These aren't dead code anymore, with the exception of `PublisherStatusSender::send_recovering`, which isn't used.
| * | | | tor-hsservice: Log the onion svc status.Gabriela Moldovan2024-09-091-1/+15
| | | | | | | | | | | | | | | | | | | | We now log the onion service status on change.
| * | | | tor-hsservice: Remove an outdated TODO.Gabriela Moldovan2024-09-091-2/+0
| | | | | | | | | | | | | | | | | | | | This is already implemented.
| * | | | tor-hsservice: Move a misplaced TODO.Gabriela Moldovan2024-09-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This TODO was added in !2353 and was supposed to be about reporting a broken/degraded onion service status if the restricted discovery config watcher fails.
* | | | | Merge branch 'socks-typo' into 'main'Nick Mathewson2024-09-101-2/+3
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | Fix a typo in WRONG_PROTOCOL_PAYLOAD. See merge request tpo/core/arti!2403
| * | | | Fix a typo in WRONG_PROTOCOL_PAYLOAD.Pier Angelo Vendrame2024-09-101-2/+3
|/ / / /
* | | | Merge branch 'bytes-deficit' into 'main'Ian Jackson2024-09-1014-36/+104
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Add a deficit field to tor_bytes::Error::Truncated Closes #1592 See merge request tpo/core/arti!2390
| * | | tor-bytes: Error::Truncated: mark the deficit as Sensitive (fmt)Ian Jackson2024-09-101-2/+4
| | | |
| * | | tor-bytes: Error::Truncated: mark the deficit as SensitiveIan Jackson2024-09-103-1/+8
| | | | | | | | | | | | | | | | | | | | Suggested in https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2390#note_3072975
| * | | tor-bytes: Add a deficit field to Error::Truncated (fmt)Ian Jackson2024-09-102-2/+6
| | | |
| * | | 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.
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 17:11:15 +0000