summaryrefslogtreecommitdiff
path: root/crates/tor-hsservice/src/publish
Commit message (Collapse)AuthorAgeFilesLines
* tor-hsservice: Remove unnecessary let-binding.Gabriela Moldovan2024-07-091-1/+2
|
* tor-hsservice: Specify the type when discarding value.Gabriela Moldovan2024-07-092-3/+3
| | | | | | | This should give us some confidence that we are not accidentally discarding results we should actually be using. Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2251#note_3046219
* tor-hsservice: Ignore uninteresting netdir events in the publisher.Gabriela Moldovan2024-07-091-1/+12
| | | | This also makes the publisher exit if the netdir event stream ends.
* tor-hsservice: Resolve warning about unused response variable.Gabriela Moldovan2024-07-091-1/+1
| | | | | | | | | It is OK to discard the response string (the response is validated below where we call `DirResponse::into_output_string()`). (Perhaps we should just make `DirResponse::check_ok()` public. That way, we could validate the response without having to handle its output string).
* tor-hsservice: Resolve a handful of unused variable warnings.Gabriela Moldovan2024-07-092-4/+4
|
* tor-hsservice: Remove unused field from TimePeriodContext.Gabriela Moldovan2024-07-091-3/+0
|
* tor-hsservice: Remove unused variables.Gabriela Moldovan2024-07-092-14/+0
|
* tor-hsservice: Remove unnecessary locking.Gabriela Moldovan2024-07-091-2/+0
|
* tor-hsservice: Remove unused code.Gabriela Moldovan2024-07-092-186/+0
| | | | This is being reimplemented as #1292
* tor-hsservice: Remove unused publisher functions.Gabriela Moldovan2024-07-091-5/+0
| | | | | | | This removes a couple of unimplemented functions. These were supposed to be the starting point for #1217, but we won't be implementing that any time soon.
* Work around clippy::doc_lazy_continuation false positivesIan Jackson2024-07-081-4/+4
| | | | | | | See https://github.com/rust-lang/rust-clippy/issues/13001 I think it's OK to change this in our tree, nevertheless.
* tor-hsservice: Move optional timeout to optionally_timeout().Gabriela Moldovan2024-04-021-7/+32
| | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2064#note_3014793
* tor-hsservice: s/timeout/overall_timeout for clarity.Gabriela Moldovan2024-04-021-2/+2
|
* tor-hsservice: Add a test for the backoff runner's single_attempt_timeout().Gabriela Moldovan2024-04-021-0/+51
|
* tor-hsservice: Use single_attempt_timeout in backoff Runner impl (fmt).Gabriela Moldovan2024-04-021-10/+10
|
* tor-hsservice: Use single_attempt_timeout in backoff Runner impl.Gabriela Moldovan2024-04-022-19/+38
| | | | | | | | | | | | | | | | | !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
* Run maint/add_warning.Nick Mathewson2024-03-133-0/+3
|
* Fix some rustdoc warnings.Nick Mathewson2024-03-122-4/+4
| | | | | These are mostly about explicitly linking to things that were already in scope.
* BackoffSchedule refactoring and improvementsNeel Chauhan2024-03-072-22/+38
|
* tor-hsservice: client auth: Only allow `dir:...` to have valid unicodeIan Jackson2024-03-051-2/+2
| | | | | | | | | | | | 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.
* tor-hsservice: prelude: Don't import postage::sink::SendErrorIan Jackson2024-03-051-4/+3
| | | | This is too vague.
* tor-hsservice: prelude: Move imports from reupload_timer.rsIan Jackson2024-03-051-4/+1
|
* tor-hsservice: prelude: Promote imports from reactor.rsIan Jackson2024-03-051-4/+0
| | | | Move these two publisher-specific imports to publish.rs.
* tor-hsservice: prelude: Move imports from reactor.rsIan Jackson2024-03-051-48/+2
|
* tor-hsservice: Refer to mpsc::{Sender, Receiver} by their module nameIan Jackson2024-03-051-4/+4
| | | | Just "Sender" and "Receiver" is rather vague.
* tor-hsservice: prelude: Promote imports from descriptor.rsIan Jackson2024-03-051-1/+0
| | | | Move these two publisher-specific imports to publish.rs.
* tor-hsservice: prelude: Move imports from descriptor.rsIan Jackson2024-03-051-21/+2
| | | | We use these via an import of the parent, publish.rs.
* tor-hsservice: prelude: Move imports from backoff.rsIan Jackson2024-03-051-11/+1
|
* tor-hsservice: Change: future::pending: use the one from futures-rsIan Jackson2024-03-051-1/+2
| | | | | | | publish::backoff used std::future::pending, whereas generally we mostly use futures::future::pending. This change ought to make no difference.
* tor-hsservice: Move svc.rs contents into lib.rsIan Jackson2024-03-051-1/+1
|
* tor-hsservice: Move publish out of svcIan Jackson2024-03-054-0/+2569