| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | 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 |
| | | |||||
| * | tor-config: load: Introduce UnrecognizedKeys enum | Ian Jackson | 2022-05-25 | 1 | -23/+52 |
| | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807078 This is in fact much clearer than the Option. | ||||
| * | Run rustfmt following renaming | Ian Jackson | 2022-05-25 | 1 | -1/+2 |
| | | | | | Split into its own commit to avoid churn in the rename commits. | ||||
| * | tor-config: Rename resolve_return_unrecognized, ..._ignore_... | Ian Jackson | 2022-05-25 | 1 | -4/+4 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807077 | ||||
| * | tor-config: Rename "ignored" to "unrecognized" throughout | Ian Jackson | 2022-05-25 | 1 | -44/+44 |
| | | | | | | As per review comments https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807076 | ||||
| * | tor-config: Example for load module | Ian Jackson | 2022-05-24 | 1 | -0/+46 |
| | | |||||
