| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | |
| | | |
| | | |
| | | |
| | | | |
I also searched this file for FromStr impls and these types were the
only ones that ought to be NormalItemArgument but weren't.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Unfortunately one derive can't invoke another, so we must open-code
them, just like the other traits.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Nothing derives just Transparent yet.
Mostly code motion, with some additional furniture and docs.
Review with git show --color-moved.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This error message is going to be used by both Transparent and
BytesTransparent. Put it in the code that is going to be part of
Transparent, and remove the macro name from it.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
This reduces clutter some more.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is in the use's at the top of the file. If we `use super::*`
(like we do in the newer modules): the code can get a lot more
readable.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Test the parsing and printing success cases.
Given that the parsing implementation is just a wrapper around B16's,
we don't need explicit tests for that.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Make this a public BytesTransparent newtype like B64, implementing all
the same traits etc.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
On reflection, and looking at the test cases, I think the name `B16`
should refer to the lowercase version. I'll introduce a type for
uppercase in a moment.
Test the display implementation.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Replace the open-coded .parse() calls with a closure.
This will let us test round trips, other types, etc., for each pairs
of test values.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Firstly, make its field pub.
Secondly, make the into_array method crate-private. It has a
crate-specific (albeit public) error type, The same basic effect can
be achieved with methods from std on arrays and slices.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Make this type public.
The new macro call means this type now implements many more traits.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
I think these are all the traits that a parsing/printing newtype
adapter ought to implement.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Actually make the macro be a proper d-d macro. Replace uses of .0
with $fname, B64 with $ftype, etc.
No functional change in this commit.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Break various open-coded impls on B64 into this macro. That will let
us reuse this code for other types, making all these bytes wrappers
similar to each other.
In this commit, we just move the code. The macro is wrong because it
ought to refer to $ttype and so on. Review with git show --color-moved.
No functional change in this commit.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This is slightly more logical and will make the next commit slightly
easier to read.
|
| |/ / /
| | |
| | |
| | | |
This will reduce some duplication and simplify some macrology.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Tweak TunnelId and TunnelScopedCircId Display impls
See merge request tpo/core/arti!3875
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Circuit IDs (UniqId) are displayed as "Circ <x>.<y>".
Prior to this MR, these TunnelScopedCircId's were displayed as
"Circ <t>.<x>.<y>" where t is the integer tunnel ID. This made corresponding
logs a bit confusing as to why some "Circ" identifiers had two parts and
some have three, and didn't make clear that the "<x>.<y>" part of the latter
were comparable with the two-part UniqIds.
The previous commit effectively changes the latter to
"Circ Tunnel <t>.<x>.<y>", which is still a bit confusing.
This commit changes the display of TunnelScopedCircId's to "Circ <x>.<y>
(Tunnel <t>)", which makes the distinction between the circuit and
tunnel IDs clearer.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is akin to how circuit `UniqId`'s are prefixed with "Circ", and
helps clarify logs where it isn't always clear from context whether a
tunnel ID or circuit ID is being displayed.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
tor-netdoc: ContactInfo
See merge request tpo/core/arti!3866
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Otherwise you might, for example, receive a String from a config file,
JSON API submission, RPC call, or whatever, containing a newline, and
then encode it into a netdoc giving a syntax error or, worse,
smuggling additional items into the document!
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Otherwise the cfg_attr for the field would need to name both features.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This will be needed for ntor handshakes.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Usually, there will only be two of these.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This also updates the key rotation task to call the setter whenever the
ntor keys get updated.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This will need to be updated each time the ntor keys change.
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
I am about to add another one of these, so I tried to deduplicate the
impls a bit.
|
| | | | | | |
|