| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | | tor-hsservice: ipt_mgr: Adjust API for mock RNG | Ian Jackson | 2023-10-16 | 1 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | We're want to be able to return a mutable borrow of a field in the mock state. This means we must make `Rng` a GAT. | |||||
| | * | | | tor-hsservice: Narrow some dead code allows | Ian Jackson | 2023-10-16 | 6 | -2/+7 | |
| | | | | | ||||||
| | * | | | tor-hsservice: timeout_track: Change semantics to do what's needed | Ian Jackson | 2023-10-16 | 1 | -32/+68 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The key point is this: +//! I.e., the timeout tracker tells you when (in the future) +//! any of the comparisons you have made, might produce different answers. +//! So, that can be used to know how long to sleep for when waiting for timeout(s). That's how the code in ipt_mgr.rs uses this. Without this change, things go wrong in the following case: we've got at least one good IPT, but not quite enough, and the others are taking too long. Ie, the timeout for "we should publish" is in the past. We decide to publish (correctly) but this *past* timeout should be disregarded. Disregarding past timeouts is correct if the code which is making the comparisons acts on the timeout when it occurs. | |||||
| | * | | | tor-hsservice: timeout_track: Provided shortest() for all trackers | Ian Jackson | 2023-10-16 | 1 | -0/+25 | |
| | | | | | ||||||
| | * | | | tor-hsservice: timeout_track: Mark some unused variables | Ian Jackson | 2023-10-16 | 1 | -2/+2 | |
| | | | | | ||||||
| | * | | | tor-hsservice: timeout_track: Add a dead code allow | Ian Jackson | 2023-10-16 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | For the same reason as the allow(unreachable_pub). | |||||
| | * | | | tor-hsservice: ipt_mgr: Improve waiting publish message | Ian Jackson | 2023-10-16 | 1 | -10/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | This involves plumbing the duration through. Also it involved breaking out the multiplication by two into an addition, since if we were to change the factor we'd want to print the scaled value. | |||||
| | * | | | tor-hsservice: ipt_mgr: Add several debug/trace messages | Ian Jackson | 2023-10-16 | 1 | -4/+22 | |
| | | | | | | | | | | | | | | | | | | | | | We're going to improve the "establishing_very_recently" one in a moment. | |||||
| | * | | | tor-hsservice: ipt_mgr: Fix a message style | Ian Jackson | 2023-10-16 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | | | | | | Add a colon. This is going to be the style for our trace and debug output. | |||||
| | * | | | tor-hsservice: ipt_mgr: Avoid looping with a zero timeout | Ian Jackson | 2023-10-16 | 1 | -0/+3 | |
| | | | | | | | | | | | | | | | | | | | | | If the time we're supposed to wait is zero, something has gone wrong. We'd probably just go round again and loop indefinitely. | |||||
| | * | | | tor-hsservice: ipt_mgr: Apply a limit to looping | Ian Jackson | 2023-10-16 | 1 | -0/+10 | |
| | | | | | | | | | | | | | | | | | This is a backstop to stop us just spinning. Instead, we crash. | |||||
| | * | | | tor-hsservice: ipt_mgr: Actually wait for (fast-ish) IPTs | Ian Jackson | 2023-10-16 | 1 | -0/+4 | |
| | | | | | | | | | | | | | | | | | | | | | We never called the `started_establishing_very_recently` machinery. Do so. | |||||
| | * | | | tor-hsservice: ipt_mgr: Remove vestigial expiry time code (2) | Ian Jackson | 2023-10-16 | 1 | -10/+1 | |
| | | | | | | | | | | | | | | | | | | | | | lifetime is passed in the IptSet and doesn't want to be made absolute here. | |||||
| | * | | | tor-hsservice: ipt_mgr: Remove vestigial expiry time code | Ian Jackson | 2023-10-16 | 1 | -4/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This calculation is now done elsewhere: the addition of IPT_PUBLISH_EXPIRY_SLOP happens in IptSet::note_publication_attempt and the storage in the manager's data structure is done by IptManager::import_new_expiry_times. | |||||
| | * | | | tor-hsservice: ipt_mgr: Mark an argument as unused | Ian Jackson | 2023-10-16 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | tor-hsservice: Better debug impl for IptLocalId | Ian Jackson | 2023-10-16 | 1 | -1/+5 | |
| | | | | | ||||||
| | * | | | tor-rtmock: MockRuntime::test_with_various: Print what we're doing | Ian Jackson | 2023-10-16 | 1 | -1/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, it can be a bit confusing since you just get the output (if any) of all the test runs, concatenated. It seems sensible to put the framing centrally. And, this would be helpful if the test fails for any reason. | |||||
| | * | | | tor-rtmock: MockExecutor: Provide n_tasks accessor | Ian Jackson | 2023-10-16 | 2 | -0/+16 | |
| |/ / / | ||||||
| * | | | Merge branch 'by_type_remove' into 'main' | Nick Mathewson | 2023-10-16 | 1 | -10/+0 | |
| |\ \ \ | | | | | | | | | | | | | | | | | tor-cell: Remove unused by_type() function See merge request tpo/core/arti!1664 | |||||
| | * | | | tor-cell: Remove unused by_type() function | Neel Chauhan | 2023-10-13 | 1 | -10/+0 | |
| | | |/ | |/| | ||||||
| * | | | Merge branch 'fix-after-script-usage' into 'main' | Nick Mathewson | 2023-10-14 | 1 | -7/+2 | |
| |\ \ \ | |/ / |/| | | | | | | | | | | | | | | move usage of ./maint/preserve out of after_script Closes #1061 See merge request tpo/core/arti!1663 | |||||
| | * | | remove artifact generation from rust-latest-test-all-features | trinity-1686a | 2023-10-13 | 1 | -5/+0 | |
| | | | | ||||||
| | * | | move usage of ./maint/preserve out of after_script | trinity-1686a | 2023-10-13 | 1 | -3/+3 | |
| |/ / | ||||||
| * | | Merge branch 'todos_proto' into 'main' | Nick Mathewson | 2023-10-12 | 12 | -129/+205 | |
| |\ \ | | | | | | | | | | | | | tor-proto: Resolve a few TODO HSS entries. See merge request tpo/core/arti!1658 | |||||
| | * | | Add downgrade-dependencies exceptions to make tor-proto build right. | Nick Mathewson | 2023-10-12 | 1 | -1/+3 | |
| | | | | ||||||
| | * | | proto::stream::incoming: Use bit-array representation for command list | Nick Mathewson | 2023-10-12 | 3 | -7/+125 | |
| | | | | | | | | | | | | | | | | This should be smaller and faster than vec, and save some allocations. It could also avoid a tiny sidechannel. | |||||
| | * | | tor-proto: remove a now-unneeded allow in stream::incoming | Nick Mathewson | 2023-10-12 | 1 | -2/+0 | |
| | | | | ||||||
| | * | | hss: use Arc to avoid copying on k_hss_ntor. | Nick Mathewson | 2023-10-12 | 3 | -15/+8 | |
| | | | | | | | | | | | | | (We try to avoid making tons and tons of copies of a secret key.) | |||||
| | * | | Remove the (fairly bogus) HsNtorServiceInput type. | Nick Mathewson | 2023-10-12 | 5 | -103/+66 | |
| | | | | | | | | | | | | | This will let avoid some copying inside our HSS code. | |||||
| | * | | proto: Remove a TODO about merging two functions. | Nick Mathewson | 2023-10-12 | 1 | -2/+0 | |
| | | | | | | | | | | | | | | | | There is some similarity, but there's not really a logical way to combine the two that actually results in less, clearer code. | |||||
| | * | | Lower and downgrade a TODO about StreamID and NonZeroU16. | Nick Mathewson | 2023-10-12 | 2 | -2/+6 | |
| | | | | ||||||
| * | | | Merge branch 'proto' into 'main' | Alexander Færøy | 2023-10-12 | 1 | -0/+2 | |
| |\ \ \ | |/ / |/| | | | | | | | | tor-proto: Add a FIXME about an allow See merge request tpo/core/arti!1657 | |||||
| | * | | tor-proto: Add a FIXME about an allow | Ian Jackson | 2023-10-12 | 1 | -0/+2 | |
| |/ / | ||||||
| * | | Merge branch 'bug1051' into 'main' | gabi-250 | 2023-10-11 | 3 | -4/+188 | |
| |\ \ | | | | | | | | | | | | | Implement build_auth_clients() See merge request tpo/core/arti!1642 | |||||
| | * | | Implement build_auth_clients() | Neel Chauhan | 2023-10-11 | 3 | -4/+188 | |
| | | | | ||||||
| * | | | Merge branch 'oneshot' into 'main' | Nick Mathewson | 2023-10-11 | 38 | -44/+117 | |
| |\ \ \ | |/ / |/| | | | | | | | | | | | | | | Provide and mandate Fuse type veneer around oneshot Closes #1059 See merge request tpo/core/arti!1656 | |||||
| | * | | oneshot: Apply deferred rustfmt churn | Ian Jackson | 2023-10-11 | 17 | -28/+21 | |
| | | | | | | | | | | | | | cargo fmt, precisely. | |||||
| | * | | oneshot: Deprecate futures::channel::oneshot | Ian Jackson | 2023-10-11 | 1 | -0/+5 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in arti-rpcserver | Ian Jackson | 2023-10-11 | 1 | -1/+2 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-hsrproxy | Ian Jackson | 2023-10-11 | 3 | -1/+4 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-dirmgr | Ian Jackson | 2023-10-11 | 4 | -2/+5 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-hsclient | Ian Jackson | 2023-10-11 | 2 | -2/+2 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-hsservice | Ian Jackson | 2023-10-11 | 5 | -6/+7 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-circmgr | Ian Jackson | 2023-10-11 | 5 | -5/+8 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-ptmgr | Ian Jackson | 2023-10-11 | 4 | -2/+7 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-guardmgr | Ian Jackson | 2023-10-11 | 4 | -2/+5 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-chanmgr | Ian Jackson | 2023-10-11 | 3 | -1/+3 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-proto | Ian Jackson | 2023-10-11 | 6 | -9/+15 | |
| | | | | ||||||
| | * | | oneshot: Use veneer in tor-rtmock | Ian Jackson | 2023-10-11 | 4 | -2/+4 | |
| | | | | ||||||
| | * | | oneshot: Avoid (one) use of .try_recv() | Ian Jackson | 2023-10-11 | 1 | -2/+4 | |
| | | | | | | | | | | | | | | | | | | | It's not so straightforward to provide this (we'd need to poll the FusedFuture and make a dummy a Context and so on), but this is just used once in a test where we can replace it with use of `poll!` | |||||
