aboutsummaryrefslogtreecommitdiff
path: root/crates/arti-rpc-client-core
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge branch 'prop351-refs' into 'main'Jim Newsome2024-10-031-1/+2
|\ | | | | | | | | | | | | Replace references to prop351 with socks-extensions.md Closes #1629 See merge request tpo/core/arti!2492
| * Replace references to prop351 with socks-extensions.mdNick Mathewson2024-10-021-1/+2
| | | | | | | | | | | | | | Now that the proposal is implemented and merged into the specs, the proposal itself is only historical. Closes #1629.
* | Merge branch 'ffi_seal' into 'main'Nick Mathewson2024-10-021-34/+41
|\ \ | |/ |/| | | | | artilib: Take a different approach to sealing a pair of traits. See merge request tpo/core/arti!2472
| * artilib: Take a different approach to sealing a pair of traits.Nick Mathewson2024-10-011-34/+41
| | | | | | | | This resolves a "TODO MSRV" comment.
* | Merge branch 'rpc_unexpected_reply' into 'main'Nick Mathewson2024-10-026-46/+121
|\ \ | | | | | | | | | | | | | | | | | | rpclib: Unify code for internally generated requests Closes #1587 See merge request tpo/core/arti!2456
| * | rpclib: Add a #[source] annotation.Nick Mathewson2024-10-021-1/+1
| | |
| * | Apply 1 suggestion(s) to 1 file(s)Nick Mathewson2024-10-021-1/+1
| | | | | | | | | Co-authored-by: gabi-250 <[email protected]>
| * | update cbindgen warnings.Nick Mathewson2024-09-251-0/+1
| | |
| * | rpclib: Unify code internally generated requestsNick Mathewson2024-09-254-45/+112
| | | | | | | | | | | | | | | | | | | | | | | | We now have a single type to represent the error "A request that we made internally didn't get a result we expected." The functions to generate these requests are now centralized too. Closes #1587.
| * | rpclib: Implement Display for Utf8CString.Nick Mathewson2024-09-251-0/+7
| | |
| * | rpc: Finish an incomplete sentence in a doc.Nick Mathewson2024-09-251-1/+1
| | |
* | | tor-socksproto: Handle 0-byte reads (EOF) correctlyIan Jackson2024-10-011-1/+1
| | | | | | | | | | | | | | | | | | | | | This restores the functionality of socks users: detect closed sockets. 0c595818f713916d94b7b0e4062f953fad7c9799 which we reverted as part of rebasing this branch onto main.
* | | arti-rpc-client-core: Use new tor-socksproto API (fmt)Ian Jackson2024-10-011-2/+1
| | |
* | | arti-rpc-client-core: Use new tor-socksproto APIIan Jackson2024-10-012-33/+17
| | |
* | | Apply deferred rustfmt churnIan Jackson2024-10-011-3/+2
| | |
* | | tor-socksproto: Move `handshake` to be a trait methodIan Jackson2024-10-011-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This deduplicates some docs and eliminates the two wrapper functiosn for `run_handshake`, which is now just `handshake`. We're going to make other API breaks too, and this isn't going to be the primary API, so we might as well do this. Proper description of the semver breakage will come at the end when it's all done.
* | | Revert "socks users: detect closed sockets."Ian Jackson2024-10-012-11/+1
| | | | | | | | | | | | This reverts commit 0c595818f713916d94b7b0e4062f953fad7c9799.
* | | Revert "rpclib: read SOCKS _after_ processing."Ian Jackson2024-10-011-12/+12
| |/ |/| | | | | This reverts commit 8da8b88481c26d49ac96e79165538e01237ec9ed.
* | Bump arti- and tor- crates to 0.23.0Nick Mathewson2024-09-301-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Per our policy, every one of these gets a minor bump. Generated with: ``` for crate in $(./maint/list_crates | grep '^\(tor\|arti\)-' ); do cargo set-version --bump minor -p $crate; done ``` (Note the use of `-` at the end end of the grep pattern to prevent matching the `arti` crate.)
* | Minor bumps on non-tor/arti cratesNick Mathewson2024-09-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These crates are not in the tor/arti namespace, but we have given them MSRV bumps: ``` oneshot-fused-workaround slotmap-careful fslock-guard hashx equix caret fs-mistrust safelog test-temp-dir retry-error ``` We are counting this as a breaking change. Since all of these crates are at 0.x.x, we have indicated the breaking change with a minor version bump. This commit was generated with the following script: ``` BUMPS=" oneshot-fused-workaround slotmap-careful fslock-guard hashx equix caret fs-mistrust safelog test-temp-dir retry-error " for crate in $BUMPS; do cargo set-version --bump minor -p $crate; done ```
* | rpclib: Note a key property of RPC error returns.Nick Mathewson2024-09-262-8/+7
| |
* | Merge branch 'msrv-1.77-and-rusqlite-update' into 'main'David Goulet2024-09-261-1/+1
|\ \ | | | | | | | | | | | | Upgrade MSRV to 1.77 , and rusqlite to 0.32.1 See merge request tpo/core/arti!2451
| * | Upgrade MSRV to 1.77Nick Mathewson2024-09-251-1/+1
| | | | | | | | | | | | This will allow us to upgrade to the latest version of rusqlite.
* | | Merge branch 'upgrades-20240925' into 'main'Nick Mathewson2024-09-261-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Upgrade dependencies in preparation for next week's releases. See merge request tpo/core/arti!2450
| * | | Upgrade to derive_more version 1.0.0Nick Mathewson2024-09-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | The `derive_more` crate broke backward compatibility with this version, so this change involved quite a few manual fixups. With luck, they'll keep compatibility for some while in the future.
* | | | rpclib: read SOCKS _after_ processing.Nick Mathewson2024-09-251-12/+12
| |/ / |/| | | | | | | | | | | | | | | | | | | | When doing the client handshake, we need to read data _after_ we check for stuff to send, since the client speaks first in the socks protocol. Without this patch, the socks handshake just stalls.
* | | Merge branch 'socks_read_zero' into 'main'Nick Mathewson2024-09-252-1/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | socks users: detect closed sockets. (TROVE-2024-011) Closes #1635 See merge request tpo/core/arti!2447
| * | | socks users: detect closed sockets.Nick Mathewson2024-09-242-1/+11
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Without this check, our socks code can enter an infinite loop if a socket is closed at the wrong time. Resolves TROVE-2024-011. Fixes #1635.
* | | Merge branch 'rename_singleton_id' into 'main'Jim Newsome2024-09-251-2/+2
|\ \ \ | | |/ | |/| | | | | | | | | | | | | rpc: Rename SingletonId to SingleIdResponse Closes #1585 See merge request tpo/core/arti!2448
| * | rpc: Rename SingletonId to SingleIdResponseNick Mathewson2024-09-241-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling it "singleton" might have suggested that it was using the [singleton pattern](https://en.wikipedia.org/wiki/Singleton_pattern), which it isn't. (Renaming done with rust-analyzer and double-checked with `git grep`.) Closes #1585.
* | | Run fixup-features in preparation for upcoming release.Nick Mathewson2024-09-251-1/+1
| |/ |/|
* | rpclib: Use prop351 protocol to open streams.Nick Mathewson2024-09-183-11/+16
|/ | | | | | | | | | | Now that prop351 is what Arti speaks, it's what the rpclib needs to provide. Note one change in particular: the `isolation` string is no longer an optional argument when opening a stream. (With prop351, there is no longer such a thing as an "absent" isolation string, and we don't want to imply that there is a difference between None and "".)
* Bump MSRV from 1.70 to 1.75.Wesley Aptekar-Cassels2024-09-161-1/+1
|
* rpclib: Remove TODO about connect=>stream renaming.Nick Mathewson2024-09-101-4/+0
| | | | | AFIACT, all of this renaming is done. I've grepped for `[Cc]onn` and didn't find any more identifiers that needed to be renamed.
* rpclib: Rename "connect" to "open_stream" as appropriate.Nick Mathewson2024-09-103-21/+21
|
* rpclib: Rename connect.rs to stream.rsNick Mathewson2024-09-102-2/+2
| | | | | | (This is about DataStreams, so we should make it less confusing. This commit is _just_ about the file renaming, with no associated function renaming.)
* 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-091-2/+2
| | | | | Ticket #1509 will probably get rid of this constant, but for now we may as well put it in one place.
* 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-091-6/+7
| | | | | | | 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-091-0/+1
|
* 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.
* rpclib: FFI for opening data streams.Nick Mathewson2024-09-0911-7/+408
| | | | | 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-095-4/+339
| | | | | | Requires #1523. Implements #1524.
* Bump all the unstable tor- and arti- crates to 0.22.0.Gabriela Moldovan2024-09-031-2/+2
| | | | | | | | | | Done using: ``` for crate in $(./maint/list_crates | rg '^(tor|arti-)'); do cargo set-version -p $crate 0.22.0 done ```
* Fix typosDimitris Apostolou2024-09-035-5/+5
|
* rpclib: Remove in_mut_ptr_opt, and document why.Nick Mathewson2024-08-273-37/+22
|
* rpclib, ffi: Allow simultaneous calls to `arti_rpc_handle_wait()`Nick Mathewson2024-08-274-13/+25
| | | | | | | | | | | | | | Its underlying function previously took `&mut RpcHandle`, which was an accident waiting to happen. Now it takes `&RpcHandle` and includes a Mutex to prevent multiple threads from waiting for updates on the same request ID at once. As an alternative, we *could* try to update connimpl::Receiver to allow multiple simultaneous listeners on the same request ID. But that would (I think) require a lot more bookkeeping, and thus would be a bit more error-prone. Closes #1532.
* rpclib: Allow ConnectionClosed in arti_socket_closed testNick Mathewson2024-08-211-0/+9
| | | | | | | | | | | | | | This test is supposed to check the case where we launch a bunch of requests but the socket is dropped before we get a response. Different operating systems handle 'drop' a bit differently, and schedule threads differently, so we shouldn't be surprised at slightly different errors here. In particular, OSX is more prone to deliver the close as a proper "close" rather than an EPIPE or such, which previously broke this test. Closes #1510.
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 06:42:19 +0000