| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Add cfg_attr allow(unused_imports) to two crates | Ian Jackson | 2023-10-31 | 1 | -0/+6 |
| | | | | | | As per this comment, and preceding discussion https://gitlab.torproject.org/tpo/core/arti/-/issues/1060#note_2959187 | ||||
| * | tor-dirclient, tor-netdoc: Add TODO SPECs. | Gabriela Moldovan | 2023-10-24 | 1 | -0/+1 |
| | | |||||
| * | tor-dirclient: Always set the Content-Length header (fmt). | Gabriela Moldovan | 2023-10-24 | 1 | -1/+4 |
| | | |||||
| * | tor-dirclient: Always set the Content-Length header. | Gabriela Moldovan | 2023-10-24 | 2 | -19/+12 |
| | | |||||
| * | tor-dirclient: Add a Content-Length header (fmt). | Gabriela Moldovan | 2023-10-24 | 1 | -1/+3 |
| | | |||||
| * | tor-dirclient: Add a Content-Length header. | Gabriela Moldovan | 2023-10-24 | 1 | -2/+16 |
| | | |||||
| * | dirclient: Seal the Requestable trait and hide most of its members. | Nick Mathewson | 2023-10-17 | 1 | -39/+67 |
| | | | | | | | | | Since none of these methods were invoked from outside `tor-dirclient` (except for debugging), and since we have had a fair amount of churn on what we actually want them to be, it seems like a good idea to use this trick to hide them. This will let us make other changes to the actual behavior of Requestable in the future. | ||||
| * | dirclient: Rename partial_docs_ok to partial_response_body_ok | Nick Mathewson | 2023-10-17 | 2 | -24/+23 |
| | | |||||
| * | dirclient: Make Requestable::anonymized mandatory. | Nick Mathewson | 2023-10-17 | 1 | -3/+21 |
| | | |||||
| * | dirclient: Explain why 1024 is a sufficient maximum response length. | Nick Mathewson | 2023-10-16 | 1 | -0/+4 |
| | | |||||
| * | dirclient: Wrap a somewhat wide comment. | Nick Mathewson | 2023-10-16 | 1 | -5/+3 |
| | | |||||
| * | Downgrade another TODO HSS and move it to TODO BREAKING | Nick Mathewson | 2023-10-16 | 1 | -2/+3 |
| | | |||||
| * | Privacy: Do not list supported encodings in hsdesc reqs. | Nick Mathewson | 2023-10-16 | 3 | -28/+87 |
| | | | | | | | | | | | | | | Since not everybody has xz and/or zstd, we don't want to admit whether we support them when we are uploading or downloading an onion service descriptor. Similarly, if we aren't advertising support for an encoding, we shouldn't accept it. Finally, while we're doing this, it made sense to have the ability to mark requests based on how anonymized they are, and reject (some) attempts to send those requests over a one-hop circuit. Closes #1062 | ||||
| * | dirclient: Change a todo to an API todo and make it less blocking. | Nick Mathewson | 2023-10-16 | 1 | -1/+9 |
| | | |||||
| * | tor-dirclient: Mark `DirResponse` as `#[must_use]`. | Gabriela Moldovan | 2023-10-03 | 1 | -0/+1 |
| | | | | | | Not checking whether the response is successful can have unintended consequences. | ||||
| * | tor-dirclient: Set a non-zero expected response len for uploads. | Gabriela Moldovan | 2023-10-03 | 1 | -5/+5 |
| | | |||||
| * | Run maint/add_warning to add lint block everywhere | Ian Jackson | 2023-08-23 | 4 | -0/+4 |
| | | |||||
| * | tor-dirclient: Fix clippy lints. | Gabriela Moldovan | 2023-08-16 | 1 | -5/+5 |
| | | |||||
| * | tor-dirclient: Remove unused `StringBody` trait. | Gabriela Moldovan | 2023-08-16 | 1 | -26/+0 |
| | | |||||
| * | tor-dirclient: Make Requestable return requests with String bodies. | Gabriela Moldovan | 2023-08-16 | 2 | -51/+23 |
| | | | | | | It's simpler to always use a `String` to represent directory request bodies. We no longer need the `StringBody` trait. | ||||
| * | tor-dirclient: Rename StringBody::str() to StringBody::as_str(). | Gabriela Moldovan | 2023-08-16 | 2 | -6/+6 |
| | | |||||
| * | tor-dirclient: Deprecate download() instead of removing it. | Gabriela Moldovan | 2023-08-16 | 1 | -0/+16 |
| | | |||||
| * | tor-dirclient: Extend format test to check the body is formatted too (fmt) | Gabriela Moldovan | 2023-08-16 | 1 | -1/+4 |
| | | |||||
| * | tor-dirclient: Extend format test to check the body is formatted too. | Gabriela Moldovan | 2023-08-16 | 1 | -8/+22 |
| | | |||||
| * | tor-dirclient: Move request building to a test helper function. | Gabriela Moldovan | 2023-08-16 | 1 | -11/+13 |
| | | |||||
| * | tor-dirclient: Add `HsDescUploadRequest`. | Gabriela Moldovan | 2023-08-16 | 1 | -0/+46 |
| | | | | | | The hsdir publisher will send the `HsDescUploadRequest`s to the appropriate directory using `send_request()`. | ||||
| * | tor-dirclient: Make the body type of a `Requestable` type configurable. | Gabriela Moldovan | 2023-08-16 | 2 | -3/+46 |
| | | | | | | | | | | Previously, the `Requestable` trait assumed the body of the request would always be empty (`http::Request<()>`). This change replaces the hardcoded `()` body type with the `Requestable::Body` associated type (which will allow implementors to create requests with non-empty bodies). This will enable us to reuse the `Requestable` trait for building `POST` requests for uploading descriptors. | ||||
| * | tor-dirclient: Rename download() to send_request(). | Gabriela Moldovan | 2023-08-16 | 2 | -6/+7 |
| | | | | | | | `download()` is actually a general-purpose function for sending HTTP requests on a stream. We will soon repurpose it for `POST`-ing descriptors, so let's rename it to `send_request`. | ||||
| * | Run add_warnings on all files. | Nick Mathewson | 2023-08-04 | 1 | -2/+2 |
| | | |||||
| * | Run maint/add_warning to actually apply new lint allows | Ian Jackson | 2023-07-10 | 4 | -0/+5 |
| | | |||||
| * | Run add_warning to remove `missing_panics_doc` deny. | Nick Mathewson | 2023-07-06 | 1 | -1/+0 |
| | | | | | Closes #950. | ||||
| * | dirclient: use correct default max_len | Nick Mathewson | 2023-06-27 | 1 | -2/+2 |
| | | | | | | | Yes, it's 50 kilobytes, not 50 kibibytes. I double-checked this with the C implementation and with param-spec.txt's documentation for `HSV3MaxDescriptorSize`. | ||||
| * | dirclient: Correct an erroneous comment. | Nick Mathewson | 2023-06-27 | 1 | -1/+2 |
| | | |||||
| * | dirclient: Make maximum hsdesc length adjustable. | Nick Mathewson | 2023-06-27 | 1 | -8/+12 |
| | | |||||
| * | dirclient: Explain why hardcoding "3" is ok. | Nick Mathewson | 2023-06-27 | 1 | -1/+7 |
| | | |||||
| * | lints: Run maint/add_warning to actually apply new lints | Ian Jackson | 2023-06-21 | 1 | -0/+2 |
| | | |||||
| * | Use an Enum for the description request | tranna | 2023-04-10 | 1 | -37/+54 |
| | | |||||
| * | dirclient: fix an "unused import" warning. | Nick Mathewson | 2023-04-06 | 1 | -1/+1 |
| | | |||||
| * | tor-dirclient: Add a comment about 50 x 1024 | gabi-250 | 2023-04-03 | 1 | -0/+1 |
| | | |||||
| * | tor-dirclient: Add test case for HsDescDownloadRequest | Ian Jackson | 2023-04-03 | 1 | -1/+21 |
| | | | | | | | | | I couldn't find a test vector in C Tor. This test case was generated from the code here. I'm fairly sure it's right since I managed to get my descriptor downloader to work. (That's not an MR yet, but uses this code.) | ||||
| * | tor-dirclient: Provide HsDescDownloadRequest | Ian Jackson | 2023-04-03 | 1 | -0/+43 |
| | | | | | | In my tests this seems to do the right thing, but I'm getting 404s. I'm not sure if actually this URL is wrong. | ||||
| * | tor-dirclient: De-genericise internal fn retire_circ | Ian Jackson | 2023-01-30 | 1 | -2/+1 |
| | | | | | | | | Requring `Display` is wrong here, because if this is actually an Error, Display would be wrong because it doesn't display causes. As it happens, the `error` parameter is only ever `&str`. | ||||
| * | Allow clippy::unchecked_duration_subtraction in tests | Nick Mathewson | 2023-01-27 | 4 | -0/+4 |
| | | | | | | This panics on error, and we're fine with a panic on misbehavior in tests. | ||||
| * | Disable clippy::unlinlined-format-args | Nick Mathewson | 2023-01-27 | 1 | -0/+1 |
| | | | | | | | | | This warning kind of snuck up on us! (See #748) For now, let's disable it. (I've cleaned it up in a couple of examples, since those are meant to be more idiomatic and user-facing.) Closes #748. | ||||
| * | Complete our migration to base64ct. | Nick Mathewson | 2023-01-20 | 1 | -1/+2 |
| | | | | | | | | | | This is in lieu of upgrading to the latest base64 crate, which has a different API from the old one. Since we have to migrate either way, we might as well use base64ct everywhere. I don't think that most of these cases _require_ constant-time base64, but it won't hurt. | ||||
| * | test lint blocks: Add many many automatically | Ian Jackson | 2022-12-12 | 3 | -0/+24 |
| | | | | | | This is precisely the result of running the rune in maint/adhoc-add-lint-blocks. | ||||
| * | tor-dirclient: Replace OwnedChanTarget with LoggedChanTarget in SourceInfo | Ian Jackson | 2022-11-23 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | | | SourceInfo primarily appears in errors, here and in (indirectly) tor-dirmgr. This 1. Makes the errors smaller 2. Redacts (currently, scrubs) the chantarget when safe logging is enabled This commit therefore deals with both tor-dirmgr and tor-dirclient errors, from the pov of error information that should become sensitive as part of bridge support. The error pyaloads directly in tor_dirmgr/src/err.rs don't seem to contain anything in that category. | ||||
| * | cfg compilation: Add two allows | Ian Jackson | 2022-11-18 | 1 | -0/+1 |
| | | |||||
| * | Suppress two clippy::large_enum_variant warnings | Nick Mathewson | 2022-11-03 | 1 | -0/+1 |
| | | | | | These are newly present on 1.65. We can address them later. | ||||
| * | Run add_warnings. | Nick Mathewson | 2022-11-03 | 1 | -0/+1 |
| | | |||||
