summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * batching_split_before: Expand on the remaining use of "meta-iterator"Ian Jackson2023-02-131-1/+2
| |
| * batching_split_before: Fix link to BatchesWithHeader::subsequentIan Jackson2023-02-131-1/+1
| |
| * batching_split_before: Provide description for BatchesWithHeaderIan Jackson2023-02-131-0/+10
| |
| * batching_split_before: Document generics on our non-private typesIan Jackson2023-02-131-0/+9
| |
| * batching_split_before: Rename "prefix" to "header"Ian Jackson2023-02-132-24/+24
| | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1016#note_2877759 "_prefixed" becomes "_with_header". That mirrors the returned struct which is nice.
| * batching_split_before: Rename "start" to "batch-starting"Ian Jackson2023-02-131-27/+27
| | | | | | | | | | Roughly as per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1016#note_2877822
| * batching_split_before: Remove false assertions about batch contentsIan Jackson2023-02-131-4/+0
| |
| * batching_split_before: Clarify heading of BatchesWithPrefixIan Jackson2023-02-131-1/+1
| |
| * batching_split_before: docs improvements.Nick Mathewson2023-02-131-4/+8
| |
| * tor-netdoc: Use batching iterator in authcert.rsIan Jackson2023-02-102-62/+16
| | | | | | | | | | This gets rid of a lot of manual boolean state variable iterator stuff.
| * tor-netdoc: Use batching_split_before in HsDescInnerIan Jackson2023-02-101-25/+10
| |
| * tor-netdoc: Provide new batching_split_beforeIan Jackson2023-02-103-0/+402
| | | | | | | | | | | | | | | | | | | | | | | | There will be two call sites to demonstrate it. Eventually maybe this will want to be in tor-basic-utils, since it doesn't depend on any of the tor-netdoc types. But it would be sensible to wait until the situation with PeekableIterator and Itertools is improved. For now we make this #[doc(hidden)] to avoid it becoming part of our stable API.
| * tor-netdoc: Provide NetDocReader::into_iterIan Jackson2023-02-101-0/+7
| | | | | | | | | | This will avoids some hairy borrowck disasters when we try to use iterator adapters.
| * tor-netdoc: util: use super::* in testsIan Jackson2023-02-101-2/+1
| | | | | | | | Not xure how my MR to try to do this everyone missed this one.
| * tor-netdoc: util: Add standard lint block to testsIan Jackson2023-02-101-0/+10
| | | | | | | | Not xure how my MR to try to do this everyone missed this one.
* | Merge branch 'test_hsdesc_decrypt' into 'main'Nick Mathewson2023-02-133-8/+294
|\ \ | |/ |/| | | | | hsdesc: Add a test case for descriptors with ClientDescEncKey encryption. See merge request tpo/core/arti!1015
| * netdoc: give hsdesc test info in its original form.Nick Mathewson2023-02-131-1/+5
| |
| * netdoc: Add a test for HSDesc decryption.Nick Mathewson2023-02-101-0/+63
| |
| * Add another test hsdesc to parse.Nick Mathewson2023-02-101-0/+223
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This one was generated (by dgoulet) using `ClientDescEncKey` encryption. Its information is: ``` Address: paozpdhgz2okvc6kgbxvh2bnfsmt4xergrtcl4obkhopyvwxkpjzvoad.onion Time period: 19397 Client: paozpdhgz2okvc6kgbxvh2bnfsmt4xergrtcl4obkhopyvwxkpjzvoad:descriptor:x25519:SDZNMD4RP4SCH4EYTTUZPFRZINNFWAOPPKZ6BINZAC7LREV24RBQ Service: descriptor:x25519:SACGOAEODFGCYY22NYZV45ZESFPFLDGLMBWFACKEO34XGHASSAMQ ```
| * hscrypto: Use minutes, not seconds, in blinding parameter.Nick Mathewson2023-02-101-8/+4
|/ | | | | | | | | | | | Fixes an instance of #768. The problem with the test vectors is that I used a random time period number (1234) and the default-in-tor period length (1440) without checking whether 1440 _meant_ minutes or seconds. I'll add another test to Tor to make sure that the time period matches now. With this change, I can test Tor-generated hsdescs with encryption, so I'm fairly confident that the new behavior is correct.
* Merge branch 'ticket_525_part2' into 'main'Nick Mathewson2023-02-0930-343/+376
|\ | | | | | | | | Apply restricted_msg to ChanMsg parts of tor-proto See merge request tpo/core/arti!1013
| * Rename OpenClientChan{Msg,Cell} => OpenChan{Msg,Cell}S2CNick Mathewson2023-02-093-31/+31
| |
| * tor-proto: Do not parse forbidden commands on inbound cells.Nick Mathewson2023-02-093-88/+63
| | | | | | | | | | | | | | | | | | | | Unlike C tor, we treat unrecognized commands as reason to kill off the connection entirely. That's fine; if we need to add an unrecognized command in the future, we can use VERSIONS to negotiate it. Also, if someday we want this code to support relay channels as well, we can use some type trickery to have that work too.
| * tor-proto: only parse allowed ChanMsg types during handshake.Nick Mathewson2023-02-093-29/+100
| |
| * tor-cell: Return a useful error when restrist_msg rejects a command.Nick Mathewson2023-02-091-2/+3
| |
| * Change tor_bytes::Error::BadMessage to a Cow.Nick Mathewson2023-02-0918-52/+87
| | | | | | | | | | | | | | | | | | | | Actually, to avoid making a breaking change, I'm deprecating BadMessage and creating a new InvalidMessage variant that takes a Cow. This way I don't need to track every crate that re-exposes tor_bytes::Error and call this a breaking change in those. Making this change will allow tor_bytes errors to be much more helpful.
| * tor-cell: Re-export paste so restrict_msg can work outside the crate.Nick Mathewson2023-02-091-3/+3
| |
| * tor-cell: Remove chancell::msg::Body::into_message.Nick Mathewson2023-02-091-52/+1
| | | | | | | | | | It's now redundant, since `restricted_msg!` defines From and Into for us.
| * tor-cell: Make RelayEarly a separate type.Nick Mathewson2023-02-097-12/+37
| | | | | | | | This allows us to remove a shenanigan from `restricted_msg!{}`.
| * tor-cell: Have restrict_msg add conversion functions.Nick Mathewson2023-02-096-104/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Every FooMsg type now implements Into<AnyFooMsg>, and TryFrom<FooMsg>. Additionally, it now implements From<X> for every distinct type that it supports. This last part lets us discard a bunch of code. Unfortunately, I needed some downright hackish trickery in order to get these macros to avoid generating `From<AnyFooMsg> for AnyFooMsg` and conflicting with the blanket implementation. The trickery to deal with RelayEarly and Relay being the same type was not necessarily worth it; I will be separating them and removing said trickery in the next commit.
| * cell: Parameterize ChannelCodec::decode and encode.Nick Mathewson2023-02-093-12/+21
| | | | | | | | | | | | | | This change lets us use ChannelCodec to encode and decode any restricted channel message type we want. (Later on, we'll turn the related Codec class in tor-proto into a more type-restricted version of this.)
* | Merge branch 'hsdir' into 'main'Ian Jackson2023-02-0914-126/+375
|\ \ | | | | | | | | | | | | Compute the HS directory hash ring See merge request tpo/core/arti!1012
| * | Fix minimal-versions (.then_some is too new)Ian Jackson2023-02-091-1/+1
| | |
| * | tor-netdir: Make is_hsdir_for_ring crate-localIan Jackson2023-02-091-1/+2
| | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1012#note_2876824
| * | tor-netdir: Document fill_from_previous_netdir semverIan Jackson2023-02-091-0/+1
| | |
| * | Clarify which netdir index is meantNick Mathewson2023-02-091-1/+2
| | |
| * | Fix typosNick Mathewson2023-02-091-2/+2
| | |
| * | Fix compilation with various feature combinationsIan Jackson2023-02-092-6/+26
| | |
| * | hsdir: rustfmtIan Jackson2023-02-092-10/+6
| | | | | | | | | | | | | | | Deferred formatting changes which would have obscured the commits they were prompted by.
| * | hsdir: Actually compute the hash ringIan Jackson2023-02-092-8/+58
| | |
| * | hsdir: Be able to enumerate the hidden service directoriesIan Jackson2023-02-092-0/+29
| | | | | | | | | | | | | | | | | | * Provide an accessor for the HSDIR flag * Provide a function for testing a relay for hsdir inclusion * Provide an iterator on NetDir that returns the hsdirs
| * | hsdir: Ring calculation plumbingIan Jackson2023-02-092-17/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Implement Netdir::compute_rings in terms of a new HsDirRing::compute, that currently does nothing. * Actually call Netdir::compute_rings (since now it doesn't panic). * Make Netdir::compute_rings not be pub. We do this unconditionally, rather than exposing the distinction between a netdir-without-hsdir and a netdir-with-hsdir.
| * | hsdir representation: Introduce HsDirs generic typeIan Jackson2023-02-094-38/+63
| | | | | | | | | | | | | | | This already allows us to get rid of some duplication, and will be useful more widely in a moment.
| * | hsdir: Implement HS hash ring index calculationsIan Jackson2023-02-092-7/+80
| | |
| * | hsdir: Make HsDirIndex be AsRef<[u8; ]>Ian Jackson2023-02-091-2/+4
| | |
| * | hsdir: impl Eq for HsDirParamsIan Jackson2023-02-091-1/+1
| | | | | | | | | | | | This will allow us to tell if we can reuse parts of a ring.
| * | hsdir: Rename HsRingParams to HsDirParamsIan Jackson2023-02-093-17/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | The file which contains this type is called hsdir_params.rs. We have a general problem with slight confusion about when to includen "dir" and when to include "ring". Resolve this in favour of the rule now added to the module-level doc comment.
| * | hsdir_params: Rename some variables for consistencyIan Jackson2023-02-092-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These variables are going to be struct fields, which will sort of enforce consistent naming. The struct fields are going to appear in a moment. We'll call the fields "current" and "secondary" after the naming in the test cases. And import hsdir_params::HsRingParams, which we're going to make more references to.
| * | hsdir: Replace open-coded params fields and args with HsRingParamsIan Jackson2023-02-093-16/+10
| | | | | | | | | | | | | | | | | | This will enable us to pass these around more conveniently. And make empty_from_params take an owned params, which it would like.
| * | hsdir: Remove a done TODO - this *is* now a TiSliceIan Jackson2023-02-091-1/+0
| | |