| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Merge branch 'encapsulate_config_rs' into 'main' | Nick Mathewson | 2024-03-13 | 6 | -87/+125 |
| |\ | | | | | | | | | Encapsulate usage of config-rs inside tor-config. See merge request tpo/core/arti!2040 | ||||
| | * | Grammar fix. | gabi-250 | 2024-03-13 | 1 | -1/+1 |
| | | | |||||
| | * | tor-config: Update documentation to not refer to config crate. | Nick Mathewson | 2024-03-13 | 3 | -10/+11 |
| | | | |||||
| | * | config: Remove From impls from config::ConfigError | Nick Mathewson | 2024-03-13 | 4 | -18/+18 |
| | | | | | | | | | These are another case of exposing the config crate. | ||||
| | * | tor-config: Put config::Config inside a wrapper type. | Nick Mathewson | 2024-03-13 | 5 | -23/+33 |
| | | | | | | | | | This will let us replace Config with something better. | ||||
| | * | Use ConfigurationSource::Verbatim in our tests. | Nick Mathewson | 2024-03-13 | 2 | -16/+24 |
| | | | |||||
| | * | tor-config: Allow ConfigurationSource to be verbatim text. | Nick Mathewson | 2024-03-13 | 1 | -12/+26 |
| | | | | | | | | | | | This will let us test our configuration logic without having to use the `config` crate directly. | ||||
| | * | tor_config: Hide macro-only re-exports. | Nick Mathewson | 2024-03-13 | 2 | -12/+17 |
| | | | | | | | | | | | Our macros use these exports, but we don't need everybody else to be able to see them. | ||||
| * | | Run maint/add_warning. | Nick Mathewson | 2024-03-13 | 10 | -0/+10 |
| |/ | |||||
| * | Fix typos in doc comments | Tobias Stoeckmann | 2024-03-06 | 1 | -1/+1 |
| | | |||||
| * | deny clippy::unchecked_duration_subtraction | trinity-1686a | 2024-02-29 | 1 | -0/+1 |
| | | |||||
| * | educe: Use std's default for two structs | Ian Jackson | 2024-02-12 | 1 | -3/+1 |
| | | | | | | | Reviewing uses of `#[educe(default)]`, I came across these two places where it was applied to a non-generic struct without any special attributes on fields. std's derive will do just as well here. | ||||
| * | educe: Use std's default for enums where default variant is unit | Ian Jackson | 2024-02-12 | 2 | -10/+6 |
| | | | | | | | | | | | | | | Since Rust 1.66, std's default works properly for enums, provided that the default variant is a unit. Review all uses of `#[educe(default)]` on enums and replace them with std where possible, which is most of them. In 1.66 and later, std's `#[derive(Default)]` doesn't infer any generic bounds on the derived impl, where it's an enum - since the unit variant can always be constructed. So this change doesn't add any generic bounds and is not API-visible. | ||||
| * | config: Fix output of NoCompileTimeSupport::within | Nick Mathewson | 2024-01-16 | 1 | -1/+1 |
| | | | | | | Previously, within() would transform this error variant into Invalid. | ||||
| * | Fix typos | Dimitris Apostolou | 2024-01-08 | 1 | -1/+1 |
| | | |||||
| * | tor-config: use black_box in test. | Nick Mathewson | 2023-11-30 | 1 | -3/+1 |
| | | |||||
| * | Merge branch 'generic-listen' into 'main' | Nick Mathewson | 2023-10-16 | 1 | -1/+50 |
| |\ | | | | | | | | | arti, tor-config: Allow listening on generic addresses for SOCKS and DNS. See merge request tpo/core/arti!1613 | ||||
| | * | Test for Listen Display | Jani Monoses | 2023-09-26 | 1 | -0/+24 |
| | | | |||||
| | * | Do not writelns in Display, undo localhost_port_legacy changes. | Jani Monoses | 2023-09-25 | 1 | -4/+3 |
| | | | |||||
| | * | arti, tor-config: Allow listening on generic addresses for SOCKS and DNS. | Jani Monoses | 2023-09-22 | 1 | -3/+29 |
| | | | |||||
| * | | tor-config: derive Default for Flatten | Nick Mathewson | 2023-10-03 | 1 | -1/+1 |
| |/ | |||||
| * | tor-config: flatten: Remove a redundant allow | Ian Jackson | 2023-09-20 | 1 | -1/+0 |
| | | | | | At some earlier point of development this struct was pub. | ||||
| * | tor-config: flatten: Sort out names for field extractor | Ian Jackson | 2023-09-20 | 1 | -12/+12 |
| | | |||||
| * | tor-config: flatten: always use field list from serde | Ian Jackson | 2023-09-20 | 2 | -48/+17 |
| | | | | | | | | | This is a better trickery! Now we don't need to derive the list of field names at all. The test case becomes much simpler too, and no longer needs to compare field lists since we have only the serde field list. | ||||
| * | tor-config: flatten: Performance of flattenable_extract_fields | Ian Jackson | 2023-09-20 | 1 | -0/+31 |
| | | | | | | Add a performance note, and the manually-verified test case to support it. | ||||
| * | tor-config: flatten: Break out flattenable_extract_fields | Ian Jackson | 2023-09-20 | 2 | -25/+41 |
| | | | | | | | | | | This involves returning the field list in the `FlattenableTesterSuccess`. This leaves the names a little off; we'll update those at the end. Export this - it's going to replace the hidden pub test function. | ||||
| * | tor-config: flatten: Introduce FieldList type alias | Ian Jackson | 2023-09-20 | 1 | -6/+9 |
| | | | | | We're going to use this some more, even. | ||||
| * | tor_config: flatten: Abolish FlattenError | Ian Jackson | 2023-09-20 | 1 | -38/+9 |
| | | |||||
| * | tor_config: flatten tests: Remove the obsolete todo | Ian Jackson | 2023-09-20 | 1 | -1/+0 |
| | | |||||
| * | tor_config: Add a note about FlattenError | Ian Jackson | 2023-09-20 | 1 | -0/+8 |
| | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1605#note_2945202 | ||||
| * | tor_config: Flatten: Explain why the inner ref Flatten for ser | Ian Jackson | 2023-09-20 | 1 | -0/+5 |
| | | |||||
| * | tor_config: Flatten: Add missing tst annotation | Ian Jackson | 2023-09-20 | 1 | -0/+1 |
| | | |||||
| * | tor_config: Provide Flatten | Ian Jackson | 2023-09-20 | 2 | -0/+687 |
| | | | | | | This is an attempt to deal with the fact that `#[serde(flatten)]` is weirdly broken and doesn't work with `serde_ignored`. | ||||
| * | tor-config: Listen: Clarify localhost_port_legacy | Ian Jackson | 2023-09-12 | 1 | -1/+1 |
| | | | | | | The previous phrasing was ambiguous and set us up for bugs, see https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1582#note_2942240 | ||||
| * | tor-config: Listen: Fix a typo | Ian Jackson | 2023-09-12 | 1 | -2/+2 |
| | | |||||
| * | Run maint/add_warning to add lint block everywhere | Ian Jackson | 2023-08-23 | 9 | -0/+9 |
| | | |||||
| * | Resolve warnings about ambiguous/redundant doc links | Nick Mathewson | 2023-08-22 | 1 | -1/+1 |
| | | | | | | Nightly rustdoc now warns if you have a link that isn't necessary, and if you have a link that might refer to two different things. | ||||
| * | tor-config: Expand NoProjectDirs error message | Ian Jackson | 2023-08-08 | 1 | -1/+1 |
| | | |||||
| * | tor-config: Add ConfigurationSources::try_from_cmdline | Ian Jackson | 2023-08-07 | 1 | -7/+34 |
| | | | | | We'll use this in crates/arti in a moment. | ||||
| * | Run add_warnings on all files. | Nick Mathewson | 2023-08-04 | 1 | -2/+2 |
| | | |||||
| * | tor-config: Remove unused ItemOrBool helper. | Gabriela Moldovan | 2023-07-13 | 1 | -63/+0 |
| | | | | | | `ItemOrBool` is currently not used anywhere (it was previously used by the keymgr config). | ||||
| * | Run maint/add_warning to actually apply new lint allows | Ian Jackson | 2023-07-10 | 9 | -0/+10 |
| | | |||||
| * | Merge branch 'allow_missing_panics' into 'main' | Ian Jackson | 2023-07-07 | 1 | -1/+0 |
| |\ | | | | | | | | | | | | | add_warning: Tolerate clippy::missing_panics_doc Closes #950 See merge request tpo/core/arti!1380 | ||||
| | * | Run add_warning to remove `missing_panics_doc` deny. | Nick Mathewson | 2023-07-06 | 1 | -1/+0 |
| | | | | | | | | | Closes #950. | ||||
| * | | tor-config: Use next_back() instead of rev().next() | Nick Mathewson | 2023-07-03 | 1 | -1/+1 |
| |/ | | | | This is a new warning^Wsuggestion from clippy nightly. | ||||
| * | tor-config: Make ItemOrBool an experimental feature. | Gabriela Moldovan | 2023-06-29 | 1 | -0/+4 |
| | | |||||
| * | config: Add ItemOrBool helper for deserializing a bool or a value. | Gabriela Moldovan | 2023-06-29 | 1 | -0/+59 |
| | | | | | | | | | | | | | | | | | | | | | This will enable us to support the new `keystore_dir` field, which can be either a boolean indicating whether the keystore is disabled or enabled and initialized with the default keystore dir, or a string which points to a custom keystore directory (and implies the keystore is enabled): ``` # use this path, fail if compiled out # keystore = "/path/to/arti/keystore" # # use default path, fail if compiled out # keystore = true # # disable # keystore = false ``` Part of #891 | ||||
| * | lints: Run maint/add_warning to actually apply new lints | Ian Jackson | 2023-06-21 | 1 | -0/+2 |
| | | |||||
| * | Use bool::then_some() as appropriate | Nick Mathewson | 2023-04-11 | 1 | -1/+1 |
| | | | | | | | Now that we require a version of Rust that allows `b.then_some(v)`, clippy complains about our use of `b.then(|| v)`. | ||||
| * | Fix typos | Dimitris Apostolou | 2023-03-24 | 1 | -1/+1 |
| | | |||||
