| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We should privilege the "is the same" case. Rename check to check2
and check1 to check.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This allows us to easily distinguish by eye, the cases where things
changed (ie, where the input string was canonical), from the ones
where they didn't.
One of these cases is a bug.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
A "mask" is a bitmask, eg 255.255.0.0 in 192.168.0.0/255.255.0.0.
These are all prefix lengths.
Using the word "mask" to refer to a prefix length is quite bad,
because prefix lengths are numbers and sometimes masks are too.
Using one as the other would be a bug.
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Derive NetdocParseableUnverified for RouterDesc
See merge request tpo/core/arti!4134
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This adds a very small RouterDesc parse2 test only verifying whether the
testdata2 gets properly parsed, with a minimal length verification
alongside two identity related fields.
Once we have encoding and verification support, we have to extend this
test of course.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Only for testing, just returns the inner body. Will be removed once we
have a proper verify method.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Finally! 🎉
This commit derives NetdocParseableUnverified for RouterDesc by adding
the respective derive_deftly attributes to the top-level definition as
well as the member fields that require it.
A .verify() method as well as unit tests will be added in the next
commits.
Encoding still depends on a few tiny papercuts but will otherwise be
added shortly too, although in a different branch.
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fix RUSTSEC-2026-0186
See merge request tpo/core/arti!4140
|
| |/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This commit upgrades memmap2 to 0.9.11 in order to fix an unchecked
pointer offset in the memmap2 crate.
We are not directly affected by this, as `tor-dirmgr`, the only crate
making use of this dependency, does not call the affected functions
listed on rustsec.org.
|
| |\ \ \ \ \ \
| | |_|_|/ /
| |/| | | |
| | | | | |
| | | | | | |
tor-netdoc: derive (and fix) encoding for microdescriptors
See merge request tpo/core/arti!4117
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
We don't need the type to be private. Indeed, we'd like it not to be
to make Microdesc construction with Constructor regular
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
"OnionKeyIntro" is particularly odd. Mostly people won't need to
refer to this, anyway.
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
This lets us skip fields which have the default values.
|
| | | | | | | |
|
| |/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
All these types already have parsing derives.
The encoding isn't quite right yet. It's legal, but doesn't match C
Tor as well as we'd like.
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Create dirauth crates (empty)
See merge request tpo/core/arti!4122
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
Let's start this now.
|
| | | | | | | |
|
| | |/ / / /
| | | | |
| | | | |
| | | | | |
We already seem to use "network-programming", which kind of fits.
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
tor-netdoc: implement verification of votes, and add round trip test
See merge request tpo/core/arti!4121
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Use roundtrip_netstatus, which does nearly all of the work for us.
But for it to round-trip, we must do a bunch of fudging.
This replaces the old parse2 vote test case (which used the poc type).
The test case has some adjustments to cope with missing fields. Move
the TODOs for those fields to the test case, so that they're on lines
that will be deleted when the fields are implemented.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Right now we're working on votes, but the same issue arises in
consensuses; we just haven't noticed yet because the testdata2
consensus doesn't have an SRV.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In the future (not quite yet in this branch), there will be tests that
want to use roundtrip_netstatus for other test files.
We'll want to be able to reuse the per-document-specific munging. So
make that into a trait. And document roundtrip_netstatus.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This will let us do more exciting munging more clearly.
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The vote test files are for the next consensus interval, so we need to
verify them at a slightly future time.
|
| | | | | | |
|
| |/ / / / |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Resolve "arti: subcommands: keys: keys list --compact displays extra empty lines"
Closes #2593
See merge request tpo/core/arti!4133
|
| | | | | | |
|
| |/ / / / |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add a rough plan for relay incoming stream handling
Closes #2574
See merge request tpo/core/arti!4107
|