| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | 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 | |
| | | ||||||
| * | tor-config: Tests for ignored config key handling | Ian Jackson | 2022-05-24 | 1 | -0/+56 | |
| | | ||||||
| * | tor-config: Track and (by default) warn on ignored config keys | Ian Jackson | 2022-05-24 | 1 | -6/+102 | |
| | | ||||||
| * | tor-config: Tests for support functions for tracking ignored keys | Ian Jackson | 2022-05-24 | 1 | -0/+80 | |
| | | ||||||
| * | tor-config: Support functions for tracking ignored config keys | Ian Jackson | 2022-05-24 | 1 | -0/+168 | |
| | | | | | This turns out to need quite a complicated algorithm. | |||||
| * | Split TorClientConfig out of ArtiConfig, and Resolvable trait | Ian Jackson | 2022-05-24 | 1 | -0/+85 | |
| | | | | | | | | | | | | | This gets rid of `#[serde(flatten)]` which prevents serde_ignored (and other kinds of introspection) from working properly. The price is now that the toplevel has to deal with two configuration objects. The Resolvable trait is overkill right now, but is going to do More Things in a moment. In particular, we need the impl on tuples, so that the whole config can be processed in one go. | |||||
| * | tor-config: Introduce Builder trait and ConfigReolveError | Ian Jackson | 2022-05-24 | 1 | -0/+29 | |
| We are going to need this for some generic code which is going to appear shortly. Having it produced by impl_standard_builder seems best. But that does mean being able to disable it, so extra stuff in the macro. Nothing uses this trait yet. ConfigResolveError is not used now either, but will be in a moment. | ||||||
