| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | |
| | |
| | |
| | | |
This *is* the spec name.
|
| | | |
| | |
| | |
| | | |
Finishes arti#1151 item 10.
|
| | | |
| | |
| | |
| | | |
This also passes the information we'll need to populate extra_info.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
This will make the macro work if call sites rebind Result.
|
| | | |
| | |
| | |
| | |
| | | |
The KeyInfoExtractor impl is still misindented but we'll change that
when we replace it.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Now the only place that knows how to do all this is the one place in
the macro.
The indentation is -4 compared to previously, because we're going to
remove the wrapping with paste::paste! later. It's convenient to do
this reindent now, while we're rewriting it.
arti#1151 item 5.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This can only happen if we tried to parse an ArtiPath as the wrong
kind of KeySpecifier, which shouldn't happen very often.
If it does, the pattern isn't that interesting. Which is just as
well, since we're about to make it a bit harder to obtain it in the
place where this error is going to be constructed.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Where we wanted this, use derive_more::Constructor isntead.
arti#1151 item 6.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
So now .arti_pattern() is a trait method.
arti#1151 item 3.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
arti_prefix is no longer used. Abolish prefix too.
arti#1151 item 7.
Closes #1147.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is a better API. The implementation has less monomorphisation.
arti#1151 item 2 and the patterns part of 4.
Closes #1126.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This reduces monomorphisation.
arti#1151 item 4.
The commentary reflects a decision not to do arti#1151
item 9 (using fmt signature for to_component) at this time.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Part of arti#1151 item 10.
Discussion in
https://gitlab.torproject.org/tpo/core/arti/-/issues/1151#note_2977278
|
| | | |
| | |
| | |
| | |
| | | |
I'm not sure why the compiler doesn't complain about this already, but
it becomes "more" unused later.
|
| | | |
| | |
| | |
| | | |
This is going to expand and gain private helper methods.
|
| | | |
| | |
| | |
| | |
| | | |
We don't handle enums and we don't want to try.
Put this here as documentation of this intent.
|
| | |/
| |
| |
| | |
arti#1151 item 1.
|
| |\ \
| | |
| | |
| | |
| | | |
tor-cell: Fix ambitious certtype
See merge request tpo/core/arti!1846
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
When using the `arti-client` crate in other contexes, the Rust compiler
sometimes has difficulties with determining the current type in this
particular case, due to a collision with the popular serde json crate,
which also provides an implementation for converting u8.
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Update to derive-adhoc 0.8 and remove an allow
See merge request tpo/core/arti!1850
|
| | | |
| | |
| | |
| | | |
Now d-a #27 is fixed, we don't need this.
|
| |/ /
| |
| |
| |
| | |
As recommended in the d-a changelog, I'm not calling this a breaking
change to our APIs.
|
| |\ \
| |/
|/|
| |
| | |
Fix various lints
See merge request tpo/core/arti!1849
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
Filed
https://gitlab.torproject.org/tpo/core/arti/-/issues/1177
proposing a final fix.
|
| | |
| |
| |
| |
| |
| | |
Filed
https://gitlab.torproject.org/tpo/core/arti/-/issues/1176
proposing a final fix.
|
| | |
| |
| |
| | |
clippy correctly identifies that this is nicer than matches!.
|
| | |
| |
| |
| |
| | |
I'm not sure about this. Leaving it this way seems the most
conservative choice for now.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
clippy in current stable thinks
|(a, b)| (a, b)
is always the identity function, but due to match ergonomics, it might
be an implicit copy.
This is fixed in nightly by
https://github.com/rust-lang/rust-clippy/pull/11792
|
| | |
| |
| |
| | |
Resolves clippy complaints about needless fallible conversions.
|
| |/
|
|
|
| |
FTR I don't think agree with clippy on this question, but then I often
don't.
|
| |\
| |
| |
| |
| | |
tor-rtcompat: use track-caller for thin wrappers
See merge request tpo/core/arti!1843
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In particular, when the (unstable) tokio tracing feature is enabled,
every tracing line includes the name of where the current task was
created. Without this change, that ends up being the name of
intermediate trait methods like TokioRuntimeHandle::block_on, which is
not very helpful.
Adding the `track_caller` attribute causes the name of the caller of
these methods to be used instead, which is typically more helpful.
IIUC this change is not breaking in terms of semver
https://rustc-dev-guide.rust-lang.org/backend/implicit-caller-location.html.
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Bump zerocopy 0.7.29 -> 0.7.31
Closes #1174
See merge request tpo/core/arti!1844
|
| |/
|
|
|
|
|
|
| |
```
cargo update -p zerocopy
```
Fixes https://gitlab.torproject.org/tpo/core/arti/-/issues/1174
|
| |\
| |
| |
| |
| |
| |
| | |
tor-netdoc: Make HsDescBuilder::auth_clients take an Option.
Closes #1019
See merge request tpo/core/arti!1840
|
| | |
| |
| |
| | |
Closes #1019
|
| |\ \
| | |
| | |
| | |
| | | |
tor-cell: Stop using deprecated name in doctest
See merge request tpo/core/arti!1842
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
tor-keymgr: Use spec.torproject.org for the SSH algo name domain.
Closes #1108
See merge request tpo/core/arti!1838
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
The algorithm name for x25519 keys has changed, so the test keys need to
be updated.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
The algorithm name for expanded ed25519 keys has changed, so the test
keys need to be updated.
|
| | | | | |
|