| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Instead of saying "no errors before authentication",
we now say "no syntax errors ever."
This should be easier to implement.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
(This is obsoleted by arti-rpc-client-core.)
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Notably, identifiers SHOULD be C identifiers,
and everything has a namespace.
Reserve "rpc" as a namespace for things that were previously
unnamespaced.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is probably still far too informal, but it is better than our
previous documentation.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
(Note what isn't implemented,
and one more kind of authentication we'll likely build someday.)
|
| | | | | | |
|
| |/ / / /
| | | |
| | | |
| | | | |
(Markdown convention deprecates having more than one top-level heading.)
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
tor-hsservice: Improve descriptor publisher status reporting
Closes #1216 and #1572
See merge request tpo/core/arti!2397
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
netdirs (fmt).
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This allows us to set SRVs for example (needed because by default, the
test `NetDir` is built from a consensus that doesn't contain any SRVs).
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We will need to return a list of descriptor upload errors.
We can't return a `Vec<RetryError<DescUploadError>>` here because
`DescUploadError` is a lower-level error type that can't express that
e.g. the upload timed out.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We never return `UploadError::Timeout` (timeouts are represented as
`BackoffError::Timeout`).
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This type is a `Result`, renaming for clarity.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
The descriptor publisher docs live in the `publisher` module.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Closes #1216
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This makes it clearer that some of these functions are essentially
infallible.
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This enables us to report a "broken" service status if restricted
discovery is enabled but the authorized_clients list is empty.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Closes #1572
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Currently, the `DegradedReachable` status is only reported by the the IPT
manager and `DegradedUnreachable` is unused.
Soon we'll the publisher reporting `DegradedReachable` or
`DegradedUnreachable` or `Running`, depending on how the descriptor
uploads went.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This will allows us determine the ComponentStatus of the publisher
(it'll be either `Running` or `Degraded`, depending on whether the
upload failed).
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
After uploading the descriptor, the publisher transitions into the
`Idle` state. This transition happens even if the upload was
unsuccessful, so it shouldn't cause the onion service status to become
`Running` (because `Running` implies the service is fully reachable, and
if the publisher failed to upload the descriptor to some or all HsDirs,
that won't necessarily be the case).
A future commit will set the publisher's onion svc `State` to
`Running`/`Recovering`/`Broken` according to the upload status.
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This resolves a TODO.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Previously, these were stored in the immutable state behind a mutex, but
since they're not really immutable (we update them if the config
changes), it makes more sense to put them in `State`.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
These TODOs were addressed a while ago (when we introduced
`IptManager::ipt_errors`).
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
These aren't dead code anymore, with the exception of
`PublisherStatusSender::send_recovering`, which isn't used.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
We now log the onion service status on change.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This is already implemented.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This TODO was added in !2353 and was supposed to be about reporting a
broken/degraded onion service status if the restricted discovery config
watcher fails.
|
| |\ \ \ \ \
| |_|_|_|/
|/| | | |
| | | | |
| | | | | |
Fix a typo in WRONG_PROTOCOL_PAYLOAD.
See merge request tpo/core/arti!2403
|
| |/ / / / |
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Add a deficit field to tor_bytes::Error::Truncated
Closes #1592
See merge request tpo/core/arti!2390
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Suggested in
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2390#note_3072975
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
This will allow us to fix #1592, but it doesn't do so yet.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is going to want to do something more complicated (as described
in the docs).
In this commit we change all the tests that are expecting Truncated
errors. That reduces noise in the next commit.
|