| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The caller uses `KeystoreSelector` to specify which keystore to insert
the new key into (only `KeystoreSelector::Id` and
`KeystoreSelector::Default` are supported for `insert`).
The ability to insert keys in a particular keystore will come in handy
when we implement the key management CLI (the CLI will have an option
for specifying the keystore to access/modify).
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
This error will be returned by `KeyMgr` if the caller tries to access a
keystore that does not exist, or if the requested `KeystoreSelector`
cannot be applied.
|
| | | |
|
| | |
| |
| |
| |
| | |
This will be used by `KeyMgr::insert` after we add an additional
argument to `insert` for specifying the keystore it should be using.
|
| | |
| |
| |
| |
| |
| | |
This will enable the `KeyMgr` to look up `Keystore`s by ID (which is
a requirement for disambiguating the semantics of `insert`, which
currently tries to "guess" which keystore it should be using).
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
This makes the code slightly less verbose.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
enabled.
Hiding the underlying value of `enabled` enables us to give it a
different `auto` value depending on whether the `keymgr` feature is
enabled or not (it defaults to `true` if `keymgr` is enabled, and
`false` otherwise).
|
| | |
| |
| |
| |
| |
| |
| | |
The `experimental-api` was only meant to apply to the use of the
unstable `ArtiNativeKeystoreConfig` in the Arti config.
`experimental-api` was _not_ supposed to be used for enabling/disabling
the keystore (that's what the `enabled` flag is for).
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
If the Arti keystore is disabled, we have nothing to initialize the
`KeyMgr` with, so we might as well make it optional.
|
| |\ \
| | |
| | |
| | |
| | | |
Add getters to a couple of config builders
See merge request tpo/core/arti!1425
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
It's a bit of a wart that tor-ptmgr calls these "protocols" and
tor-guardmgr calls these "transport names".
|
| | | |
| | |
| | |
| | |
| | | |
BridgeConfigBuilder is Serialize so this isn't making any new API
promises. Ideally we'd have getters like this everywhere.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
cargo doc --locked --document-private-items --workspace --all-features
warning: unclosed HTML tag `CountryCode`
--> crates/tor-geoip/src/lib.rs:90:54
|
90 | /// We store these as NonZeroU8 so that an Option<CountryCode> only has to
| ^^^^^^^^^^^^^
|
= note: `#[warn(rustdoc::invalid_html_tags)]` on by default
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
tor-linkspec: impl AsRef<str> for PtTransportName
See merge request tpo/core/arti!1426
|
| | |/ / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Update pwd-grp to 0.1.1 to fix MacOS build etc.
See merge request tpo/core/arti!1427
|
| | |/ /
| | |
| | |
| | | |
This also gets rid of a duplicate copy of derive-adhoc.
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Bump requirement to rlimit 0.10.1
See merge request tpo/core/arti!1423
|
| | | |
| | |
| | |
| | |
| | |
| | | |
There was a bug in 0.10.0 that broke MacOS.
Part of #963.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
geoip: Enable the niche optimization for CountryCode.
See merge request tpo/core/arti!1384
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Since we're going to be using `Option<CountryCode>` all over, let's
save the extra byte.
Sadly this required std::mem::transmute(), which is unsafe, so maybe
we should think twice.
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
geoip: Allow ASNs as zeros when creating NetDefn
Closes #961
See merge request tpo/core/arti!1417
|
| | | | |
| | | |
| | | |
| | | | |
to be able to debug it, for instance.
|
| | | |/
| |/|
| | |
| | |
| | |
| | |
| | | |
so that GeoipDb can be created from files including ASNs generated with
tor/scripts/maint/geoip/geoip-db-tool.
Closes #961
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This would be a break in higher-layer crates which incorproate this
error but:
1. That's just arti-client which hides it behind the detailed errors
cargo feature
2. I'm hoping cargo-semver-checks would spot it, anyway.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The effect is that everywhere a RetryError is used, the error sources
for the contained errors will be Display'd.
In tor-hsclient we no longer need to explicitly wrap things up in
tor_error::Report.
|
| | | |
| | |
| | |
| | |
| | | |
We're going to require that a RetryError contains things that are
AsRef<dyn Error> and ParseIntError isn't so we need a newtype.
|
| | | |
| | |
| | |
| | |
| | | |
This code came from tor-error. So now tor-error depends on
retry-error.
|
| |/ /
| |
| |
| | |
We're about to want this.
|
| |\ \
| | |
| | |
| | |
| | | |
Mid-month dependency upgrades
See merge request tpo/core/arti!1412
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
(Everything else is already on 0.11.0.)
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
Replace use of unmaintained users crate with homegrown pwd-grp
Closes #877
See merge request tpo/core/arti!1410
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We don't use OsString now except where it appears in our public API,
or where we get it from std::env.
Moving the `use` statements into the use sites enabled me to see
that I had found all the places I wanted to change.
|
| | | |
| | |
| | |
| | | |
This function is actually (properly) fallible now.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
MockPwdGrpProvider has internal mutability and is Sync, so its add
functions take &self.
|
| | | |
| | |
| | |
| | |
| | | |
This gets rid of some unsafe code here, with doubtful error handling,
in favour of the unit-tested version in pwd-grp.
|