| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | channel usage: Rename CU::UserTraffic from CU::Exit | Ian Jackson | 2022-08-17 | 6 | -25/+37 | |
| | | | | | | | | | | | As suggested in https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2827246 | |||||
| | * | chanmgr padding: Correct docs for padding_parameters* | Ian Jackson | 2022-08-17 | 1 | -2/+5 | |
| | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2827242 These texts were out of date; one of them still mentioned `all_zeroes`. | |||||
| | * | Improve comment for ChannelUsage::Exit | Nick Mathewson | 2022-08-17 | 1 | -1/+1 | |
| | | | ||||||
| | * | Fix typo in comment | Nick Mathewson | 2022-08-17 | 1 | -1/+1 | |
| | | | ||||||
| | * | tor-chanmgr tests: Fix a minimal-versions failure (actually) | Ian Jackson | 2022-08-17 | 1 | -1/+1 | |
| | | | ||||||
| | * | tor-chanmgr tests: Fix a minimal-versions failure | Ian Jackson | 2022-08-17 | 1 | -2/+2 | |
| | | | ||||||
| | * | arti: Fix a minimal-versions failure | Ian Jackson | 2022-08-17 | 1 | -1/+1 | |
| | | | ||||||
| | * | Add semver notes | Ian Jackson | 2022-08-17 | 6 | -0/+17 | |
| | | | ||||||
| | * | 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 | 2 | -6/+13 | |
| | | | | | | | | | 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. | |||||
| | * | channel engage_padding_activities: swap docs to tor0proto | Ian Jackson | 2022-08-17 | 2 | -11/+13 | |
| | | | | | | | | | This allow us to make a working cross-reference. | |||||
| | * | channel fake_channel_details: Use precise cfg | Ian Jackson | 2022-08-17 | 1 | -1/+1 | |
| | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826169 | |||||
| | * | tor-netdir: testnet: construct_custom_netdir_with_params: Fix doc | Ian Jackson | 2022-08-17 | 1 | -1/+2 | |
| | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826168 | |||||
| | * | 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 | 4 | -7/+22 | |
| | | | | | | | | | Run rustfmt; no other changes. | |||||
| | * | Rename ChannelsParams types to ChannelPaddingInstructions | Ian Jackson | 2022-08-17 | 8 | -63/+63 | |
| | | | | | | | | | | | | | | | | | 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. | |||||
| | * | chanmgr: Fix typo | Ian Jackson | 2022-08-17 | 1 | -1/+1 | |
| | | | | | | | | | | | Reported here https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826166 | |||||
| | * | chanmgr parameterize: Correct doc comment | Ian Jackson | 2022-08-17 | 1 | -1/+4 | |
| | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826164 | |||||
| | * | Rename PaddingParameters::all_zeroes constructor to disabled | Ian Jackson | 2022-08-17 | 1 | -4/+2 | |
| | | | ||||||
| | * | chamgr NetParamsExtract: Replace direct nf_ito access | Ian Jackson | 2022-08-17 | 1 | -4/+23 | |
| | | | | | | | | | | | It turns out that the TryFrom was infallible, which wasn't obvious with the previous code structure. | |||||
| | * | Channel: Make mutable() and engage_padding_activities infallible | Ian Jackson | 2022-08-17 | 5 | -18/+12 | |
| | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826151 This gets rid of quite some Bug error paths. | |||||
| | * | Move ChannelUsage from tor_proto to tor_chanmgr | Ian Jackson | 2022-08-17 | 9 | -76/+85 | |
| | | | | | | | | | | | | | | | | | | | | | | | Replace Channel::note_usage with Channel::engage_padding_activities, which unconditionally causes the channel to (start to) do netflow padding things. The condition now lives in chanmgr. Addresses https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826094 | |||||
| | * | tor-chanmgr: Expand docs for map::Inner dormancy field | Ian Jackson | 2022-08-17 | 1 | -0/+4 | |
| | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826093 | |||||
| | * | 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 | 4 | -6/+9 | |
| | | | ||||||
| | * | chanmgr padding_parameters: Use Option | Ian Jackson | 2022-08-17 | 2 | -40/+41 | |
| | | | | | | | | | | | | | | | | | | | The network consensus parameters use (0,0) to mean "no padding" (which is not the same as (0,0) means in a PADDING_NEGOTIATE cell). Representing "no padding" this way is actually quite convoluted and un-Rustic. Ensure that we convert (0,0) to None, and do the primary logic in Option. | |||||
| | * | tor-cell: PaddingNegotiate: give better spec xrefs | Ian Jackson | 2022-08-17 | 1 | -0/+6 | |
| | | | ||||||
| | * | channel: Clarify (and in some places replace) "frontend" terminology | Ian Jackson | 2022-08-17 | 3 | -7/+10 | |
| | | | ||||||
| | * | 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 | |||||
| | * | chanmgr: padding: Split IEFI into named function | Ian Jackson | 2022-08-17 | 1 | -19/+27 | |
| | | | | | | | | | | | | | | | Roughly as suggested in https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2826160 IMO this is less clear, because of the way it splits the logic up. | |||||
| | * | chanmgr: NetParamsExtract: extend docs for nf_ito | Ian Jackson | 2022-08-17 | 1 | -2/+2 | |
| | | | ||||||
| | * | chanmgr: Get rid of final Result relating to netdir/netparams | Ian Jackson | 2022-08-17 | 2 | -32/+29 | |
| | | | | | | | | | No functional change. (The error tyep was Void.) | |||||
| | * | chanmgr: Use NetDirProvider::params and Arc<dyn..Netparameters> | Ian Jackson | 2022-08-17 | 7 | -91/+100 | |
| | | | | | | | | | | | | | | | | | | | This gets rid of many Result(). Many parameters are renamed. Test cases of the now-impossible branch are removed. Deleting the match from padding_parameters will come in a moment. I've split off that commit since it has much whitespace noise. for now, change the error type to Void. | |||||
| | * | chanmgr: Add a TODO for renaming NetDirExtract | Ian Jackson | 2022-08-17 | 1 | -0/+3 | |
| | | | ||||||
| | * | chanmgr ChannelUsage: Fix and clarify docs | Ian Jackson | 2022-08-17 | 1 | -2/+3 | |
| | | | ||||||
| | * | chanmgr set_dormancy: Remove obsolete comment | Ian Jackson | 2022-08-17 | 1 | -4/+0 | |
| | | | ||||||
| | * | chanmgr: Extend doc comments for Dormancy | Ian Jackson | 2022-08-17 | 1 | -0/+6 | |
| | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/657#note_2825962 | |||||
| | * | Fix typos in comments. | eta | 2022-08-17 | 2 | -2/+2 | |
| | | | ||||||
| | * | channel padding: Rename low_ms and high_ms | Ian Jackson | 2022-08-17 | 4 | -23/+27 | |
| | | | | | | | | | | | These have the unit in the type. Putting that in the field name too is otiose. | |||||
| | * | channel padding: Test through most of the layers | Ian Jackson | 2022-08-17 | 3 | -0/+366 | |
| | | | ||||||
| | * | chanmgr testing: FakeChannel: retain the ChannelsParamsUpdates | Ian Jackson | 2022-08-16 | 1 | -1/+4 | |
| | | | ||||||
| | * | tor-proto, testing: Provide accessors for ChannelsParamsUpdates | Ian Jackson | 2022-08-16 | 1 | -0/+11 | |
| | | | ||||||
| | * | tor-proto, testing: Provide new_fake_channel | Ian Jackson | 2022-08-16 | 1 | -0/+22 | |
| | | | | | | | | | To test the padding control we will want this. | |||||
| | * | tor-proto, testing: Make fake_channel_details available | Ian Jackson | 2022-08-16 | 1 | -16/+18 | |
| | | | | | | | | | Now it's not just cfg(test), but feature testing. | |||||
| | * | tor-proto: Make "testing" feature that exports some things | Ian Jackson | 2022-08-16 | 4 | -5/+26 | |
| | | | | | | | | | | | We are going to want this for through-the-layers padding control testing. | |||||
| | * | tor-cell, testing: Provide PaddingNegotiate::from_raw | Ian Jackson | 2022-08-16 | 1 | -0/+8 | |
| | | | | | | | | | | | This allows test cases to describe precisely the contents of the negotiation cell ought to be generated. | |||||
| | * | tor-netdir, testnet: Provide construct_custom_netdir_with_params | Ian Jackson | 2022-08-16 | 1 | -2/+16 | |
| | | | | | | | | | | | We're going to want this for testing netdirs with padding parameters that diff ere from the spec's default. | |||||
| | * | tor-netdoc: NetParams: impl FromIterator | Ian Jackson | 2022-08-16 | 1 | -0/+8 | |
| | | | | | | | | | | | Otherwise a caller who has a set of parameters must open-code the loop. | |||||
