summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | maint/extract-md-links: New scriptIan Jackson2024-05-021-0/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will be used by gen_md_links, instead of the regexp-based ad-hoc parser there. The existing parser gets wrong things like " `#[must_use]` ", thinking that's a link. Use a proper Markdown parser. But, we must commit some abuse, sadly.
| * | | | maint/gen_md_links: Soup up argument parserIan Jackson2024-05-021-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I was going to want to call this from another program with a filename, which meant I needed to unparse the arguments, which meant I wanted a competent parser to feed things to. As it happens my caller doesn't use this any more, but let's keep the improvement anyway.
| * | | | maint/gen_md_links: Honour \ escapingIan Jackson2024-05-021-0/+1
| | | | |
* | | | | Merge branch 'vanguards-stub-terminology' into 'main'gabi-2502024-05-163-71/+83
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-circmgr: Replace STUB/STUB+ terminology with SHORT/EXTENDED. Closes #1339 See merge request tpo/core/arti!2161
| * | | | | tor-circmgr: Fix a broken doc link.Gabriela Moldovan2024-05-161-1/+1
| | | | | |
| * | | | | tor-circmgr: Clarify module-level docs.Gabriela Moldovan2024-05-161-0/+6
| | | | | |
| * | | | | tor-circmgr: Replace STUB/STUB+ terminology with SHORT/EXTENDED (fmt).Gabriela Moldovan2024-05-161-1/+3
| | | | | |
| * | | | | tor-circmgr: Replace STUB/STUB+ terminology with SHORT/EXTENDED.Gabriela Moldovan2024-05-163-46/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous STUB/STUB+ terminology was confusing, because STUB and STUB+ are both "circuit stubs" (but STUB is shorter than STUB+). Closes #1339
| * | | | | tor-circmgr: Rename HsCircStubKind::Stub to HsCircStubKind::Short.Gabriela Moldovan2024-05-163-25/+31
| | |_|/ / | |/| | | | | | | | | | | | | Part of #1339
* | | | | Merge branch 'enforce_sendme_max' into 'main'Nick Mathewson2024-05-163-5/+16
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | proto: Explicitly enforce maxima on SENDME windows. See merge request tpo/core/arti!2150
| * | | | proto: Explicitly enforce maxima on SENDME windows.Nick Mathewson2024-05-143-5/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No actual bug here, just technical debt: For `SendWindow`s, our tag system already ensured that we rejected any SENDME that didn't correspond to an appropriate drain. Still, it doesn't hurt to check. For `RecvWindow`s, it would have been a protocol violation if we ever did this, but it makes sense to make it an internal error if we try. Part of #1383.
* | | | | Merge branch 'script-suffixes' into 'main'Ian Jackson2024-05-164-2/+41
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Forbid script suffixes See merge request tpo/core/arti!2153
| * | | | | Forbid scripts names from specifying their implementation languageIan Jackson2024-05-162-0/+39
| | | | | |
| * | | | | Rename tests/shadow/run.sh to runIan Jackson2024-05-163-2/+2
| | |/ / / | |/| | |
* | | | | Merge branch 'enforce_method_names' into 'main'Nick Mathewson2024-05-167-11/+138
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RPC: Enforce method name format. Closes #823 See merge request tpo/core/arti!2149
| * | | | | RPC: Give error from RpcMgr::new if method name is sufficiently bad.Nick Mathewson2024-05-142-5/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (We don't give an error about unrecognized namespaces (for now), since we have no way to opt in to them.)
| * | | | | RPC: Enforce method name format.Nick Mathewson2024-05-144-1/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to do this carefully, since we want our system to be extensible with new namespaces. First, when we are constructing an RpcMgr, we _warn_ about any method names that are misformed. Second, we add a test in the `arti` crate to fail if any method names are invalid. This will only catch method names in crates that `arti` depends on.
| * | | | | RPC: Use RPC method names in a consistent format.Nick Mathewson2024-05-143-6/+6
| | |/ / / | |/| | | | | | | | | | | | | | | | | | Specifically, we want a single colon, and we want our method names to be in snake_case.
* | | | | Merge branch 'use-locked-build' into 'main'Ian Jackson2024-05-1610-13/+13
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | doc: Use locked build See merge request tpo/core/arti!2157
| * | | | doc: Use locked buildpinkforest2024-05-1610-13/+13
| | | | |
* | | | | Merge branch 'cargo-publish' into 'main'gabi-2502024-05-151-3/+18
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Soup up maint/cargo-publish a bit See merge request tpo/core/arti!2158
| * | | | | maint/cargo-publish: Allow overriding of origin and branchIan Jackson2024-05-151-0/+6
| | | | | |
| * | | | | maint/cargo-publish: Split handling of origin and branchIan Jackson2024-05-151-3/+3
| | | | | |
| * | | | | maint/cargo-publish: Add --force optionIan Jackson2024-05-151-0/+9
| |/ / / /
* | | | | Merge branch 'tor-circmgr-0.18.1' into 'main'Ian Jackson2024-05-154-4/+44
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | arti: 1.2.3 release Closes #1400 and #1409 See merge request tpo/core/arti!2156
| * | | | Merge remote-tracking branch 'upstream/main' into HEADGabriela Moldovan2024-05-15129-1286/+2752
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge branch 'example_one_hop' into 'main'gabi-2502024-05-153-0/+69
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | arti-client: Add one-hop circuit example See merge request tpo/core/arti!2148
| * | | | arti-client: Add one-hop circuit examplejuga2024-05-153-0/+69
|/ / / / | | | | | | | | | | | | Co-authored-by: gabi-250 <[email protected]>
* | | | Merge branch 'vanguards-extend-stub' into 'main'gabi-2502024-05-142-22/+98
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-circmgr: If necessary, extend the circuit to become STUB+. Closes #1400 and #1409 See merge request tpo/core/arti!2145
| * | | | tor-circmgr: Reword a somewhat inaccurate comment about vanguards.Nick Mathewson2024-05-141-1/+1
| | | | |
| * | | | tor-circmgr: Fix path not being extended if lite vanguards are enabled.Gabriela Moldovan2024-05-141-1/+10
| | | | |
| * | | | tor-circmgr: Assign error-handling closure to variable.Gabriela Moldovan2024-05-141-7/+10
| | | | | | | | | | | | | | | | | | | | We will soon need to reuse this.
| * | | | tor-circmgr: Rename neighbor_exclusion variables for clarity.Gabriela Moldovan2024-05-141-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I am about to reuse one of these on the "lite" vanguards branch. I am renaming them to make it easier to see which one of the two I will be using.
| * | | | tor-circmgr: After building the HS path, assert the length is correct.Gabriela Moldovan2024-05-141-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One of these assertions currently fails, because we have a bug in the vanguard path builder: if lite vanguards are enabled, we only build 2-hop circuits instead of 3.
| * | | | tor-circmgr: If necessary, extend the circuit to become STUB+.Gabriela Moldovan2024-05-131-8/+42
| | | | | | | | | | | | | | | | | | | | Closes #1400
| * | | | tor-circmgr: Add a helper for extending HsCircStubs by one hop.Gabriela Moldovan2024-05-131-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're about to use this in `maybe_extend_stub_circuit` too. Part of #1400
* | | | | Merge branch 'allow_unused_store_method' into 'main'Alexander Færøy2024-05-141-2/+5
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | DirMgr: Clarify dead_code status on storage methods. See merge request tpo/core/arti!2151
| * | | | DirMgr: Clarify dead_code status on storage methods.Nick Mathewson2024-05-141-2/+5
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per comments on #1383, we're keeping these methods. This commit replaces the "TODO" comments with comments explaining why it's okay that this methods are unused. Part of #1383.
* | | | Merge branch 'rpc_stream_integration_v3' into 'main'Nick Mathewson2024-05-1412-75/+527
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | RPC: Allow SOCKS applications to create streams. See merge request tpo/core/arti!2143
| * | | | RPC: Use a slightly less awful workaround in socks.rsNick Mathewson2024-05-141-20/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The problem was that Rust won't let us say ``` type ConnTarget<R> = Arc<dyn ClientConnectionTarget>; ``` because the R parameter wasn't used. Previously we solved this by using a macro instead of a type definition, which is ugly. I had been thinking previously I would need to declare some kind of additional wrapper type, and had shrunk from the verbosity. But @diziet pointed out that I could just use a 2-tuple unconditionally. It's still not beautiful, but it is less hideous than before.
| * | | | RPC: Add notes about ClientConnectionTargetNick Mathewson2024-05-141-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | Document its relation to the method system, and possible future evolution.
| * | | | RPC: Require DynClone for ClientConnectionError.Nick Mathewson2024-05-144-5/+9
| | | | |
| * | | | RPC: Seal ClientConnectionError.Nick Mathewson2024-05-141-2/+19
| | | | |
| * | | | RPC: Document RpcDataSTream state transitions.Nick Mathewson2024-05-141-1/+19
| | | | |
| * | | | RPC: Rename RpcDataStream::{Waiting => Launching}Nick Mathewson2024-05-141-5/+9
| | | | | | | | | | | | | | | | | | | | Also, improve documentation.
| * | | | proto: Fix compilation with stream-ctrl but not experimental-api.Nick Mathewson2024-05-142-2/+2
| | | | |
| * | | | Remove excess indentation from last commit.Nick Mathewson2024-05-141-14/+11
| | | | | | | | | | | | | | | | | | | | (This is a separate commit to make the branch more readable)
| * | | | RPC: Refactor socks interpretation to remove stream id, add isolation.Nick Mathewson2024-05-141-41/+41
| | | | |
| * | | | RPC: RpcDataStream type to act as a "stream-shaped hole"Nick Mathewson2024-05-147-0/+243
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The application creates these, using a new-stream-handle RPC command, on an object that can actually create streams. Then later, the application provides the (global) identity of one of these objects when it's making a SOCKS connection. This causes the object to take hold of a `DataStreamCtrl`.
| * | | | RPC: Add a trait that can be the target of SOCKS requestsNick Mathewson2024-05-127-28/+176
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (These will later become objects that can receive any application request, once we have HTTP connect.) For now, Session and TorClient implement this trait; but soon there will be a new type to hold on to the created DataStreamCtrl. There are some XXXXs here, marking code that is too ugly to live. I should fix it before I merge this branch.