| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | | | minimal features test: Introduce new --cfg arti_features_precise | Ian Jackson | 2024-10-22 | 3 | -1/+23 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Work around awkward cargo behaviour and allow us to more reliably test disabled features, even if they're enabled by default at lower levels. | |||||
| | * | | | | test-all-crates: New --start-from option | Ian Jackson | 2024-10-22 | 1 | -0/+11 | |
| | | | | | | | | | | | | | | | | | | | | | This is very useful when iterating through CI. | |||||
| | * | | | | test-all-crates: conditional options, to test more crates | Ian Jackson | 2024-10-22 | 5 | -6/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new feature lets us provide the "enable these options which are needed to make the tests pass" featrure, which is different for each of the afflicted crates. Then we can test these crates tor-hsservice arti arti-client which minimal features. This will be important in a moment, as we're going to want to be relying on actually minimal features tests in arti cfg.rs. | |||||
| | * | | | | test-all-crates: Introduce proper argument parser | Ian Jackson | 2024-10-22 | 2 | -4/+8 | |
| | |/ / / | ||||||
| * | | | | Merge branch 'rpc-cookie-auth-sketch' into 'main' | Nick Mathewson | 2024-10-22 | 2 | -0/+136 | |
| |\ \ \ \ | |/ / / |/| | | | | | | | | | | | RPC cookie authentication spec sketch. See merge request tpo/core/arti!2440 | |||||
| | * | | | rpc-cookie: Formatting cleanup wrt `P`. | Nick Mathewson | 2024-10-21 | 1 | -3/+3 | |
| | | | | | ||||||
| | * | | | rpc-cookie: Make more text normative; defined a malformed cookie file | Nick Mathewson | 2024-10-21 | 1 | -6/+11 | |
| | | | | | ||||||
| | * | | | rpc-cookie: More naming consistency | Nick Mathewson | 2024-10-21 | 1 | -8/+9 | |
| | | | | | ||||||
| | * | | | rpc-sketches: Unify, clarify, and clean up socket_canonical | Nick Mathewson | 2024-10-21 | 2 | -31/+32 | |
| | | | | | ||||||
| | * | | | rpc-cookie-sketch: unify non-address field names. | Nick Mathewson | 2024-10-21 | 1 | -9/+8 | |
| | | | | | ||||||
| | * | | | rpc-cookie-sketch: re-order SADDR_USE and SADDR_VERIFY. | Nick Mathewson | 2024-10-21 | 1 | -7/+8 | |
| | | | | | ||||||
| | * | | | rpc-cookie-sketch: specify cookie IO failures. | Nick Mathewson | 2024-10-21 | 1 | -0/+13 | |
| | | | | | ||||||
| | * | | | cookie-sketch: Revise "in arti-rpc" section for clarity | Nick Mathewson | 2024-10-21 | 1 | -6/+12 | |
| | | | | | | | | | | | | | | | | | In particular, specify which methods are invoked on which objects. | |||||
| | * | | | rpc-cookie-sketch: Clarifications from @diziet | Nick Mathewson | 2024-10-21 | 1 | -6/+7 | |
| | | | | | ||||||
| | * | | | cookie-sketch: More security concerns on addr_canonical | Nick Mathewson | 2024-10-21 | 1 | -0/+13 | |
| | | | | | ||||||
| | * | | | cookie-sketch: Use a separate set of methods for cookie-auth. | Nick Mathewson | 2024-10-21 | 1 | -10/+16 | |
| | | | | | ||||||
| | * | | | cookie-sketch: Describe use of TupleHash | Nick Mathewson | 2024-10-21 | 1 | -15/+17 | |
| | | | | | | | | | | | | | | | | | | | | | Also, rename values to S_MAC and C_MAC, and include SADDR_CANONICAL in messages. | |||||
| | * | | | cookie-sketch: Use agreed-upon mitm prevention. | Nick Mathewson | 2024-10-21 | 1 | -5/+12 | |
| | | | | | ||||||
| | * | | | cookie-sketch: "nonce", not "secret". | Nick Mathewson | 2024-10-21 | 1 | -3/+3 | |
| | | | | | ||||||
| | * | | | RPC cookie authentication spec sketch. | Nick Mathewson | 2024-10-21 | 1 | -0/+81 | |
| | |/ / | | | | | | | | | | Part of #1521. | |||||
| * | | | Merge branch 'mq-fix' into 'main' | Ian Jackson | 2024-10-22 | 26 | -27/+124 | |
| |\ \ \ | | | | | | | | | | | | | | | | | memquota: Fix account lifetime bugs, and arrange to test mq in shadow See merge request tpo/core/arti!2560 | |||||
| | * | | | memquota: Use _ rather than allow(dead_code) (fmt) | Ian Jackson | 2024-10-22 | 1 | -1/+4 | |
| | | | | | ||||||
| | * | | | memquota: Use _ rather than allow(dead_code) | Ian Jackson | 2024-10-22 | 2 | -12/+16 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Promote the associated comments. As suggested here: https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2560#note_3097188 | |||||
| | * | | | tor-memquota: Fix rustfmt-damaged indentation | Ian Jackson | 2024-10-22 | 1 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2560#note_3097189 | |||||
| | * | | | shadow tests: Enable and turn on memquota | Ian Jackson | 2024-10-21 | 2 | -1/+6 | |
| | | | | | | | | | | | | | | | | | This detected the account lifetime bugs fixed in this branch. | |||||
| | * | | | shadow tests: Provide an arti.extra.toml to arti-extra binary | Ian Jackson | 2024-10-21 | 17 | -0/+26 | |
| | | | | | ||||||
| | * | | | memquota: Fix error message strings | Ian Jackson | 2024-10-21 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | These errors aren't necessarily memory pressure. They can occur due to bugs, and during teardown. | |||||
| | * | | | memquota: In debug builds, reliably detect use of dropped Account | Ian Jackson | 2024-10-21 | 1 | -0/+13 | |
| | | | | | | | | | | | | | | | | | | | | | This detects the bugs I have just fixed - in Shadow tests with the feature enabled and a (large) limit set. | |||||
| | * | | | memquota: Fix a test not to rely on accessing cache with dead Account | Ian Jackson | 2024-10-21 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | memquota: fix data stream account lifetime | Ian Jackson | 2024-10-21 | 1 | -5/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The DataStream is sometimes disassembled, eg by split. When that happens, the StreamAccount would be dropped - and that was the only strong reference. Put a StreamAccount in each of the pieces, instead of just in the combined DataStream struct. | |||||
| | * | | | memquota: Fix resolve stream account lifetime | Ian Jackson | 2024-10-21 | 2 | -8/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need the mq account for the stream not to collapse. The ResolveStream object needs to contain a strong reference to it. Have begin_stream_impl return the StreamAccount, rather than taking it as a parameter. That makes this bug a little more obvious. It also centralises the StreamAccount creation. | |||||
| | * | | | memquota: Fix circuit account lifetime (fmt) | Ian Jackson | 2024-10-21 | 2 | -5/+24 | |
| | | | | | ||||||
| | * | | | memquota: Fix circuit account lifetime (clippy churn) | Ian Jackson | 2024-10-21 | 2 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Now that it doesn't call CircuitAccount::new() it has no error paths, and clippy demands we remove the Result, so it must once again become infallible. | |||||
| | * | | | memquota: Fix circuit account lifetime | Ian Jackson | 2024-10-21 | 2 | -5/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We foolishly made *two* CircuitAccounts, one of which gets immediately dropped. But we need to hold onto the account somewhere, because an mq_queue doesn't keep the account alive. Otherwise everything breaks when mq tracking is enabled. | |||||
| | * | | | memquota: Log a trace message when a claim fails | Ian Jackson | 2024-10-21 | 2 | -1/+9 | |
| | | | | | ||||||
| | * | | | memquota: Log a message at info on startup, if enabled | Ian Jackson | 2024-10-21 | 1 | -0/+3 | |
| | | | | | ||||||
| * | | | | Merge branch 'inet_not_tcp' into 'main' | David Goulet | 2024-10-22 | 2 | -40/+40 | |
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | general::SocketAddr: Say "inet" rather than "tcp" Closes #1701 See merge request tpo/core/arti!2554 | |||||
| | * | | | | general::SocketAddr: Say "inet" rather than "tcp" | Nick Mathewson | 2024-10-17 | 2 | -40/+40 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "inet" makes more sense, since in principle these can also be used for udp, etc. Also making a corresponding change in rpc-connect-sketch.md, which uses this format. Closes #1701. | |||||
| * | | | | | Merge branch 'peek' into 'main' | Ian Jackson | 2024-10-22 | 1 | -4/+0 | |
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Remove an obsolete TODO See merge request tpo/core/arti!2562 | |||||
| | * | | | | | Remove an obsolete TODO | Ian Jackson | 2024-10-21 | 1 | -4/+0 | |
| | | |/ / / | |/| | | | | | | | | | | | | | This *is* in tor-async-utils :-). | |||||
| * | | | | | Merge branch 'rpc-api-cleaning' into 'main' | Nick Mathewson | 2024-10-22 | 12 | -77/+326 | |
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Tests and Improvements to python rpc client API See merge request tpo/core/arti!2567 | |||||
| | * | | | | | python: Enforce that a response has just one type. | Nick Mathewson | 2024-10-21 | 1 | -3/+13 | |
| | | | | | | | ||||||
| | * | | | | | Typo fixes from @jnewsome | Nick Mathewson | 2024-10-22 | 1 | -2/+2 | |
| | | | | | | | ||||||
| | * | | | | | python: add tests for opening streams via RPC. | Nick Mathewson | 2024-10-21 | 2 | -0/+35 | |
| | | | | | | | ||||||
| | * | | | | | python: Expose RPC status code from errors. | Nick Mathewson | 2024-10-21 | 4 | -3/+52 | |
| | | | | | | | ||||||
| | * | | | | | python: Rename connect to open_stream, for consistency. | Nick Mathewson | 2024-10-21 | 1 | -1/+1 | |
| | | | | | | | ||||||
| | * | | | | | python: add more tests for the rpc code. | Nick Mathewson | 2024-10-21 | 1 | -1/+38 | |
| | | | | | | | ||||||
| | * | | | | | python: avoid redundant error messages. | Nick Mathewson | 2024-10-21 | 1 | -1/+4 | |
| | | | | | | | ||||||
| | * | | | | | rpclib: Rename RequestCancelled to be accurate. | Nick Mathewson | 2024-10-21 | 4 | -18/+9 | |
| | | | | | | | ||||||
| | * | | | | | python: Move meta.features test to new module. | Nick Mathewson | 2024-10-21 | 3 | -47/+50 | |
| | | | | | | | ||||||
