| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
| |
This had somehow got duplicated; remove the one in derive.rs.
|
| | |
|
| |
|
|
|
|
| |
`KeyMgrBuilderError` is used in `KeyMgrBuilder`'s public API.
See https://gitlab.torproject.org/tpo/core/arti/-/issues/1358#note_3016025
|
| |
|
|
|
|
|
| |
This is a new feature in d-d 0.10.0.
Our currrent semver policy doesn't care about this, but let's not
encode that property in the tree and leave ourselves a booby-trap.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the combination of a number of separate commits, many of which
were generated by seddery, and then rebased and squashed.
Cargo.toml
perl -i~ -pe 's{^derive-adhoc}{derive-deftly = "0.10"}' crates/*/Cargo.toml
(not regenerated during rebase)
update Cargo.lock
`cargo fetch` without --locked
(regenerated during rebase)
seddery
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{^use derive_adhoc}{use derive_deftly}'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\bdefine_derive_adhoc\b}{define_derive_deftly}g'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\bAdhoc\b}{Deftly}g if m{derive}'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\#\[derive_adhoc\b}{#[derive_deftly}g'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{use derive_adhoc}{use derive_deftly}'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\bderive_adhoc\b}{derive_deftly_adhoc} if m{use.*deftly}'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\bderive_adhoc!}{derive_deftly_adhoc!}'
(not regenerated during rebase)
Manually add `#[derive_deftly_adhoc]` where needed.
seddery
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\#\[adhoc\b}{#[deftly}g'
git-ls-files | grep '\.rs$' | xargs perl -i~ -pe 's{\bderive_adhoc_template}{derive_deftly_template}'
(not regenerated during rebase)
Manually fix up an import
Manually update some builder attrs
Manually fix up tor_rtmock::time_core
This was missed in my seddery, due to me rebasing the branch and not
redoing the seddery.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
Change Path::display to this new function, in call sites where it's
being used for a diagnostic.
|
| |\
| |
| |
| |
| |
| |
| | |
deny clippy::unchecked_duration_subtraction
Closes #1304
See merge request tpo/core/arti!2008
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
Fix rust-analyzer problems seen with default features on Windows
See merge request tpo/core/arti!2009
|
| | |/
| |
| |
| |
| | |
The PermissionsExt::from_mode function is not available on Windows and
disabled in all other places of the file. Do so here as well.
|
| |/
|
|
|
|
|
| |
The slugs used for ArtiPath must be separated by slashes, which Windows
systems wouldn't do properly because paths are separated by backslashes.
Implement a platform-independent way of slug creation.
|
| |
|
|
| |
Closes #1202
|
| | |
|
| |
|
|
|
| |
`InvalidKeyPathComponentValue::Slug` now has to wrap a `String`, because
we can't get `HsIdParse` errors as static strings.
|
| | |
|
| | |
|
| |
|
|
| |
Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1960#note_2994317
|
| |
|
|
|
| |
The tor-keymgr crate *is* documented, but could use a better README and
some examples and/or usage guides.
|
| |
|
|
| |
This removes a remnant of a now-gone TODO.
|
| |
|
|
|
| |
It appears within the public API (it is the error type returned from
`KeySpecifierComponent`), so let's make it public.
|
| |
|
|
|
|
|
|
|
| |
Previously, `InvalidKeyPathComponentValue` was a single error type, with
the error message hard-coded to "invalid key denotator". This was wrong,
because this error is used for other types of invalid components too
(not just for invalid denotators).
Part of #1115
|
| | |
|
| |
|
|
|
|
|
| |
All `KeyPathError` variants now include the `ArtiPath` that caused the
error.
Part of #1115
|
| | |
|
| |
|
|
| |
Part of #1115
|
| |
|
|
|
|
| |
The offending `ArtiPath` should be included in the error.
Part of #1115
|
| |
|
|
|
|
|
|
|
| |
Originally, these functions converted to and from `ArtiPathComponent`.
In !1931, we replaced `ArtiPathComponent` with `Slug` without renaming
the conversion functions. Since we're converting to and from `Slug` now,
I think it makes sense to rename them too.
Part of #1115
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
I am not so sure it makes sense to unify these functions, so let's
remove the TODO.
|
| | |
|
| | |
|
| |
|
|
| |
Part of #1271
|
| |
|
|
| |
Part of #1271
|
| | |
|
| |
|
|
|
|
|
| |
NB: `KeyMgr::remove_with_type` will need to be renamed to
`KeyMgr::remove_entry`.
Part of #1271
|
| |
|
|
| |
We will soon use this in `KeyMgr`.
|
| |
|
|
| |
Part of #1271
|
| |
|
|
|
|
|
|
|
|
| |
This type will soon replace `(KeyPath, KeyType)` in
`KeyMgr::list_matching`.
The KeystoreEntry documentation mentions a couple of functions that
don't exist right now (they will be added in a subsequent commit).
Part of #1271
|
| |
|
|
|
|
|
| |
I don't think we need to do this right now, so I am downgrading this
TODO.
Part of #1115
|
| | |
|
| |
|
|
| |
Part of #1115
|
| |
|
|
|
|
| |
Not all algorithms have exactly 2 variants (`Public` and `Keypair`). For
example, for `ed25519`, we need `Public`, `Keypair`, and
`ExpandedKeypair`.
|
| | |
|
| |
|
|
|
|
|
| |
The `KeySpecifier` argument isn't needed yet, so let's defer this for
now.
Part of #1115
|
| | |
|