summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | tor-proto: Add a suppression to "fix" the CIIan Jackson2023-07-121-0/+2
| | | | | | | | | | | | | | | | | | | | | This is failing in CI. I have no idea what the rules are and AFAICT no-one is alleging that there is an actual bug in the attributes. Empirically this suppression causes the script to pass.
| * | tor-circmgr: Add two possibly-needed docsrs annotationsIan Jackson2023-07-122-0/+2
| | | | | | | | | | | | | | | I don't know if these are needed because the rules are not documented afaict. But it seems like probably they ought to be there?
| * | tor-proto: Conversation: drop two otiose cfg(feature)Ian Jackson2023-07-122-2/+0
| | | | | | | | | | | | These fns are in a feature-gated impls on feature-gated structs.
* | | Merge branch 'channel-experimental-api' into 'main'Nick Mathewson2023-07-121-1/+1
|\ \ \ | |/ / |/| | | | | | | | Gate builder() behind experimental-api feature See merge request tpo/core/arti!1403
| * | Gate builder() behind experimental-api featureSaksham Mittal2023-07-121-1/+1
|/ /
* | Merge branch 'channelfactory' into 'main'Nick Mathewson2023-07-121-1/+7
|\ \ | | | | | | | | | | | | Expose channel builder in order to create channels more efficiently in external code See merge request tpo/core/arti!1374
| * | Make CompoundFactory private for external codeSaksham Mittal2023-07-071-1/+0
| | |
| * | Mark builder() as experimentalSaksham Mittal2023-07-061-0/+1
| | |
| * | Import ChannelFactory instead of CompoundFactorySaksham Mittal2023-07-061-3/+3
| | |
| * | Rename get_channelbuilder() to builder()Saksham Mittal2023-07-061-1/+1
| | |
| * | Conditionally make CompoundFactory publicSaksham Mittal2023-07-061-1/+2
| | |
| * | Remove now unnecessary lines for importSaksham Mittal2023-07-051-2/+0
| | |
| * | Import CompoundFactory regardless of feature levelSaksham Mittal2023-07-051-3/+1
| | |
| * | Create new method to expose CompoundFactorySaksham Mittal2023-07-051-0/+9
| | |
| * | Expose CompoundFactory for later exposure in APISaksham Mittal2023-07-051-1/+1
| | |
* | | Merge branch 'ci-full-build' into 'main'gabi-2502023-07-121-1/+29
|\ \ \ | | | | | | | | | | | | | | | | CI: Add a job for building arti with additional features. See merge request tpo/core/arti!1401
| * | | CI: Remove unnecessary --feature flag.Gabriela Moldovan2023-07-121-1/+1
| | | | | | | | | | | | | | | | | | | | `onion-service-client` is now a default feature, so we don't need to explicitly enable it anymore.
| * | | CI: Add a job for building arti with additional features.Gabriela Moldovan2023-07-111-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is needed for testing `experimental-api` features (such as the keymgr) in shadow and chutney. Prompted by this discussion: https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1399#note_2921294
* | | | Merge branch 'remove-outdated-todo' into 'main'Nick Mathewson2023-07-121-3/+0
|\ \ \ \ | |/ / / |/| | | | | | | | | | | arti-client: Remove outdated TODO. See merge request tpo/core/arti!1400
| * | | arti-client: Remove outdated TODO.Gabriela Moldovan2023-07-111-3/+0
|/ / / | | | | | | | | | These errors aren't ignored anymore.
* | | Merge branch 'explain_952_fix' into 'main'Nick Mathewson2023-07-111-0/+3
|\ \ \ | | | | | | | | | | | | | | | | Explain the code for the #952 fix. See merge request tpo/core/arti!1391
| * | | Explain the code for the #952 fix.Nick Mathewson2023-07-101-0/+3
| | | | | | | | | | | | | | | | | | | | Let's explain what Trinity did in its fix for #952, so that we know why this code is here the next time we find it.
* | | | Merge branch 'clippy-allow' into 'main'Ian Jackson2023-07-11233-0/+265
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | clippy: Allow some of our existing code patterns See merge request tpo/core/arti!1396
| * | | | clippy: tor-error: Use convert::identity for into_internal!Ian Jackson2023-07-101-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This launders the closure so that clippy's clippy::redundant_closure_call can't see it. We can't have a local #[allow] because it would be on an expression, which isn't allowed on stable. This avoids having to use more clumsy idioms at call sites.
| * | | | tor-llcrypto: Add a missing instance of the standard lint blockIan Jackson2023-07-101-0/+10
| | | | | | | | | | | | | | | | | | | | This fixes a needless_vec lint on nightly.
| * | | | Run maint/add_warning to actually apply new lint allowsIan Jackson2023-07-10231-0/+249
| | | | |
| * | | | clippy: in tests, allow clippy::useless_vecIan Jackson2023-07-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a perf issue, only. If tests are too slow, we will notice and ca speed them up. We should optimise for clarity and convenience, rather than speed. Forbidding this can result in churn between vec![] and [] as tests are updated and changed.
| * | | | clippy: globally allow clippy::needless_raw_string_hashesIan Jackson2023-07-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rationale: no-one writes these by default without thinkinh. If they are unnecessary, then either the string must have had " in it before (in which case it might do again), or it is near other strings which *do* need it. And having it does no harm; indeed IMO it can increase clarity. Alternative to !1388's Fix new "needless_raw_string_hashes" lint from clippy +nightly
* | | | | Merge branch 'fix-chutney-ci' into 'main'Nick Mathewson2023-07-102-7/+15
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | fix chutney in CI See merge request tpo/core/arti!1393
| * | | | | make chutney setup script more portabletrinity-1686a2023-07-101-3/+1
| | | | | |
| * | | | | fix chutney in CItrinity-1686a2023-07-082-5/+15
| | | | | |
* | | | | | Merge branch 'range' into 'main'Ian Jackson2023-07-1012-16/+144
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Forbid use of panicky Rng::gen_range Closes #920 See merge request tpo/core/arti!1387
| * | | | | | rng docs: Fix two linksIan Jackson2023-07-101-2/+2
| | | | | | |
| * | | | | | rng ranges: Provide examples (doctests)Ian Jackson2023-07-101-0/+42
| | | | | | |
| * | | | | | rng ranges: Forbid use of panicky Rng::gen_rangeIan Jackson2023-07-102-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #920
| * | | | | | rng ranges: Use gen_range_infallible() for Duration::ZERO..=TIan Jackson2023-07-106-7/+10
| | | | | | |
| * | | | | | rng ranges: Introduce gen_range_infallibleIan Jackson2023-07-101-0/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are a number of places where we generate random Durations in a range which starts at zero. These call sites currently (i) have to write out Duration::ZERO or equivalent, and (ii) would have to use gen_range_checked and expect the result, even though it can be statically proven to be OK. To make this slightly smoother, provide `GenRangeInfallible` and `gen_range_infallible`.
| * | | | | | rng ranges: Use gen_range_checked().unwrap() in test caseIan Jackson2023-07-101-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ideally we would be allowed to use vanilla gen_range() here, but there doesn't seem to be a way to allow a specific clippy-forbidden method using #[allow] and we probably don't want to make a blanket allow.
| * | | | | | rng ranges: Use gen_range_checked().expect() in obvious cases (fmt)Ian Jackson2023-07-102-3/+5
| | | | | | |
| * | | | | | rng ranges: Use gen_range_checked().expect() in obvious casesIan Jackson2023-07-103-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In each of these, it is locally obvious that the range is nonempty.
| * | | | | | tor-basic-utils: retry: Use and justify gen_range_checkedIan Jackson2023-07-101-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | delay_bounds's implementation ensures the postcondition, so the potential p[anic in next_delay_msec cannot happen.
| * | | | | | rng ranges: Introduce RngExt and gen_range_checkedIan Jackson2023-07-101-0/+27
|/ / / / / / | | | | | | | | | | | | | | | | | | We will use this in many places instead of gen_range.
* | | | | | Merge branch 'keymgr-erased-key' into 'main'gabi-2502023-07-106-10/+23
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | keymgr: Use Box<dyn EncodableKey> instead of Box<dyn Any>. Closes #937 See merge request tpo/core/arti!1398
| * | | | | | keymgr: Add semver.md.Gabriela Moldovan2023-07-101-0/+2
| | | | | | |
| * | | | | | keymgr: Use Box<dyn EncodableKey> instead of Box<dyn Any>.Gabriela Moldovan2023-07-105-10/+21
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1337#note_2917701 This will make it harder to accidentally return the wrong value from `Keystore::get` (the returned value is now at least guaranteed to implement `EncodableKey`). Closes #937
* | | | | | Merge branch 's101-q2' into 'main'gabi-2502023-07-101-0/+26
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update s101 stats. See merge request tpo/core/arti!1397
| * | | | | | Update s101 stats.Alexander Færøy2023-07-101-0/+26
| | |_|/ / / | |/| | | |
* | | | | | Merge branch 'conversation' into 'main'Alexander Færøy2023-07-106-132/+233
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Overhaul send_control_message See merge request tpo/core/arti!1367
| * | | | | | tor-proto: run rustfmtIan Jackson2023-06-304-9/+21
| | | | | | |
| * | | | | | tor-proto conversations: semverIan Jackson2023-06-301-0/+1
| | | | | | |
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 15:13:39 +0000