| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
| |
Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2064#note_3014793
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
!2024 introduced `BackoffSchedule::single_attempt_timeout()`, partially
implementing #1259. This completes the implementation by moving the
per-iteration timeout from the publisher reactor to `backoff::Runner`.
Addresses https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2024?commit_id=a4f2bf5cb7d4027a49b74ec5a5298f7b9a3cf62e#note_3005097
Note: the `if should_retry` block is intentionally misindented, to make
reviewing the actual changes easier. A future commit will fix the
indentation. I originally wanted to suggest reviewing this commit using
`--ignore-space-change`, but I found that it makes things a bit
confusing (it shows some funky indentation around the parts that have
both whitespace changes and modifications).
Closes #1259
|
| | |
|
| |
|
|
|
| |
These are mostly about explicitly linking to things that were
already in scope.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
We have Display/FromStr for AuthorizedClientConfig. We can't have
Display if it can containa PathBuf because a PathBuf may not have a
string representation.
If we want to support non-unicode in the future, we'll need an
escaping scheme and messing around with OsStr.
The config scheme here may not be the best, but right now I'm just
fixing uses of Path::display.
|
| |
|
|
| |
This is too vague.
|
| | |
|
| |
|
|
| |
Move these two publisher-specific imports to publish.rs.
|
| | |
|
| |
|
|
| |
Just "Sender" and "Receiver" is rather vague.
|
| |
|
|
| |
Move these two publisher-specific imports to publish.rs.
|
| |
|
|
| |
We use these via an import of the parent, publish.rs.
|
| | |
|
| |
|
|
|
|
|
| |
publish::backoff used std::future::pending, whereas generally we
mostly use futures::future::pending.
This change ought to make no difference.
|
| | |
|
| |
|