| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Replace poc code with new ConsensusAuthoritySection.
This is an entirely new parsing approach.
Key differences:
* We parse the arguments to dir-source, which poc didn't.
* We don't clone the input and fully parse the first item; instead,
we dismantle the dir-source line ad-hoc to determine the item type.
* Rather than a Vec of Enums, we parse into two Vecs
of the different entry types.
The few lines that aren't rewritten show up as moved in
git show --color-moved --color-moved-ws=allow-indentation-change
but I suggest reviewing the new implementation de novo.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is the type corresponding to a "superseded authority key entry",
which is a single dir-source line with a weird nickname.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is going to handle the bizarre `-legacy` nickname encoding for
superseded authority keys. That's going to be a fair amount of code,
some derived from DirSource. Make a module for it all.
We're going to take a different approach that in poc. So there's
going to be all-new code and then the poc version will be deleted.
They don't resemble each other very much.
In this commit: just code motion, and the creation of the new module
with associated doc comment. Review with --color-moved.
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We must add to our ad-hoc derive, since the normal derive can't do the
encoding either.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Use deftly's field loop, rather than handcoding the two fields.
(We don't expect to add any fields, but if we did this would be more correct.)
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Almost entirely code motion. Review with --color-moved.
This is new correct for votes, and uses the non-poc names for things.
We'll improve up a few minor things in a moment.
The cfg situation is getting very tangled. There are already various
TODOs about this. I think it is time to bite the bullet; I will do
that separately from this MR.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Prepare for the namespace environment in the mainline doc/netstatus
modules. (Add a couple of temporary use's.)
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We are going to move this code to the main doc/netstatus modules.
Rename the struct and its associated derive macro, to the naming
approach used outside poc.
We leave a `use` behind in poc. That can remain until poc is
eventually abolished.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Purely whitespace change: deindent certain lines by 4.
`git show -w` is helpful.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This has a side effect of testing the parsing.
(Testing of the encoding will have to come a lot later.)
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Deprecate the compatibility alias.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Use the terminology from the spec.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This makes the struct more legible.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
We'll want this with Constructor's __non_exhaustive.
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
Prep work for EmbeddedCert
See merge request tpo/core/arti!3918
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
As discussed in
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3918#note_3401409
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
tor-netdoc: Introduce EmbeddedCert, and a demo of how to use it
Closes #2485
See merge request tpo/core/arti!3922
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fixes #2485.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
This reverts commit 68caf324a320fff1a4f0e9b0f5014a34d0e3729f.
|
| | |/ / / /
| | | | |
| | | | |
| | | | |
| | | | | |
This is more sensible and will make the code in tor-netdoc less strange.
We'll revert the TryFrom in a moment.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Implement the Writeable trait.
Explain why this approach is correct and leave a comment near the
decoder (to avoid future changes making this implementation buggy) and
a test case.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Otherwise we can't implement trait-based decoding in tor-netdoc.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Somehow FixedB64 was not reachable previously, but this wasn't
detected by the compiler despite it being `pub`.
|