| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | | fs-mistrust: link to documentation about "obeying constraints" | Nick Mathewson | 2025-01-28 | 1 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Since "obeying the constraints of a FileAccess" is a term of art, this commit introduces links from the places where it is used to the place where it is defined. | |||||
| | * | | | rpc-connect: Remove remaining TODO RPCs | Nick Mathewson | 2025-01-27 | 1 | -4/+1 | |
| | | | | | | | | | | | | | | | | | I have checked the relevant code, and it seems okay. | |||||
| | * | | | rpc-connect: Make Connect::validate more straightforward | Nick Mathewson | 2025-01-27 | 1 | -11/+12 | |
| | | | | | ||||||
| | * | | | rpc-connect: check that all paths inside a connpt are absolute. | Nick Mathewson | 2025-01-27 | 1 | -13/+49 | |
| | | | | | ||||||
| | * | | | rpc-connect: remove todo; confirm that we do want to mkdir. | Nick Mathewson | 2025-01-27 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | rpc-connect: Detect and decline connpts with invalid "auth" feields | Nick Mathewson | 2025-01-27 | 3 | -20/+18 | |
| | | | | | ||||||
| | * | | | rpc-connect: Abort on misformed cookie file, and explain why. | Nick Mathewson | 2025-01-27 | 1 | -6/+3 | |
| | | | | | ||||||
| | * | | | rpc-connect: Use file_access() API, and permit symlinks. | Nick Mathewson | 2025-01-27 | 2 | -17/+20 | |
| | |/ / | ||||||
| * | | | Merge branch 'more-tor-proto' into 'main' | David Goulet | 2025-01-28 | 4 | -92/+146 | |
| |\ \ \ | | | | | | | | | | | | | | | | | tor-proto: Various refactorings and cleanups See merge request tpo/core/arti!2733 | |||||
| | * | | | tor-proto: Use SENDME_TAG_LEN constant instead of magic value. | Gabriela Moldovan | 2025-01-28 | 2 | -7/+8 | |
| | | | | | ||||||
| | * | | | tor-proto: Use CircTag::try_from for clarity. | Gabriela Moldovan | 2025-01-28 | 2 | -7/+12 | |
| | | | | | ||||||
| | * | | | tor-proto: Use CircTag instead of raw byte slice. | Gabriela Moldovan | 2025-01-28 | 2 | -7/+6 | |
| | | | | | ||||||
| | * | | | tor-proto: Move stream msg delivery to a separate function (fmt). | Gabriela Moldovan | 2025-01-28 | 1 | -33/+32 | |
| | | | | | ||||||
| | * | | | tor-proto: Move stream msg delivery to a separate function. | Gabriela Moldovan | 2025-01-28 | 1 | -33/+47 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is mostly code motion (I recommend reviewing with `--color-moved`). The code is intentionally misindented to make diffing easier. A future commit will fix the indentation. | |||||
| | * | | | tor-proto: Move stream ID extraction out of handle_relay_msg(). | Gabriela Moldovan | 2025-01-28 | 1 | -9/+18 | |
| | | | | | ||||||
| | * | | | tor-proto: Wrap an extra long lone. | Gabriela Moldovan | 2025-01-28 | 1 | -1/+5 | |
| | | | | | ||||||
| | * | | | tor-proto: Use continue instead of () for clarity. | Gabriela Moldovan | 2025-01-28 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | tor-proto: Adjust handle_cell() docs. | Gabriela Moldovan | 2025-01-28 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | tor-proto: Split handle_relay_cell(). | Gabriela Moldovan | 2025-01-28 | 1 | -4/+15 | |
| | | | | | ||||||
| | * | | | tor-proto: Split out send_relay_cell(). | Gabriela Moldovan | 2025-01-28 | 2 | -13/+28 | |
| | | | | | ||||||
| | * | | | tor-proto: Group imports. | Gabriela Moldovan | 2025-01-28 | 1 | -2/+1 | |
| | | | | | ||||||
| | * | | | tor-proto: Replace fully-qualified paths with use. | Gabriela Moldovan | 2025-01-28 | 1 | -6/+6 | |
| | | | | | | | | | | | | | | | | | For readability. | |||||
| | * | | | tor-proto: Derive Default for MutableState. | Gabriela Moldovan | 2025-01-28 | 2 | -4/+2 | |
| |/ / / | | | | | | | | | | | | | This enables us to get rid of two lines from the circuit reactor constructor. | |||||
| * | | | Merge branch 'strongrefs_are_not_deduped' into 'main' | Nick Mathewson | 2025-01-28 | 3 | -8/+10 | |
| |\ \ \ | | | | | | | | | | | | | | | | | rpc: Clean up comments surrounding strong references See merge request tpo/core/arti!2741 | |||||
| | * | | | rpc: Clean up comments surrounding strong references | Nick Mathewson | 2025-01-27 | 3 | -8/+10 | |
| | |/ / | | | | | | | | | | They used to be deduplicated, but they haven't been for a while. | |||||
| * | | | Merge branch 'todos-in-rpcbase-1' into 'main' | Nick Mathewson | 2025-01-27 | 5 | -104/+110 | |
| |\ \ \ | |/ / |/| | | | | | | | | Fix some TODOs in tor-rpcbase See merge request tpo/core/arti!2740 | |||||
| | * | | rpcbase: Enforce validity on error datum keywords. | Nick Mathewson | 2025-01-27 | 2 | -4/+12 | |
| | | | | ||||||
| | * | | rpcbase: Make recognized namespace list optional. | Nick Mathewson | 2025-01-27 | 2 | -8/+11 | |
| | | | | ||||||
| | * | | rpcbase: Move is_valid_rpc_identifier to crate root. | Nick Mathewson | 2025-01-27 | 2 | -91/+89 | |
| | | | | ||||||
| | * | | rpc: Rename InvalidMethodName to InvalidRpcIdentifier. | Nick Mathewson | 2025-01-27 | 3 | -17/+19 | |
| | | | | | | | | | | | | | We're about to use this error type for other things too. | |||||
| | * | | Remove RPC TODO about bypass_method_dispatch method. | Nick Mathewson | 2025-01-27 | 1 | -6/+1 | |
| |/ / | | | | | | | | | | | There was formally a redundant method of this name, which could get out-of-sync with invoke_without_dispatch. But now that method is gone, and this TODO is wrong. | |||||
| * | | Merge branch 'ahf/win-build-fix' into 'main' | Nick Mathewson | 2025-01-26 | 1 | -2/+2 | |
| |\ \ | |/ |/| | | | | | Fix not(unix) build error introduced in cb090c7b4c. See merge request tpo/core/arti!2738 | |||||
| | * | Fix not(unix) build error introduced in cb090c7b4c. | Alexander Hansen Færøy | 2025-01-26 | 1 | -2/+2 | |
| |/ | | | | | | | This patch fixes a relatively harmless build error introduced for non-unix builds. Additional comma introduced in b5aa8f7a88. | |||||
| * | Merge branch 'rpc-docs-tool-fixes' into 'main' | Nick Mathewson | 2025-01-23 | 1 | -28/+87 | |
| |\ | | | | | | | | | | | | | Fixes and maintenance on rpc-docs-tool Closes #1708 See merge request tpo/core/arti!2736 | |||||
| | * | rpc-docs-tool: Fix documentation for methods that apply to all objects | Nick Mathewson | 2025-01-23 | 1 | -7/+30 | |
| | | | | | | | | | | | | | | | This is not yet strictly automated yet, but since there is only one such method this is probably good enough for now. Closes #1708. | |||||
| | * | rpc-doc-tool: Fix dangling RustDoc links | Nick Mathewson | 2025-01-23 | 1 | -11/+47 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We already successfully handled links of the forms: * `[RustId]` * `[text](https://...)` * `[text][label]` But we didn't handle links of the form: * `[text](RustId)` These latter links aren't correct markdown on their own; but they're simple enough to fix up after a little hacking on the marko backend representation. | |||||
| | * | rpc-docs-tool: Emit methods and objects in sorted order. | Nick Mathewson | 2025-01-23 | 1 | -4/+7 | |
| | | | ||||||
| | * | rpc-docs-tool: Update version recommendation | Nick Mathewson | 2025-01-23 | 1 | -2/+2 | |
| | | | | | | | | | (Current nightly works fine, so we can stop recommending a downgrade.) | |||||
| | * | rpc-docs-tool: Use new connect-point API to connect to arti. | Nick Mathewson | 2025-01-23 | 1 | -4/+1 | |
| | | | | | | | | | Fortunately, the new API has simplified matters greatly! | |||||
| * | | Merge branch 'rpc-missing-dir' into 'main' | Nick Mathewson | 2025-01-23 | 2 | -10/+20 | |
| |\ \ | | | | | | | | | | | | | rpc: Tolerate NotFound on configured connect point directory. See merge request tpo/core/arti!2735 | |||||
| | * | | connpts: Document behavior on nonexistent directory | Nick Mathewson | 2025-01-23 | 2 | -2/+6 | |
| | | | | ||||||
| | * | | rpc: Tolerate NotFound on configured connect point directory. | Nick Mathewson | 2025-01-23 | 1 | -8/+14 | |
| | |/ | | | | | | | | | | | | | | | If Arti tries to bind to a directory of connect points, and that directory isn't there, the right behavior is to treat the directory as if it were empty. Otherwise, the user would have to create the default connect point directory (as empty) before Arti would start. | |||||
| * | | Merge branch 'avoid_execute_internal_ok' into 'main' | Nick Mathewson | 2025-01-23 | 4 | -31/+69 | |
| |\ \ | | | | | | | | | | | | | rpclib: Avoid overuse of execute_internal_ok. See merge request tpo/core/arti!2730 | |||||
| | * | | rpclib: More explanations about internal method usage. | Nick Mathewson | 2025-01-23 | 1 | -3/+5 | |
| | | | | ||||||
| | * | | arti-rpc-client-core: Rename Authentication{Rejected=>Failed} | Nick Mathewson | 2025-01-23 | 3 | -9/+9 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Also change corresponding message. This way, it will be clear that we're just reporting that Arti gave us an error—not that the error necessarily means that the authentication itself was "wrong". | |||||
| | * | | rpclib: Avoid overuse of execute_internal_ok. | Nick Mathewson | 2025-01-23 | 4 | -24/+60 | |
| | |/ | | | | | | | | | | | | | | | | | | | | | The execute_internal_ok method converts every error response into an internal error; as such, it's only appropriate when there is no way for a well-behaved Arti instance to give an error response. But we had been using it in a few places where errors were possible under other circumstances. This commit fixes that behavior, and adds documentation to help avoid it. | |||||
| * | | Merge branch 'rpc-todos-1' into 'main' | Nick Mathewson | 2025-01-23 | 2 | -20/+3 | |
| |\ \ | | | | | | | | | | | | | rpc: Resolve a couple of dead code TODOs See merge request tpo/core/arti!2731 | |||||
| | * | | rpcserver: remove dead code exception in objmap.rs | Nick Mathewson | 2025-01-22 | 1 | -3/+1 | |
| | | | | ||||||
| | * | | rpcserver: remove dead code in codecs.rs | Nick Mathewson | 2025-01-22 | 1 | -17/+2 | |
| | | | | ||||||
| * | | | Merge branch 'rpc-todos-2' into 'main' | Nick Mathewson | 2025-01-23 | 6 | -44/+47 | |
| |\ \ \ | |_|/ |/| | | | | | | | | Fix several 'TODO RPC' notes in the arti crate. See merge request tpo/core/arti!2737 | |||||
