summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Document "full", "experimental" in toplevel crate documentation.Nick Mathewson2022-06-132-82/+121
| | | | | | | | | | | | | | | | | | | | | | | | | Also, unify the features documentation format for those two crates, and document previously undocumented features there.
| * | | | Add "full" and "experimental" features to arti.Nick Mathewson2022-06-101-0/+7
| | | | |
| * | | | Add "full" and "experimental" features to arti-client and below.Nick Mathewson2022-06-1010-4/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "full" feature is a catch-all for all features, _except_: * Those that select a particular implementation (like tor-llcrypto/with-openssl) or build flag (like "static") * Those that are experimental or unstable (like "experimental-api") * Those that are testing-only.
* | | | | Merge branch 'clippy_20220614' into 'main'Nick Mathewson2022-06-157-12/+21
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Fix several clippy issues, most with nightly See merge request tpo/core/arti!588
| * | | | Add one more "allow" to tor_netdoc::parse::macros.Nick Mathewson2022-06-151-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Without this, clippy is error-free, but `cargo test` somehow gives a warning.
| * | | | Permit eprintln in channel-padding test.Nick Mathewson2022-06-151-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | There is not, apparently, an eprintln equivalent for allow-dbg-in-tests.
| * | | | Fix clippy::significant_drop_in_scrutinee warningsNick Mathewson2022-06-154-12/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is apparently a new warning from clippy nightly, documented in https://rust-lang.github.io/rust-clippy/master/index.html#significant_drop_in_scrutinee . I'm not in love with the temporary variables that this warning wants me to introduce, but it does seem like a decent way to avoid some kinds of deadlock.
| * | | | Work around an erroneous warning from nightly clippyNick Mathewson2022-06-151-0/+3
| | |/ / | |/| |
* | | | Merge branch 'unwrap' into 'main'Nick Mathewson2022-06-138-45/+15
|\ \ \ \ | |/ / / |/| | | | | | | | | | | tor-netdir: testnet: Make construct_netdir infallible See merge request tpo/core/arti!585
| * | | tor-netdir: testnet: Make construct_netdir infallible (rustfmt)Ian Jackson2022-06-136-30/+10
| | | | | | | | | | | | | | | | | | | | Run rustfmt. Separate commit to make review of the substantive commit easier.
| * | | tor-netdir: testnet: Make construct_netdir infallibleIan Jackson2022-06-138-15/+5
| | | | | | | | | | | | | | | | | | | | This is a *lot* of unwraps. The function takes no parameters and is used only for testing. It ought to be infallible.
* | | | Merge branch 'dormant_directory' into 'main'Nick Mathewson2022-06-138-90/+219
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use TaskSchedule to sleep in directory bootstrapping Closes #497 See merge request tpo/core/arti!571
| * | | | DirMgr: Use scopeguard crate in place of ad hoc RAII substitutesNick Mathewson2022-06-133-68/+20
| | | | |
| * | | | Put the task schedule back when the dl task exits.Nick Mathewson2022-06-131-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | This will allow somebody else to call bootstrap() if bootstrap() fails the first time.
| * | | | New API to expose and use TaskHandle from DirMgr.Nick Mathewson2022-06-133-2/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change also means that we need to create the handle and scheduler earlier in the process of creating the DirMgr. If we don't, we won't have a way to manage the task before bootstrap() returns.
| * | | | DirMgr: Refactor bootstrap to use a TaskSchedule for sleeping.Nick Mathewson2022-06-102-32/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change (not yet exposed as an API) will let the TorClient have a `TaskHandle` corresponding to the directory task, letting it make the directory task dormant as needed.
| * | | | TaskSchedule: Add a sleep_until_wallclock method too.Nick Mathewson2022-06-102-2/+15
| | | | |
| * | | | Add a `sleep` function to TaskSchedule.Nick Mathewson2022-06-071-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Having this alias makes it easier to implement more complex schedules, like those used in DirMgr.
| * | | | Add "suspend" and "resume" to TaskSchedule.Nick Mathewson2022-06-071-0/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unlike "cancel" and "fire", "suspend" and "resume" don't change any pending timers or events: they just prevent execution of those events for a while, and let them resume later on.
* | | | | Merge branch 'option' into 'main'eta2022-06-133-1/+7
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | config: Be consistent about Option Closes #488 See merge request tpo/core/arti!582
| * | | | | config: document semver changeIan Jackson2022-06-101-0/+1
| | | | | |
| * | | | | config: Suppose that we might extend resolve_option to non-T::DefaultIan Jackson2022-06-101-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per point 3 in https://gitlab.torproject.org/tpo/core/arti/-/issues/488
| * | | | | config: Do not strip_option for journald (and in future)Ian Jackson2022-06-102-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per point 1 in https://gitlab.torproject.org/tpo/core/arti/-/issues/488
* | | | | | Merge branch 'doc' into 'main'eta2022-06-131-1/+2
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | tor-error: Fix a broken intra-doc link See merge request tpo/core/arti!580
| * | | | | tor-error: Fix a broken intra-doc linkIan Jackson2022-06-101-1/+2
| |/ / / / | | | | | | | | | | | | | | | I didn't spot this in review amongst the other messages from Nightly.
* | | | | Merge branch 'nested-read-write' into 'main'Nick Mathewson2022-06-102-40/+61
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | tor-bytes: read_nested_*: Take a closure Closes #498 See merge request tpo/core/arti!579
| * | | | tor-bytes: read_nested_*: rustfmtIan Jackson2022-06-101-13/+23
| | | | | | | | | | | | | | | | | | | | I disagree with almost all of these layout decisions...
| * | | | tor-bytes: read_nested_*: Take a closureIan Jackson2022-06-102-40/+51
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This eliminates the possibility of writing the bug of failing to call `should_be_exhausted`. As per this discussion https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/572#note_2811464 Fixes #498
* | | | Merge branch 'channel' into 'main'Nick Mathewson2022-06-106-10/+718
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Have channel reactor able to send channel padding See merge request tpo/core/arti!574
| * | | | channel padding timer: Explain why next() in select_biased!Ian Jackson2022-06-091-0/+7
| | | | |
| * | | | channel padding timer: document state invariantsIan Jackson2022-06-091-2/+24
| | | | | | | | | | | | | | | | | | | | There aren't very many.
| * | | | Fix more typos in comments, from a spellcheckerIan Jackson2022-06-091-6/+6
| | | | |
| * | | | Fix typo in commentNick Mathewson2022-06-091-1/+1
| | | | |
| * | | | Fix erroneous commentNick Mathewson2022-06-091-1/+1
| | | | |
| * | | | Fix typos in commentsNick Mathewson2022-06-091-3/+3
| | | | |
| * | | | tor-proto: padding: Test padding timer distributionIan Jackson2022-06-083-0/+235
| | | | |
| * | | | tor-proto: channel: Use padding::TimerIan Jackson2022-06-083-12/+44
| | | | |
| * | | | tor-proto: channel: Provide padding::TimerIan Jackson2022-06-085-0/+412
| | | | |
* | | | | Merge branch 'format-ids' into 'main'Ian Jackson2022-06-103-28/+52
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Replace four very similar "ids in request" See merge request tpo/core/arti!577
| * | | | dirclient: Fix comments re conssensus request without authoritiesIan Jackson2022-06-101-5/+3
| | | | |
| * | | | dirclient: Replace four very similar "ids in request"Ian Jackson2022-06-093-28/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In reviewing !553 I noticed that the empty digest list error had to be handled in two places. I filed #492 about the duplication. In fact it turns out to have been quadruplication. The new code also avoids cloning the underlying objects, instead sorting a Vec of references.
| * | | | dirclient: Test consensus request based on empty ConsensusRequestIan Jackson2022-06-091-0/+8
| |/ / / | | | | | | | | | | | | | | | | | | | | I'm not sure this is right. But providing a test case for it before my next change ensures that my refactoring doesn't change the behaviour.
* | | | Merge branch 'typos' into 'main'Ian Jackson2022-06-102-4/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix typos See merge request tpo/core/arti!578
| * | | | Fix typosDimitris Apostolou2022-06-102-4/+4
|/ / / /
* | | | Merge branch 'nested-read-write' into 'main'Ian Jackson2022-06-097-24/+223
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Introduce and use nested length reader/writers See merge request tpo/core/arti!572
| * | | | Fix typos in docs and commentsNick Mathewson2022-06-091-2/+2
| | | | |
| * | | | tor-cell: udp: Remove manual length calculationsIan Jackson2022-06-092-23/+11
| | | | | | | | | | | | | | | | | | | | | | | | | This does away entirely with `wire_addr_len`, which recapitulates the length calculation.
| * | | | tor-bytes: Provide nested writersIan Jackson2022-06-093-1/+118
| | | | |
| * | | | tor-bytes: Provide nested readersIan Jackson2022-06-092-0/+70
| | | | |
| * | | | tor-bytes: Reader: Provide take_restIan Jackson2022-06-092-0/+20
| | | | | | | | | | | | | | | | | | | | We'll want this in a moment.