summaryrefslogtreecommitdiff
path: root/crates/tor-hsclient/src/connect.rs
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'timeouts' into 'main'Ian Jackson2023-06-291-16/+109
|\ | | | | | | | | tor-hsclient: Use estimated timeouts See merge request tpo/core/arti!1342
| * tor-hsclient: Add some notes about timeouts and hopsIan Jackson2023-06-291-0/+8
| |
| * tor-hsclient: Use estimated timeouts for introduction and rendezvous (fmt)Ian Jackson2023-06-291-2/+6
| |
| * tor-hsclient: Use estimated timeouts for introduction and rendezvousIan Jackson2023-06-291-12/+44
| | | | | | | | Replacing RPT_IPT_TIMEOUT is not entirely straightforward.
| * tor-hsclient: Use estimated timeout for hsdir fetches (fmt)Ian Jackson2023-06-291-1/+1
| |
| * tor-hsclient: Use estimated timeout for hsdir fetchesIan Jackson2023-06-291-5/+7
| |
| * tor-hsclient: Provide bodge constant for circuit lengthIan Jackson2023-06-291-0/+11
| |
| * tor-hsclient: Plumb timeout estimation machineryIan Jackson2023-06-291-0/+36
| |
* | Merge branch 'maxintro3' into 'main'Ian Jackson2023-06-291-4/+0
|\ \ | |/ |/| | | | | tor-netdoc: Handle anomalous numbers of introduction points See merge request tpo/core/arti!1332
| * tor-netdoc: Reject hsdescs with no intro pointsIan Jackson2023-06-281-4/+0
| | | | | | | | None of the existing NetdocErrorKinds seemed right.
* | tor-hsclient: Use proper CircParametersIan Jackson2023-06-291-2/+2
| | | | | | | | Fixes #935
* | tor-hsclient: err: Rename to RendezvousCompletionCircuitErrorIan Jackson2023-06-291-1/+1
| | | | | | | | | | | | | | Thsi is a bit of a mouthful now, but it is comprehensible. As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1336#note_2917235
* | tor-hsclient: Handle handshake completion error correctlyIan Jackson2023-06-281-6/+10
| |
* | tor-hsclient: Rename RendezvousCompletionCircuit from RendezvousCompletionIan Jackson2023-06-281-1/+1
|/ | | | | | We need to handle errors during the handshake. That also produces a `tor_proto::Error` but we want to print a different message for that, so let's rename this before adding the new variant.
* Merge branch 'reuse' into 'main'Ian Jackson2023-06-281-1/+17
|\ | | | | | | | | tor-hsclient: Discuss intro circuit extend-and-reuse See merge request tpo/core/arti!1298
| * tor-hsclient: Add a TODO SPEC re HS intro extend-and-reuseIan Jackson2023-06-231-0/+2
| |
| * tor-hsclient: Discuss intro circuit extend-and-reuseIan Jackson2023-06-231-1/+15
| | | | | | | | This is a summary of a discussion I had on IRC.
* | HS timeout parameters: Restore some TODOsIan Jackson2023-06-281-0/+4
| | | | | | | | | | | | | | | | | | Conceptually this is a fixup to HS configuration: Add retry parameters to configuration which erroneously removed these comments - they were intended to apply to *all* these parameters, not just the max attempts. Not marking it as fixup! because that would involve merge conflicts.
* | HS configuration: Use configured maximum attempts - commentIan Jackson2023-06-281-0/+4
| | | | | | | | | | | | | | Explain why this fallback to MAX is good. Discussion https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1305#note_2915927
* | HS configuration: Use configured maximum attempts (fmt)Ian Jackson2023-06-281-4/+12
| |
* | HS configuration: Use configured maximum attemptsIan Jackson2023-06-281-10/+9
| |
* | HS configuration: Plumb configuration through (fmt)Ian Jackson2023-06-281-2/+11
| | | | | | | | Apply deferred rustfmt churn.
* | HS configuration: Plumb configuration throughIan Jackson2023-06-281-2/+11
| | | | | | | | | | Invent a trait a la circmgr config for the hs client connector config. Plumb a suitable value all the way through to the code that will use it.
* | HS configuration: Add retry parameters to configurationIan Jackson2023-06-281-4/+2
| | | | | | | | | | | | | | I think these should go in `[circuit_timing]`. That section already has some retry parameters, so is not strictly *timing*. This is not honoured yet.
* | tor-hsclient: Completely hide hsdir identities in errorsIan Jackson2023-06-281-2/+2
| |
* | hsclient: Use HsDescDownloadRequest::set_max_len.Nick Mathewson2023-06-271-1/+12
| |
* | Merge branch 'circuit-bis' into 'main'Nick Mathewson2023-06-231-0/+6
|\ \ | | | | | | | | | | | | Add TODO SPEC for issues dealt with in !1299 See merge request tpo/core/arti!1303
| * | tor-hsclient: Add TODO SPEC for issues dealt with in !1299Ian Jackson2023-06-231-0/+6
| | |
* | | Merge branch 'circuit' into 'main'Alexander Færøy2023-06-231-21/+29
|\| | | | | | | | | | | | | | Drop intro circuit after ack, and deal with TODOs See merge request tpo/core/arti!1299
| * | tor-hsclient: Discuss HS descriptor expiry and revision countersIan Jackson2023-06-231-2/+4
| | | | | | | | | | | | And delete the corresponding TODO.
| * | tor-hsclient: Downgrade multi-ipt TODOIan Jackson2023-06-231-3/+4
| | |
| * | tor-hsclient: Downgrade rend/ipt parallelism TODOs and add linksIan Jackson2023-06-231-5/+8
| | |
| * | tor-hsclient: Downgrade ipt parallelism to a noteIan Jackson2023-06-231-2/+4
| | |
| * | tor-hsclient: Downgrade hsdir fetch parallelism to a noteIan Jackson2023-06-231-2/+4
| | |
| * | tor-hsclient: Drop intro circuit after ackIan Jackson2023-06-231-7/+5
| |/
* | tor-hsclient: Write a test plan for IPT state managementIan Jackson2023-06-231-0/+23
| | | | | | | | | | This won't be trivial. Put it in as a TODO HS rather than as part of this fix MR.
* | tor-hsclient: Expire old IPT dataIan Jackson2023-06-231-0/+9
|/ | | | | | | | If we don't use it for a whole, the whole of `Data` will be expired by the new machinery in !1290. However, we also need to expire individual IPTs. This bug doesn't seem to have had a relevant TODO.
* tor-hsclient: Apply deferred rustfmt churnIan Jackson2023-06-211-6/+3
|
* tor-hsclient: Drop most allows that are marked for removalIan Jackson2023-06-211-5/+0
|
* tor-hsclient: Drop a large number of unneeded importsIan Jackson2023-06-211-7/+4
|
* tor-hsclient: Do Debug the descriptor as part of DataIan Jackson2023-06-211-1/+0
| | | | | Empirically, this does not cause the descriptor to be spat out in normal operation. And it seems right.
* tor-hsclient: Drop some temporary trace callsIan Jackson2023-06-211-6/+2
| | | | | These were added as part of a debugging #885, and are no longer needed.
* tor-hsclient: Drop two eprintlns in production codeIan Jackson2023-06-211-2/+0
|
* tor-hsclinet: Promote dead code allows to cover all testsIan Jackson2023-06-211-1/+3
| | | | | We want to remove the crate-level dead code allows. The tests are incomplete and there are multiple todos for that.
* tor-hsclient: Use a formulaic TODO mark for incompleteness of testsIan Jackson2023-06-211-5/+5
|
* tor-hsclinet: Add a narrow allow, pending answer to open questionIan Jackson2023-06-211-0/+1
| | | | | I need to talk to HS expert(s) about when this circuit can be torn down.
* tor-hsclient: Remove unused Context.hs_blind_id_keyIan Jackson2023-06-211-3/+0
|
* tor-hsclient: Add an allow for a deliberately-unused variableIan Jackson2023-06-211-0/+1
|
* Merge branch 'netdoc-todos' into 'main'Nick Mathewson2023-06-161-0/+2
|\ | | | | | | | | Resolve many TODO HS items in tor-netdoc See merge request tpo/core/arti!1251
| * hsclient: note that we should check revctr.Nick Mathewson2023-06-161-0/+2
| |