| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Run maint/add_warning to actually apply new lint allows | Ian Jackson | 2023-07-10 | 3 | -0/+3 |
| | | |||||
| * | tor-proto: Remove use of arrayref. | Nick Mathewson | 2023-06-01 | 1 | -2/+5 |
| | | |||||
| * | Replace usage of KeyUnknownCert::check_key. | Nick Mathewson | 2023-05-16 | 1 | -2/+2 |
| | | |||||
| * | async-utils: rename SinkExt to SinkPrepareExt | Nick Mathewson | 2023-04-19 | 1 | -1/+1 |
| | | |||||
| * | Move functionality from tor_basic_utils to tor_async_utils | Nick Mathewson | 2023-03-29 | 1 | -1/+1 |
| | | | | | | This commit is mostly code movement; I'd recommend reviewing it with git's `--color-moved` option. | ||||
| * | Fix a bunch of needless-conversion warnings. | Nick Mathewson | 2023-03-10 | 1 | -1/+0 |
| | | | | | | Apparently 1.68 now warns when you call into_iter() on something that's already an iterator. Fair enough. Let's stop doing that. | ||||
| * | Rename OpenClientChan{Msg,Cell} => OpenChan{Msg,Cell}S2C | Nick Mathewson | 2023-02-09 | 1 | -20/+20 |
| | | |||||
| * | tor-proto: Do not parse forbidden commands on inbound cells. | Nick Mathewson | 2023-02-09 | 1 | -84/+28 |
| | | | | | | | | | | | Unlike C tor, we treat unrecognized commands as reason to kill off the connection entirely. That's fine; if we need to add an unrecognized command in the future, we can use VERSIONS to negotiate it. Also, if someday we want this code to support relay channels as well, we can use some type trickery to have that work too. | ||||
| * | tor-proto: only parse allowed ChanMsg types during handshake. | Nick Mathewson | 2023-02-09 | 2 | -28/+97 |
| | | |||||
| * | tor-cell: Rename ChanMsg and ChanCell-related types. | Nick Mathewson | 2023-02-07 | 3 | -42/+42 |
| | | |||||
| * | tor-cell: Remove ChanMsg methods that are duplicated in ChanMsgClass. | Nick Mathewson | 2023-02-07 | 3 | -2/+3 |
| | | |||||
| * | tor-cell: Use macro to generate ChanMsg too. | Nick Mathewson | 2023-02-07 | 2 | -3/+3 |
| | | |||||
| * | Allow clippy::unchecked_duration_subtraction in tests | Nick Mathewson | 2023-01-27 | 3 | -0/+3 |
| | | | | | | This panics on error, and we're fine with a panic on misbehavior in tests. | ||||
| * | tor-proto: rustfmt | Ian Jackson | 2023-01-06 | 1 | -1/+2 |
| | | | | | Not sure why the tree didn't have this newline already. "Whatever". | ||||
| * | tor-proto: When relay IDs mismatch, the IDs are sensitive in errors | Ian Jackson | 2023-01-06 | 1 | -2/+4 |
| | | |||||
| * | Merge branch 'test-lints' into 'main' | eta | 2023-01-06 | 2 | -0/+17 |
| |\ | | | | | | | | | Add test lint blocks to all "mod test" See merge request tpo/core/arti!937 | ||||
| | * | test lint blocks: Add many many automatically | Ian Jackson | 2022-12-12 | 2 | -0/+17 |
| | | | | | | | | | | | This is precisely the result of running the rune in maint/adhoc-add-lint-blocks. | ||||
| * | | Use parse_rfc3339() in the tor-proto crate | Neel Chauhan | 2022-12-16 | 1 | -1/+2 |
| | | | |||||
| * | | Merge branch 'err-dyn-report' into 'main' | Ian Jackson | 2022-12-15 | 1 | -1/+1 |
| |\ \ | | | | | | | | | | | | | Error logging (ErrorReport, .report()) POC See merge request tpo/core/arti!936 | ||||
| | * | | tor-proto: reactor tests: Use debug fmt on an error | Ian Jackson | 2022-12-12 | 1 | -1/+1 |
| | |/ | | | | | | | | | | | | | In tests, debug fmt is usually appropriate. Found while looking for a candidate for https://gitlab.torproject.org/tpo/core/arti/-/issues/680 | ||||
| * / | Tackling issue #663 (Use humantime in tests) | coral | 2022-12-12 | 1 | -1/+1 |
| |/ | |||||
| * | Merge branch 'rename_for_to_from' into 'main' | Nick Mathewson | 2022-10-25 | 1 | -2/+2 |
| |\ | | | | | | | | | tor-cell: Rename for_client and for_relay See merge request tpo/core/arti!793 | ||||
| | * | tor-cell: Rename for_client and for_relay | Emil Engler | 2022-10-21 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | This commit renames the for_client and for_relay functions to from_client and from_relay respectively, in order to indicate their origin, as the term "for" is more likely to indicate a destination, which is not true in that situation. | ||||
| * | | linkspec: Remove now-useless declared_peer_addr | Nick Mathewson | 2022-10-24 | 1 | -3/+4 |
| |/ | | | | The singleton variation here is almost never what we want. | ||||
| * | Allow "clippy::single_char_pattern" in tests. | Ian Jackson | 2022-10-12 | 1 | -0/+1 |
| | | | | | | | | This lint exists for perf reasons, and this is rarely relevant in tests. Using double quoted str is generally cognitively less burdensome. | ||||
| * | Change multiplicity of ChannelMethod and addresses | Nick Mathewson | 2022-10-11 | 1 | -2/+2 |
| | | | | | | | | Now each `ChanTarget` has at most one `ChannelMethod`, and only `Direct` `ChannelMethods` can have multiple addresses. Closes #600. | ||||
| * | Clarify limits on socket address from ChannelMethod | Nick Mathewson | 2022-10-06 | 1 | -2/+2 |
| | | |||||
| * | tor-linkspec: Remove the old OwnedFoo::new() functions | Nick Mathewson | 2022-10-06 | 2 | -5/+15 |
| | | | | | These are now builders. | ||||
| * | tor-proto: Preserve the ChannelMethod, not the SocketAddr | Nick Mathewson | 2022-10-06 | 1 | -21/+42 |
| | | |||||
| * | add feature annotation not added by doc_auto_cfg | trinity-1686a | 2022-08-24 | 1 | -0/+1 |
| | | |||||
| * | channel padding: Only do anything with link protocol 5 | Ian Jackson | 2022-08-17 | 1 | -0/+7 |
| | | | | | | | | Or rather, if we *didn't* negotiate 4, which is too old. As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2828354 | ||||
| * | channel padding: Properly advertise link protocol 5 | Ian Jackson | 2022-08-17 | 1 | -3/+2 |
| | | | | | | | | | | We already actually send and negotiate the padding, since !657, but we ought to negotiate a protocol version where that's not a violation! As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2828354 | ||||
| * | channel padding: Move initial config of ChannelPaddingInstructions | Ian Jackson | 2022-08-17 | 1 | -6/+5 |
| | | | | | | | | | | | | | | Making ChannelPaddingInstructions::default() accurately reflect the initial state of the reactor's padding timer simplifies the code somewhat. (When padding is wanted, parameters are computed and inserted explicitly, so the only change is that if we start out dormant, we defer setting the timer parameters until necessary.) As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2827249 | ||||
| * | channel padding: Parameters: default_padding replaces Default impl | Ian Jackson | 2022-08-17 | 2 | -1/+7 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2827249 | ||||
| * | channel reactor: Add some commented-out debug for padding | Ian Jackson | 2022-08-17 | 1 | -1/+5 |
| | | |||||
| * | channel padding: Correctly always send the timer parameters | Ian Jackson | 2022-08-17 | 1 | -1/+8 |
| | | | | | Fixes "chanmgr configuration: Avoid sending needless initial update(s)" | ||||
| * | channel padding: Abolish ARTI_EXPERIMENTAL_CHANNEL_PADDING env var | Ian Jackson | 2022-08-17 | 1 | -8/+0 |
| | | | | | This was for testing and is no longer needed. | ||||
| * | ChannelPaddingInstructions: Update and expand docs | Ian Jackson | 2022-08-17 | 2 | -11/+41 |
| | | |||||
| * | ChannelPaddingInstructions: Rename `params` field to `insns` | Ian Jackson | 2022-08-17 | 1 | -6/+6 |
| | | |||||
| * | Rename ChannelsParams types to ChannelPaddingInstructions (fmt) | Ian Jackson | 2022-08-17 | 1 | -1/+4 |
| | | | | | Run rustfmt; no other changes. | ||||
| * | Rename ChannelsParams types to ChannelPaddingInstructions | Ian Jackson | 2022-08-17 | 2 | -38/+38 |
| | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826167 This makes some lines too long; I will run rustfmt in a separate commit for clarity. | ||||
| * | Rename PaddingParameters::all_zeroes constructor to disabled | Ian Jackson | 2022-08-17 | 1 | -4/+2 |
| | | |||||
| * | tor-cell: PaddingNegotiate::default_reduced | Ian Jackson | 2022-08-17 | 1 | -10/+0 |
| | | | | | | | | | | | | Get rid of unneeded constructor. We never need to use hardcoded reduced padding parameters during negotiation cell construction. If we are using reduced padding parameters, the layers which decide this have netparams to use. Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826092 | ||||
| * | tor-cell: PaddingNegotiate::start: take IntegerMilliseconds | Ian Jackson | 2022-08-17 | 1 | -2/+1 |
| | | |||||
| * | channel: Clarify (and in some places replace) "frontend" terminology | Ian Jackson | 2022-08-17 | 2 | -3/+5 |
| | | |||||
| * | tor-proto channel padding: Document PADDING_NEGOTIATE overwriting | Ian Jackson | 2022-08-17 | 1 | -0/+3 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2825979 | ||||
| * | channel padding: Rename low_ms and high_ms | Ian Jackson | 2022-08-17 | 1 | -13/+13 |
| | | | | | | These have the unit in the type. Putting that in the field name too is otiose. | ||||
| * | tor-proto, testing: Provide accessors for ChannelsParamsUpdates | Ian Jackson | 2022-08-16 | 1 | -0/+11 |
| | | |||||
| * | tor-proto: Make "testing" feature that exports some things | Ian Jackson | 2022-08-16 | 1 | -1/+3 |
| | | | | | | We are going to want this for through-the-layers padding control testing. | ||||
| * | channel padding: Send negotiation cells | Ian Jackson | 2022-08-16 | 2 | -0/+9 |
| | | |||||
