| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Fix a rustdoc link error. | Nick Mathewson | 2022-08-31 | 1 | -1/+1 |
| | | |||||
| * | Merge branch 'refactor-anyhow-arti-crate' into 'main' | Nick Mathewson | 2022-08-31 | 2 | -3/+4 |
| |\ | | | | | | | | | Remove `anyhow` from `tor-config` crate See merge request tpo/core/arti!707 | ||||
| | * | Remove `anyhow` from `tor-config` crate | Arturo Marquez | 2022-08-28 | 2 | -3/+4 |
| | | | |||||
| * | | 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 |
| |/ | |||||
| * | Merge branch 'clippy' into 'main' | Nick Mathewson | 2022-08-26 | 1 | -0/+8 |
| |\ | | | | | | | | | tor-config: tests: Apply standard lint block in sources.rs See merge request tpo/core/arti!694 | ||||
| | * | tor-config: tests: Apply standard lint block in sources.rs | Ian Jackson | 2022-08-25 | 1 | -0/+8 |
| | | | | | | | | | Fixes a spurious clippy warning on nightly, about a dbg! | ||||
| * | | Improve error from bad escapes in a toml config. | Nick Mathewson | 2022-08-25 | 4 | -16/+62 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 Listen: Rename localhost_port_legacy (from _deprecated) | Ian Jackson | 2022-08-25 | 1 | -2/+2 |
| | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/602#note_2830847 | ||||
| * | | tor-config Listen: Add a note about EADDRINUSE | Ian Jackson | 2022-08-25 | 1 | -0/+1 |
| | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/602#note_2830848 | ||||
| * | | tor-config; Listen: Return addresses in groups for error behaviour | Ian Jackson | 2022-08-25 | 1 | -8/+20 |
| | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/602#note_2830766 | ||||
| * | | tor-config: Provide resolve_alternative_specs | Ian Jackson | 2022-08-25 | 1 | -0/+78 |
| | | | |||||
| * | | tor-config: Support tracking deprecated config keys | Ian Jackson | 2022-08-25 | 1 | -9/+68 |
| | | | |||||
| * | | tor-config: Introduce ResolutionResults | Ian Jackson | 2022-08-25 | 3 | -21/+49 |
| | | | | | | | | | 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. | ||||
| * | | tor-config: Provide misc::Listen | Ian Jackson | 2022-08-25 | 1 | -0/+310 |
| | | | |||||
| * | | tor-config misc tests: Add standard lint suppression block | Ian Jackson | 2022-08-25 | 1 | -0/+8 |
| |/ | |||||
| * | tor-config source: just ConfigurationSource, not FoundConfigFile | Ian Jackson | 2022-08-25 | 1 | -60/+21 |
| | | | | | | FoundConfigFile existed to hide something that ConfigurationSource now exposes. | ||||
| * | tor-config: Replace dir detection with ConfigurationSource enum | Ian Jackson | 2022-08-25 | 2 | -31/+81 |
| | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/682#note_2830860 And subsequent IRC discussion. Having done the work as per review comments, I don't much like the result. It's quite un-ergonomiuc. If we can't have fs autodetection, I think syntactic autodetection within sources.rs would be nearly as nice. However, I seem to be outvoted. At least the externally visible functionality (of an arti binary, say) is reasonably ergonomic. | ||||
| * | tor-config: Provide is_syntactically_directory helper function | Ian Jackson | 2022-08-25 | 1 | -0/+48 |
| | | |||||
| * | tor-config: MustRead: Make public | Ian Jackson | 2022-08-25 | 1 | -1/+2 |
| | | | | | I think this ought to be exhaustive. | ||||
| * | config sources: Read arti.d as well as arti.toml | Ian Jackson | 2022-08-25 | 1 | -8/+10 |
| | | | | | Fixes #474 aka #271 | ||||
| * | config sources tests: Test results of directory scan | Ian Jackson | 2022-08-25 | 1 | -1/+12 |
| | | |||||
| * | config sources tests: Break out sources_nodefaults | Ian Jackson | 2022-08-25 | 1 | -5/+12 |
| | | |||||
| * | config sources tests: Introduce test of reading directory | Ian Jackson | 2022-08-25 | 1 | -0/+29 |
| | | |||||
| * | config sources: Supporting reading directories | Ian Jackson | 2022-08-25 | 1 | -12/+106 |
| | | |||||
| * | config sources: Introduce scan() and FoundConfigFiles | Ian Jackson | 2022-08-25 | 1 | -46/+135 |
| | | | | | | | | | | | | | | | We're going to need to do config file reading in two phases. Right now this isn't actually necessary, because the set of files is fixed since we don't support dynamically scanning directories. But the new API will be needed in a moment. Code motion and API changes, but no overall functional change. Review with `git show -b` may be helpful. The new API also provides for dealing with directories, but right now that doesn't happen. | ||||
| * | tor-config sources: Remove some unneeded .to_string() from tests | Ian Jackson | 2022-08-25 | 1 | -4/+4 |
| | | |||||
| * | enable doc_auto_cfg feature on every crate when documenting for docs.rs | trinity-1686a | 2022-08-24 | 1 | -0/+1 |
| | | |||||
| * | tor-config: resolve_option_general: Add TODO about example | Ian Jackson | 2022-08-23 | 1 | -0/+5 |
| | | |||||
| * | Fix syntax in doc comment. | Nick Mathewson | 2022-08-23 | 1 | -1/+1 |
| | | |||||
| * | tor-config: Provide resolve_option_general, for T: !Default etc. | Ian Jackson | 2022-08-22 | 1 | -4/+43 |
| | | | | | | | | | | At one point in this MR I thought I was going to want this for arti::cfg::ListenConfig (which we don't want to be Default). In fact ListenConfig is being handled specially, but having written this function it seemed sensible to keep it. Since resolve_option becomes a wrapper for it, the existing tests exercise it. | ||||
| * | Fix typos in comments. | eta | 2022-08-17 | 1 | -1/+1 |
| | | |||||
| * | tor-config: Introduce PaddingLevel | Ian Jackson | 2022-08-16 | 2 | -0/+114 |
| | | | | | This will be used for controlling channel padding, for now. | ||||
| * | tor-config: Introduce ReconfigureError::Bug | Ian Jackson | 2022-08-16 | 1 | -0/+4 |
| | | | | | Reconfigurations might fail due to internal errors. | ||||
| * | fix nighly clippy | trinity-1686a | 2022-07-23 | 2 | -5/+5 |
| | | |||||
| * | clippy: Consolidate many lints in maint/add_warning | Ian Jackson | 2022-06-24 | 1 | -0/+9 |
| | | | | | | | Found these by disabling the nightly dbg macro special case. Now, we have a mechanism for globally adding suppressions to tests, we can use that instead. | ||||
| * | Run maint/add_warning crates/*/src/{lib,main}.rs | Ian Jackson | 2022-06-23 | 1 | -0/+3 |
| | | | | | Update all lint blocks | ||||
| * | tor-config: style fixes on CfgPathError. | Nick Mathewson | 2022-06-22 | 1 | -6/+6 |
| | | |||||
| * | 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. | ||||
| * | impl_standard_builder: Allow for !Default | Ian Jackson | 2022-06-16 | 1 | -13/+54 |
| | | |||||
| * | tor-config: impl_standard_builder: handle contexts with local Result | Ian Jackson | 2022-06-16 | 1 | -1/+1 |
| | | |||||
| * | config: Suppose that we might extend resolve_option to non-T::Default | Ian Jackson | 2022-06-10 | 1 | -0/+2 |
| | | | | | | As per point 3 in https://gitlab.torproject.org/tpo/core/arti/-/issues/488 | ||||
| * | config: Do not strip_option for journald (and in future) | Ian Jackson | 2022-06-10 | 1 | -0/+3 |
| | | | | | | As per point 1 in https://gitlab.torproject.org/tpo/core/arti/-/issues/488 | ||||
| * | Fix typos | Dimitris Apostolou | 2022-06-05 | 2 | -5/+5 |
| | | |||||
| * | tor-config: Fix a doc link | Ian Jackson | 2022-06-01 | 1 | -1/+1 |
| | | | | | Nightly cargo doc complaints about this. | ||||
| * | Merge branch 'lint' into 'main' | Ian Jackson | 2022-05-31 | 1 | -0/+3 |
| |\ | | | | | | | | | | | | | lints: Make lint blocks consistent and ensure they stay that way Closes #469 See merge request tpo/core/arti!557 | ||||
| | * | lints: Add let_unit_value allow to all crates | Ian Jackson | 2022-05-31 | 1 | -0/+1 |
| | | | | | | | | | | | From running add_warning, with manual picking of the right hunks/lines. | ||||
| | * | lints: Add lint block delimiters to every crate | Ian Jackson | 2022-05-31 | 1 | -0/+2 |
| | | | | | | | | | | | | | This was the result of: maint/add_warning crates/*/src/{lib,main}.rs and then manually curating the results. | ||||
