summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | rpc: State the integer round-trip range limitsIan Jackson2023-04-031-1/+5
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889617
| * | | rpc: Don't talk about "properties" of objects: rather, "members"Ian Jackson2023-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889616
| * | | rpc: Move notes about cancellation to right sectionIan Jackson2023-04-031-7/+7
| | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889614
| * | | rpc: Be clearer about updates contentIan Jackson2023-04-031-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | You can't parse an update without knowing the request method (this was already stated elsewhere). Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889613
| * | | rpc: Change how we talk about objectsIan Jackson2023-04-031-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use just "object" in the introduction, but be specific that the abstract data type is I-JSON, even if we later invent other representations. Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889612
| * | | rpc: Right at top, say I-JSONIan Jackson2023-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889612
| * | | rpc: Change wording about responsesIan Jackson2023-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1078#note_2889611
| * | | rpc: Start on a list of the differences with JSON-RPCIan Jackson2023-03-241-0/+23
| | | |
| * | | rpc: Discuss ordering and bufferingIan Jackson2023-03-241-0/+13
| | | |
| * | | rpc: Define term `field`Ian Jackson2023-03-241-0/+2
| | | |
| * | | rpc: Discuss cancellation request objIan Jackson2023-03-241-0/+9
| | | |
| * | | rpc: Clarify what precisely the responses depend onIan Jackson2023-03-241-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The format of an update or result depends on the *method* but not the parameters or the subject. Declare that the format of an error is uniform. (It could have an enum in it.)
| * | | rpc: State that we're using I-JSONIan Jackson2023-03-241-0/+8
| | | |
| * | | rpc: Forbid troublesome numbers as idsIan Jackson2023-03-241-2/+4
| | | | | | | | | | | | | | | | | | | | This is a bit sad but I think we should have a conservative JSON profile.
| * | | rpc: Be more explicit about ignoring JSON fieldsIan Jackson2023-03-241-1/+4
| | | |
| * | | rpc: Clarify/restate method definition requirementsIan Jackson2023-03-241-9/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In particular, abolish the notion of a "response type". Since responses don't come with a discriminant, each method may have only one success response format (although of course that format might itself have optional fields or be an enum or something).
| * | | rpc: Use Object for method targets and JSON object for document objectsIan Jackson2023-03-241-29/+31
| | | |
| * | | rpc: Make `params` in a request non-optionalIan Jackson2023-03-241-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | *If* it is optional that would depend on the request, which is a bit complicated to specify precisely. And making it mandatory is more orthogonal.
* | | | Merge branch 'x25519' into 'main'Nick Mathewson2023-04-042-3/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-llcrypto: Pin x25519-dalek version, bump our crate version Closes #807 See merge request tpo/core/arti!1108
| * | | | tor-llcrypto: Pin x25519-dalek version, bump our crate versiontor-llcrypto-v0.4.4Ian Jackson2023-04-042-3/+3
| | | | | | | | | | | | | | | | | | | | Fixes #807
* | | | | Merge branch 'timerange' into 'main'gabi-2502023-04-031-1/+35
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | Ergonomic improvements to TimerangeBound See merge request tpo/core/arti!1105
| * | | | TimerangeBound: Provide .as_ref() and .as_deref()Ian Jackson2023-04-031-1/+33
| | | | |
| * | | | TimerangeBound: derive Debug and Clone, and Eq/PartialEq in testsIan Jackson2023-04-031-0/+2
| | | | |
* | | | | Merge branch 'debug-hex' into 'main'Nick Mathewson2023-04-034-5/+92
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Debug two types as compact hex strings See merge request tpo/core/arti!1104
| * | | | Debug as hex strings for HsBlindId and HsDirIndexIan Jackson2023-03-313-5/+10
| | | | |
| * | | | tor-basic-utils: Provide impl_debug_hexIan Jackson2023-03-311-0/+82
| | | | |
* | | | | Merge branch 'hserror' into 'main'Ian Jackson2023-03-313-0/+52
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-error: Errors for hidden services See merge request tpo/core/arti!1099
| * | | | | tor-error: Add missing footnoteIan Jackson2023-03-311-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | I C&P this from tor-dirmgr, and missed this part.
| * | | | | Fix duplicated licence in Cargo.tomlgabi-2502023-03-311-1/+0
| | | | | |
| * | | | | tor-error: Add OnionServiceNotRunning errorIan Jackson2023-03-311-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We'll want this later. Define it now, though, since we've discussed it here https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1099#note_2892020
| * | | | | tor-error: Put OnionService errors behind a new experimental-api featureIan Jackson2023-03-313-0/+14
| | | | | |
| * | | | | tor-error: Document cargo featureIan Jackson2023-03-311-0/+4
| | | | | |
| * | | | | tor-error: Errors for hidden servicesIan Jackson2023-03-311-0/+18
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So far these are just the errors that occur during descriptor fetch. There will be more later as we have more code in tor-hsconn. This is very user-facing; use the "onion service" terminology.
* | | | | Merge branch 'netdir' into 'main'Ian Jackson2023-03-312-29/+90
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-netdir: API changes to support hsconn hsdir fetch See merge request tpo/core/arti!1094
| * | | | | tor-netdir: iter_filter_secondary: Make it less of a messIan Jackson2023-03-301-13/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1094#note_2891857
| * | | | | tor-netdir: Expand on comment for HsRingRings::itergabi-2502023-03-301-1/+1
| | | | | |
| * | | | | tor-netdir: Minor docs fixesgabi-2502023-03-301-2/+3
| | | | | |
| * | | | | tor-netdir: Provide accessor for params field of HsDirParamsIan Jackson2023-03-301-0/+5
| | | | | |
| * | | | | tor-netdir: Implement hs_dirs accessorIan Jackson2023-03-301-2/+16
| | | | | |
| * | | | | tor-netdir: hs_dirs accessor: change semantics, type, and nameIan Jackson2023-03-301-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't have it take the TP, so that the caller must call it multiple times. Instead, have it return all the relevant relays.
| * | | | | tor-netdir: Provide hs_all_time_periods instead of ..._secondary_...Ian Jackson2023-03-301-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't think the server-side support will want to explicitly call current and then secondary. Rather, it will want to iterate over all the relevant ones. And fix the name, and add another comment about whether we need this.
| * | | | | tor-netdir: Rename hs_time_periodIan Jackson2023-03-301-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change its name to hs_* like we do with things at this layer. But, it turns out, that at least for hs client connections to fetch the descriptor, I don't seem to need to call it yet ? Maybe it's not needed.
| * | | | | tor-netdir: Sort out HsDirOpIan Jackson2023-03-301-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Change its name to Hs* like we do with things at this layer * Make the Upload variant cfg-conditional
| * | | | | tor-netdir: Reorganise iteration over hsdir ringsIan Jackson2023-03-301-4/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provide iter_for_op, by changing iter into iter_filter_secondary and having a new entrypoint iter.
| * | | | | tor-netdir: Provide relay_by_rs_idxIan Jackson2023-03-301-0/+16
| | | | | |
* | | | | | Merge branch 'docs-runes' into 'main'Ian Jackson2023-03-311-2/+5
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | CONTRIBUTING.md: Improve wording about docs builds recommendations See merge request tpo/core/arti!1090
| * | | | | CONTRIBUTING.md: Explicitly support --document-private-itemsIan Jackson2023-03-301-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have many internal doc comments with curated links etc., so we should actually mention that formatting and reading those can be useful.
| * | | | | CONTRIBUTING.md: Recommend --workspace when building docsIan Jackson2023-03-301-2/+3
| | | | | |
* | | | | | Merge branch 'hsdesc-encoder-client-auth' into 'main'gabi-2502023-03-315-157/+310
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Stop requiring the caller to supply `AuthClient`s. See merge request tpo/core/arti!1087
| * | | | | | Generate a new KP_hss_desc_enc keypair for each new descriptor.Gabriela Moldovan2023-03-312-33/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, to build descriptors for hidden services with client auth enabled, in addition to the list of authorized clients, users of `HsDescBuilder` were required to also provide a descriptor encryption keypair and a descriptor cookie. This was potentially dangerous and/or error-prone, because the ephemeral encryption key and the descriptor cookie are expected to be randomly generated and unique for each descriptor. This change makes `ClientAuth` private to the `hsdesc::build` module and updates `HsDescBuilder` to build `ClientAuth`s internally. Users now only need to provide the list of authorized client public keys. Signed-off-by: Gabriela Moldovan <[email protected]>