| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Fix Windows shortcut target
See merge request tpo/core/arti!849
|
| |/ / / |
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Temporarily disable it_cancels_delayed_firings.
See merge request tpo/core/arti!846
|
| | |/
| |
| |
| |
| | |
This test is timing-dependent in a way that seems to fail on heavily
loaded CI machines. See #545.
|
| |\ \
| | |
| | |
| | |
| | | |
Resolve most "TODO pt-client" comments in GuardMgr
See merge request tpo/core/arti!840
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This is the only way I could find in which parameter interpretation
differs between bridge guards and relay guards; with it documented,
I can remove a TODO about identifying such ways.
|
| | | |
| | |
| | |
| | |
| | | |
Have the check only apply to non-bridge universes, and explain why
in a bunch more comments.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously we would call extend_sample_as_needed in only two places,
one of which called the other unconditionally. That's obviously not
necessary.
I've selected just one of them (`update_guardset_internal`) since it
fits better with the theme if that function. I've added comments
explaining what is going on.
This commit also introduces a yes/no enum for "were any guards added
while extending this set". Formerly we had a boolean, but it got
passed around so many times that I think its intent became obscure.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
What this function actually does is return the number of primary
guards whose presence (by identity) is ambiguous in a current
universe. The new name and documentation should help avoid
confusion.
The method's old name had led me astray when identifying whether it
should apply to bridges in one case. This commit also removes the
corresponding `TODO pt-client`.
|
| | | | |
|
| | | |
| | |
| | |
| | | |
The code seems happy enough here for now.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Also, add a bunch of reminders around these implementations that
`HasAddrs` returns all the address associated with you for GeoIp or
family purposes, even if they are _not_ ones that we should actually
contact you at.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
The BridgeSet type does not necessarily need further changes... and
if it gets them, it won't be because of this comment.
|
| |\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
Increase MSRV to 1.60
Closes #591, #526, #613, and #621
See merge request tpo/core/arti!837
|
| | | |
| | |
| | |
| | | |
rsa 0.7.0 has been yanked.
|
| | | |
| | |
| | |
| | | |
It was introduced in Rust 1.57.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
Closes #621
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | | |
Use ErrorKind::TorAccessFailed for misbehaving bridges
See merge request tpo/core/arti!842
|
| | | |
| | |
| | |
| | | |
(Use semantic linefeed for the doc comment.)
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
bridge descs: Add tests for process_document
See merge request tpo/core/arti!841
|
| | |/ / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Remove TODO re BridgeDescDownloadConfig defaults
See merge request tpo/core/arti!839
|
| | |/ / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Check more targets
See merge request tpo/core/arti!844
|
| |/ / / |
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
bridge desc: Deal with two TODOs
See merge request tpo/core/arti!843
|
| | | |
| | |
| | |
| | |
| | | |
We have a separate BridgeDescMgr, rather than reusing DirMgr, so we
will not be implementing BridgeDescProvider for DirMgr.
|
| |/ / |
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Upgrade to clap 3
Closes #616
See merge request tpo/core/arti!830
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Also replace Arg::takes_value with Arg::action to configure flags
that don't take parameters vs options that do.
This makes it easy to upgrade to clap 4.
Tested deprecations with:
cargo check --workspace --all-features --features clap/deprecated
|
| | |
| |
| |
| |
| |
| |
| |
| | |
This removes the last cargo audit override (for the unmaintained
ansi_term).
Don't mark options as required when they have default values:
see <https://github.com/clap-rs/clap/pull/3793>.
|
| |\ \
| | |
| | |
| | |
| | | |
Downgrade and/or remove a few misc "TODO pt-client" comments
See merge request tpo/core/arti!838
|
| | | |
| | |
| | |
| | | |
These are structure-internal things we can solve later, if ever.
|
| | | |
| | |
| | |
| | | |
renamings are now #623. Code movement can happen later.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Persistently cache bridge descriptors
Closes #619
See merge request tpo/core/arti!831
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is more consistent with our naming elsewhere.
Suggested-by: Nick Mathewson <[email protected]>
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/831#note_2851764
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
The current test case doesn't use this.
|