| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | tor-config: small typo fix in docs | Steven Engler | 2026-01-07 | 1 | -1/+2 |
| | | |||||
| * | tor-config: New Buildable trait. | Nick Mathewson | 2025-12-09 | 1 | -2/+18 |
| | | | | | | | | | This trait will be implemented by every type that our derive_deftly(TorConfig) template generates. It will, among other things, help us figure out the Builder type for a given config type in cases where string-pasting magic is intractable, or where we want to use assert_not_impl to double-check the attributes. | ||||
| * | Fix name of clippy lint to unchecked_time_subtraction (2) | Ian Jackson | 2025-11-06 | 1 | -1/+1 |
| | | | | | Run maint/add_warning | ||||
| * | Switch Cargo.toml files to edition 2024. | Nick Mathewson | 2025-08-07 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | First, run ``` git grep -l "^edition =" | xargs perl -i -pe 's/^edition *=.*/edition = "2024"/;' ``` Second, manually verify that all Cargo.toml files have changed, and nothing else has changed. Third, run cargo fmt again. | ||||
| * | tor-config: fix `clippy::double_ended_iterator_last` | Steven Engler | 2025-04-03 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | ```text warning: called `Iterator::last` on a `DoubleEndedIterator`; this will needlessly iterate the entire iterator --> crates/tor-config/src/load.rs:606:18 | 606 | .last() | ^^^^^^ help: try: `next_back()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#double_ended_iterator_last note: the lint level is defined here --> crates/tor-config/src/lib.rs:9:9 | 9 | #![warn(clippy::all)] | ^^^^^^^^^^^ = note: `#[warn(clippy::double_ended_iterator_last)]` implied by `#[warn(clippy::all)]` ``` | ||||
| * | Switch to figment 1.0.18 and extract_lossy | Nick Mathewson | 2024-04-22 | 1 | -2/+2 |
| | | | | | | | | The new (dubiously named) `Figment::extract_lossy` method can treat decode numbers from strings, and booleans from numbers or strings. This makes the behavior compatible with config-rs, and should (I hope) finally allow us to migrate. | ||||
| * | Convert tor-config to use figment. | Nick Mathewson | 2024-04-22 | 1 | -20/+53 |
| | | | | | | There are probably ways to make this a bit more elegant, but at least the tests still pass. | ||||
| * | Merge branch 'encapsulate_config_rs' into 'main' | Nick Mathewson | 2024-03-13 | 1 | -31/+31 |
| |\ | | | | | | | | | Encapsulate usage of config-rs inside tor-config. See merge request tpo/core/arti!2040 | ||||
| | * | tor-config: Update documentation to not refer to config crate. | Nick Mathewson | 2024-03-13 | 1 | -4/+4 |
| | | | |||||
| | * | config: Remove From impls from config::ConfigError | Nick Mathewson | 2024-03-13 | 1 | -7/+1 |
| | | | | | | | | | These are another case of exposing the config crate. | ||||
| | * | tor-config: Put config::Config inside a wrapper type. | Nick Mathewson | 2024-03-13 | 1 | -9/+9 |
| | | | | | | | | | This will let us replace Config with something better. | ||||
| | * | Use ConfigurationSource::Verbatim in our tests. | Nick Mathewson | 2024-03-13 | 1 | -11/+17 |
| | | | |||||
| * | | Run maint/add_warning. | Nick Mathewson | 2024-03-13 | 1 | -0/+1 |
| |/ | |||||
| * | Run maint/add_warning to add lint block everywhere | Ian Jackson | 2023-08-23 | 1 | -0/+1 |
| | | |||||
| * | Run maint/add_warning to actually apply new lint allows | Ian Jackson | 2023-07-10 | 1 | -0/+1 |
| | | |||||
| * | Allow clippy::unchecked_duration_subtraction in tests | Nick Mathewson | 2023-01-27 | 1 | -0/+1 |
| | | | | | | This panics on error, and we're fine with a panic on misbehavior in tests. | ||||
| * | test lint blocks: Do some semi-manually | Ian Jackson | 2022-12-12 | 1 | -1/+9 |
| | | | | | | | | This is the hunks from running the rune in maint/adhoc-add-lint-blocks but which require some subsequent manual fixup: usually, deleting now-superfluous outer allows, but in some cases manually putting back lints that the adhoc script deleted. | ||||
| * | Merge branch 'refactor-anyhow-arti-crate' into 'main' | Nick Mathewson | 2022-08-31 | 1 | -1/+1 |
| |\ | | | | | | | | | Remove `anyhow` from `tor-config` crate See merge request tpo/core/arti!707 | ||||
| | * | Remove `anyhow` from `tor-config` crate | Arturo Marquez | 2022-08-28 | 1 | -1/+1 |
| | | | |||||
| * | | tor-config: DisfavouredKey: Linkify Display | Ian Jackson | 2022-08-30 | 1 | -1/+1 |
| | | | |||||
| * | | tor-config: DisfavouredKey: Widen applicability of .to_string() | Ian Jackson | 2022-08-30 | 1 | -1/+1 |
| | | | | | | | | | And linkify it. | ||||
| * | | Add documentation note on `to_string()` usage for DisfavouredKey. | Alexander Færøy | 2022-08-30 | 1 | -0/+2 |
| |/ | |||||
| * | Improve error from bad escapes in a toml config. | Nick Mathewson | 2022-08-25 | 1 | -1/+7 |
| | | | | | | | | | | | | | | | | | | Whereas previously we would say: ``` target/debug/arti: error: invalid escape character in string: `Z` at line 9 column 14 in ../../.config/arti/arti.toml ``` we now say: ``` target/debug/arti: error: invalid escape character in string: `Z` at line 9 column 14 in ../../.config/arti/arti.toml (If you wanted to include a literal \ character, you need to escape it by writing two in a row: \\) ``` The implementation is a bit of a hack, I'm afraid, but I don't think it's all that bad. Closes #549. | ||||
| * | tor-config: Support tracking deprecated config keys | Ian Jackson | 2022-08-25 | 1 | -9/+68 |
| | | |||||
| * | tor-config: Introduce ResolutionResults | Ian Jackson | 2022-08-25 | 1 | -19/+41 |
| | | | | | This will allow us to handle new kinds of warnigns etc. | ||||
| * | tor-config: Rename UnrecognizedKey to DisfavouredKey | Ian Jackson | 2022-08-25 | 1 | -18/+18 |
| | | | | | We're going to want the to use the same type for deprecated keys. | ||||
| * | fix nighly clippy | trinity-1686a | 2022-07-23 | 1 | -1/+1 |
| | | |||||
| * | tor-config: Don't format source error as part of error message | Nick Mathewson | 2022-06-22 | 1 | -2/+2 |
| | | | | | This is one I missed earlier. | ||||
| * | Fix typos | Dimitris Apostolou | 2022-06-05 | 1 | -3/+3 |
| | | |||||
| * | Merge branch 'tor-config-tests' into 'main' | Nick Mathewson | 2022-05-31 | 1 | -0/+61 |
| |\ | | | | | | | | | Add a few coverage-based tests to tor-config. See merge request tpo/core/arti!540 | ||||
| | * | Add a few coverage-based tests to tor-config. | Nick Mathewson | 2022-05-31 | 1 | -0/+61 |
| | | | | | | | | | There's nothing major here, but it does fill in a few gaps. | ||||
| * | | Fix a link with nightly rustdoc | Nick Mathewson | 2022-05-26 | 1 | -1/+1 |
| |/ | |||||
| * | tor-config: load: clippy: adopt two suggestions | Ian Jackson | 2022-05-26 | 1 | -3/+2 |
| | | | | | We can have mem::take, hooray. | ||||
| * | tor-config: load: Add missing "not". | Ian Jackson | 2022-05-26 | 1 | -1/+1 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807331 | ||||
| * | tor-config: load: Make UnrecognizedKeys not pub | Ian Jackson | 2022-05-26 | 1 | -1/+1 |
| | | | | | | | | This was a slip. As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807330 | ||||
| * | tor-config: load: Add reversed test | Ian Jackson | 2022-05-25 | 1 | -0/+3 |
| | | |||||
| * | tor-config: Add duplicative doc comments as per review | Ian Jackson | 2022-05-25 | 1 | -1/+6 |
| | | |||||
| * | tor-config: load: example: Document what it does and who might write it | Ian Jackson | 2022-05-25 | 1 | -0/+16 |
| | | |||||
| * | tor-config: load: Add a newline | Ian Jackson | 2022-05-25 | 1 | -0/+1 |
| | | | | | This makes the function a tiny bit clearer. | ||||
| * | tor-config: load: Write actual definition of the Resolvable trait | Ian Jackson | 2022-05-25 | 1 | -2/+18 |
| | | | | | | This is not a doc comment because we don't want it to be public: it must refer to private fields, etc. | ||||
| * | tor-config: load: Explain why not write out macro call n times | Ian Jackson | 2022-05-25 | 1 | -0/+2 |
| | | |||||
| * | tor-config: docs: add more docs about load, esp. traits | Ian Jackson | 2022-05-25 | 1 | -0/+23 |
| | | |||||
| * | tor-config: docs: add a lot of context and overview and xrefs | Ian Jackson | 2022-05-25 | 1 | -0/+16 |
| | | |||||
| * | tor-config: load: Avoid destructuring assignment | Ian Jackson | 2022-05-25 | 1 | -2/+2 |
| | | | | | Not available in our MSRV. | ||||
| * | tor-config: load: Intersection: add more commentary | Ian Jackson | 2022-05-25 | 1 | -5/+48 |
| | | |||||
| * | tor-config: load: Drop a needless call to .into_iter() | Ian Jackson | 2022-05-25 | 1 | -1/+0 |
| | | | | | I think this is a leftover from a previous version of this expression. | ||||
| * | tor-config: load: Fix a misleading comment | Ian Jackson | 2022-05-25 | 1 | -1/+1 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807070 | ||||
| * | tor-config: load: Rename shorter_let | Ian Jackson | 2022-05-25 | 1 | -2/+2 |
| | | | | | | Instead of the wrong "prefix_len". As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807068 | ||||
| * | tor-config: load: Docs | Ian Jackson | 2022-05-25 | 1 | -2/+2 |
| | | |||||
| * | tor-config: Typo fixes (as per review comments) | Ian Jackson | 2022-05-25 | 1 | -4/+4 |
| | | |||||
