| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | |
|
| | | |
|
| |/ |
|
| |\
| |
| |
| |
| |
| |
| | |
safelog: Add new DisplayRedacted/DebugRedacted APIs, and use it for HsId.
Closes #2012 and #2066
See merge request tpo/core/arti!3107
|
| | |
| |
| |
| | |
Closes #2012.
|
| |/
|
|
|
|
|
| |
These APIs differ from Redactable by forcing the caller to choose
whether an object should be displayed in redacted or non-redacted
form. (Redactable objects still implement Display, which
creates the risk of accidentally displaying an un-redacted object.)
|
| |\
| |
| |
| |
| | |
arti: hss: Add `arti hss ctor-migrate`
See merge request tpo/core/arti!3102
|
| |/ |
|
| |\
| |
| |
| |
| | |
arti-client: Re-export `ConfigurationSources`
See merge request tpo/core/arti!3044
|
| | |
| |
| |
| |
| |
| |
| | |
This commit re-exports `ConfigurationSources` in `arti_client::config`.
The similar `ConfigurationSource` is already publicly exported, but it
unfortunately it is pretty useless without a `ConfigurationSources` to
which it belongs.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
check_licenses: allow outputs from cargo-license 0.7.0, and pin to 0.7.0
Closes #2083
See merge request tpo/core/arti!3111
|
| | | |
| | |
| | |
| | | |
This should prevent recurrences of issues like #2083.
|
| | | |
| | |
| | |
| | |
| | | |
cargo-license version 0.7 now normalizes the "-only" variant into
"LGPL-3.0".
|
| |/ /
| |
| |
| |
| | |
cargo-license 0.7 has started normalizing SPDX expressions, which
includes sorting the licenses in an AND or OR.
|
| |\ \
| | |
| | |
| | |
| | | |
ci: Downgrade cargo-license to 0.6.1 as a workaround
See merge request tpo/core/arti!3108
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | | |
proto: Add a ConfluxSet::remove_unchecked method.
See merge request tpo/core/arti!3105
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This command is supposed to return an error when handled by a tunnel
reactor that has more than one circuit.
Prevents the `ConfluxSet::take_single_leg()` bug fixed in 00bb0628.
|
| | | |
| | |
| | |
| | | |
This was fixed in !3091
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This slightly reduces the amount of code we need to maintain.
This also drive-by fixes a bug in `ConfluxSet::take_single_leg`, which
previously never actually checked if the conflux set was of size one.
|
| |/ /
| |
| |
| |
| | |
This method will enable us to remove some duplicated code, as well as
the `element_idx` function.
|
| |\ \
| | |
| | |
| | |
| | | |
Implement prop 362
See merge request tpo/core/arti!3093
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
This ensures that even in situations where multiple requests are
received at the exact same time, we will not drop any requests.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
This basically converts everything to use the num_traits conversion
function, and explicitly panics on errors.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This pulls in num_traits (which is already a dependency for PoW) to
allow f64 casts that for reasons I do not understand are not implemented
via TryFrom in the standard library.
|
| | | |
| | |
| | |
| | |
| | | |
This should never happen, as the queue size is limited to well below
u32. However, it's still nicer not to use `as`.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We need to divide a Duration by this, and Duration only supports
division by u32.
This is, in a sense, just pushing around where the overflow would
happen, (from the conversion in the suggested update to the increment
when a item is dequeued). However, this overflow is so unlikely to
happen (it would require more that 14 million requests per second) that
it does not seem worth slowing down the increment operation to try to
handle it.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This has fewer weird edge cases than RwLock does.
It might also be reasonable to make this a AtomicU32 (or AtomicEffort
which wraps AtomicU32), but that's slightly more complex, so I've opted
for a mutex for now.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This modifies the code to be more testable (making it generic over
RendRequest, getting the time from the runtime rather than
Instant::now(), etc) and adds some tests for the PoW control loop.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This will allow the PowManager to have a copy of RendRequestReceiver,
which is important to allow the PowManager update loop to control the
suggseted_effort updating.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This makes the suggested_effort value a Arc<RwLock<Effort>>, which is
shared between the PowManager (as a reader) and the RendRequestReceiver
(as a writer), since the RendRequestReceiver has the information needed
to update the suggested_effort value.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The updated PoW control loop in prop362 requires this data structure to
be double ended, which BinaryHeap is not. I benchmarked [email protected]
against BTreeSet on a synthetic benchmark based on what I expected
typical load to look like, and found that BTreeSet performed much
better. It is additionally in the standard library, and is maintained,
which no third-party double ended priority queue crate seems to be.
Given that, BTreeSet seems like a reasonable thing to build on, although
we should provide metrics on the performance of this queue so operators
can let us know if it seems to be a problem in real-world use.
|
| |/ /
| |
| |
| |
| |
| |
| | |
Since this will be used in a BTreeSet, we need to ensure that in
practice no two requests will be equal. Ordering by a combination of
effort and time received should be sufficient for this, and gives us the
behaviour we want.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Negotiate and use CGO.
Closes #1947 and #1945
See merge request tpo/core/arti!3069
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This commit prevents us from ever deciding that another relay
supports CGO, which prevents us from trying to negotiate it.
It also adds an assertion to make sure that we haven't tried to
negotiate it.
We can remove this once we're ready to have CGO turned on:
right now it's blocked on being able to negotiate CC.
I'm adding this so we can merge this branch (so that I can stop
rebasing it.)
|
| | | | |
|
| | | | |
|