| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Ticket #1509 will probably get rid of this constant,
but for now we may as well put it in one place.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Here we make sure that we can actually skip over other proxy formats
in the future.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Renamed address to tcp_address, and made it optional, so that later
we can have a unix_path, etc.
On deser side, add support for unrecognized listener types.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is done so that we can make "not authenticated" a non-internal
error, under the theory that someday unauthenticated connections
might be exposed.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This belongs in a spec, but adding things to a spec is slow and
fraught. Instead we'll put it here for now and move it later.
There are some XXXXs about "finalizing" the design that we need to
resolve before we can merge !2373 and implement stream creation in
`arti-rpc-client-core`.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Currently this behaves the same as get_proxy_info,
but this may change in the future, if we introduce RPC-unsuitable
proxy ports.
(Also rename the get_proxy_info method to avoid x_ prefix.)
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
There's a blocking TODO here about exposing socks error codes that
I still need to solve.
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
Requires #1523.
Implements #1524.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
maint/downgrade-dependencies: Update exceptions.
See merge request tpo/core/arti!2398
|
| | | | |
| | | |
| | | |
| | | | |
Adding an exception for now, as per #1608
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fixes the `minimal-versions` failure:
```
error: package ID specification `[email protected]` did not match any packages
Did you mean one of these?
[email protected]
```
(we don't actually need to un-downgrade `futures-lite` anymore)
But we *do* need to un-downgrade `event-listener`
(`async-global-executor` needs `event_listener::EventListener` to be `UnwindSafe`,
and in `event-listener 3.0.0`, they are not)
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
arti: Temporarily disable a flaky test.
See merge request tpo/core/arti!2391
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This effectively reverts 71e3d52f5aeb34ca7bca80053079a8c7505b99ac,
which was itself a revert of disabling this test the first time.
Sadly it still doesn't seem to be reliable. We have agreed on IRC to
disable it for now.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
Use dpkg-buildpackage -uc -b in CI
See merge request tpo/core/arti!2368
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
The effect of this is just to make the job run sooner. But it's
actually quite unlikely to fail, and nothing now depends on it, so we
should run it later along with the rest of the jobs in the `test`
stage.
|
| | | |
| | |
| | |
| | |
| | | |
This avoids any inherited data (eg from previous jobs) influencing the
output.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Pass -b to dpkg-buildpackage, meaning "build binaries only".
* Don't `need` the source package job
* Don't try to import source package artifacts
This means we're not building the .debs from the .dsc, but we weren't
doing that anyway. (.dscs are IMPO obsolete...)
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
By default dpkg-buildpackage signs the output if the d/changelog isn't
UNRELEASED.
If we ever change the d/changelog, we don't want it to fail because it
tries to sign things.
In the future, if and when we want to sign things, that probably won't
be done directly in gitlab CI when the package is built.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
arti: Make the config watcher debounce interval configurable.
Closes #1589
See merge request tpo/core/arti!2387
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously, the `reload_cfg::test::watch_multiple` test would take about
3s to run. This test modifies 3 files and waits for the corresponding
`reconfigure()` events to fire. Because of the
`sleep(DEBOUNCE_INTERVAL)` in `run_watcher`, it would wind up waiting
for about 1s for each of them.
This makes the event debouncing optional, and disables it in the tests.
Closes #1589
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
tor-keys: New crate for Tor key declaration
Closes #1137
See merge request tpo/core/arti!2356
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: David Goulet <[email protected]>
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Everything copied in the previous commits to tor-keys is now removed and
tor-keys crate is used accross the code.
Minor changes to tor-keys to accomodate this change.
Part of #1137
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As stated in the comment added in this commit, this is temporary as we
want tor-hscrypto to start using tor-keys and define these
implementation there.
Part of #1137
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Move EncodableKey, ToEncodableKey and Keygen to tor-keys crate from
tor-keymgr.
To pull this off, an err.rs was added taken from tor-keymgr but stripped
down to what was needed only. No code was changed in any of the files
copied from tor-keymgr.
Our deftly macro now auto implement these traits for the key wrapper
type created. This gives the ability of the wrapper to be used by a
tor-keymgr::Keystore.
In order to pull this off, most of the SSH code has been moved into this
crate (ssh.rs) meaning that its ABI/API now resides in the tor-keys
trait outside of tor-keymgr.
Note that Sealed was not put back because the deftly macro is designed
to be used outside of this crate and thus implementing EncodableKey. The
alternative is that we would need to force all arti keys to be declared
in this crate which is not great because it then exposes all these key
types to the world outside their subsystem.
Part of #1137
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Some object use path where put explicitly in order to avoid the macro
user to need to import "ed25519" module.
Signed-off-by: David Goulet <[email protected]>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The derive ed25519 keypair macro now implements the keymgr trait so the
key wrapper can now be used with a keystore without needing to specify
it in the tor-keymgr crate.
For this to work, a slight change to the KeygenRng trait was needed as
in to expect the CryptoRngCore trait which is what ed25519-dalek
requires.
And also, the removal of the Sealed trait since now it is accepted to
implement these traits outside tor-keymgr.
Fixes #1137
Signed-off-by: David Goulet <[email protected]>
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
At this commit, we also add a derive-deftly macro for ed25519 keypair
along a helper macro that can define a wrapper around a lower-level
ed25519::Keypair.
Part of #1137
Signed-off-by: David Goulet <[email protected]>
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Remove semver.md files.
See merge request tpo/core/arti!2385
|
| | | | |
| | | |
| | | |
| | | | |
The 1.2.7 release is out so we won't be needing these anymore.
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
arti: Write the test config atomically.
Closes #1549
See merge request tpo/core/arti!2375
|
| | | | |
| | | |
| | | |
| | | | |
This reverts commit ececf6be2eaffc201666cd3413dc19c025be66de.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The flakiness from the `watch_single_file` test was caused by the race
between the config write, and the fake SIGHUP "signal" sent on
`sighup_tx`: sometimes, the sighup would get handled between creating
the config file and writing its contents. In those cases, the config
received in `TestModule::reconfigure` would be `Default::default()`,
which caused an assertion to fail (because the test is expecting to
receive the `ArtiConfig` it wrote to disk).
Closes #1549
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
Configure and set up a MemoryQuotaTracker in TorClient
See merge request tpo/core/arti!2374
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2374#note_3070500
|
| | | | |
| | | |
| | | |
| | | | |
Nothing uses this yet.
|
| | | | | |
|
| | | | | |
|