| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
`handle_ipt_change` no longer handles the new intro points: it now
handles the IPT _change_ by updating the publish state of the reactor.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We don't really need to keep the handles around, so let's just discard
them.
This also updates `upload_all` to propagate any errors coming from
`build_sign`.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
It doesn't make sense to use an mpsc channel for rescheduling the
rate-limited uploads. If we use an mpsc channel and `upload_all` is
called multiple times in a short timespan, we will end up scheduling
more than 1 upload reattempt. So let's use postage::watch instead.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This updates the docs (and renames the channel for rescheduling
rate-limited uploads) for clarity.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We don't actually need an intermediate `Descriptor` struct (and keeping
a builder around doesn't make much sense either):
* the information from the onion service config can just be read from
the `config` field of the `Inner` (mutable) reactor state
* the IPT information is retrieved right before building the
descriptor
* any key information will (eventually) be read from the `KeyMgr`
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
CI: Remove amd64 tags
See merge request tpo/core/arti!1600
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We have a new runner, and apparently to use it better we want to
remove these tags.
TBH this seems rather unprincipled. In theory, shouldn't we tag with
`amd64` all the jobs that we definitely want to run on that arch, and
the others can run on whatever is the default? Then in the glorious
distant RISCV future everything will run on riscv64 by default? But
maybe we wouldn't want to switch that as part of the infrastructure
anyway?
Anyway, for discussion see
https://gitlab.torproject.org/tpo/tpa/team/-/issues/41295#note_2943129
|
| |\ \ \ \ \
| |_|_|/ /
|/| | | |
| | | | |
| | | | | |
HSS: Finish most of our configuration work
See merge request tpo/core/arti!1599
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
See thread at
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1599#note_2944510
for information about why `flatten` doesn't work here.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This matches our design.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We now have a single ProxyTarget action for the case where we accept
a request and forward it: `Forward`. It takes an encapsulation
method (currently only "direct") and a target address which can
be inet or unix.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This implementation uses rangemap to detect _all_ unreachable
patterns, which might be a little overkill, but it's actually
simpler to write it this way than to use an O(N^2) search to make
sure that no pattern is covered by a _single_ previous pattern.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Temporarily disable descriptor encryption configuration while we
figure out how it should work (see #1028)
|
| | | | | | |
|
| |/ / / / |
|
| |\ \ \ \
| | |/ /
| |/| |
| | | |
| | | | |
tor-hsservice: Implement PartialEq for HsClientDescEncKey, Anonimity, DescEncryptionConfig
See merge request tpo/core/arti!1601
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This will enable the descriptor publisher to tell whether it needs to
update the descriptor.
When the publisher is notified of an `OnionServiceConfig` change, it
checks whether the `anonimity` or `encrypt_descriptor` fields have
changed. If they have, it rebuilds and republishes the descriptor
using the new values (note: the publisher changes will be implemented in
a future commit).
|
| | | |/
| |/|
| | |
| | | |
Needed to implement `PartialEq` for `AuthorizedClientConfig`.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
tor-hsservice: ipt mgr: Notify Establisher to start accepting
See merge request tpo/core/arti!1598
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
We're going to want to talk about this in the Mockable trait.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The call site is going to want to do something other with the selected
IPTs than just publish them.
(This seems better than putting the Establisher notification in a
function called `publish_set`.)
|
| | | |
| | |
| | |
| | |
| | |
| | | |
We're going to split this function, and this will be in the 2nd half.
Code motion only.
|
| | |/
| |
| |
| |
| | |
We want a separate block so we can do other things besides just making
an IptPublishSet.
|
| |\ \
| |/
|/|
| |
| | |
tor-rtmock: task: Provide advance_* to replace advance
See merge request tpo/core/arti!1594
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
This will change the ordering in the docs.
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
We might loop, sleeping for zero. In real life this wouldn't last
long, but with the mock executor it can busy loop.
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
advance_by can be used in a more natural way, rather than having to
manually advance the clock in chunks to avoid triggering only the
first sleep.
It can also detects some "timeout triggered precisely" edge case bugs.
|
| | |
| |
| |
| | |
We should encourage the use of this, rather than Spawn::spawn()
|