summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | tor-config: load: Write actual definition of the Resolvable traitIan Jackson2022-05-251-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | This is not a doc comment because we don't want it to be public: it must refer to private fields, etc.
| * | | | arti: Better doc for ArtiCombinedConfigIan Jackson2022-05-251-1/+3
| | | | |
| * | | | tor-config: load: Explain why not write out macro call n timesIan Jackson2022-05-251-0/+2
| | | | |
| * | | | tor-config: docs: add more docs about load, esp. traitsIan Jackson2022-05-251-0/+23
| | | | |
| * | | | tor-config: docs: add a lot of context and overview and xrefsIan Jackson2022-05-252-1/+57
| | | | |
| * | | | tor-config: load: Avoid destructuring assignmentIan Jackson2022-05-251-2/+2
| | | | | | | | | | | | | | | | | | | | Not available in our MSRV.
| * | | | tor-config: load: Intersection: add more commentaryIan Jackson2022-05-251-5/+48
| | | | |
| * | | | tor-config: load: Drop a needless call to .into_iter()Ian Jackson2022-05-251-1/+0
| | | | | | | | | | | | | | | | | | | | I think this is a leftover from a previous version of this expression.
| * | | | tor-config: load: Fix a misleading commentIan Jackson2022-05-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807070
| * | | | tor-config: load: Rename shorter_letIan Jackson2022-05-251-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: DocsIan Jackson2022-05-251-2/+2
| | | | |
| * | | | tor-config: Typo fixes (as per review comments)Ian Jackson2022-05-251-4/+4
| | | | |
| * | | | tor-config: load: Introduce UnrecognizedKeys enumIan Jackson2022-05-251-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 renamingIan Jackson2022-05-252-2/+3
| | | | | | | | | | | | | | | | | | | | Split into its own commit to avoid churn in the rename commits.
| * | | | tor-config: Rename resolve_return_unrecognized, ..._ignore_...Ian Jackson2022-05-253-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807077
| * | | | tor-config: Rename "ignored" to "unrecognized" throughoutIan Jackson2022-05-253-48/+48
| | | | | | | | | | | | | | | | | | | | | | | | | As per review comments https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807076
| * | | | semvar_status: Document config changesIan Jackson2022-05-241-0/+6
| | | | |
| * | | | tor-config: Example for load moduleIan Jackson2022-05-243-0/+48
| | | | |
| * | | | tor-config: Tests for ignored config key handlingIan Jackson2022-05-242-8/+60
| | | | |
| * | | | tor-config: Track and (by default) warn on ignored config keysIan Jackson2022-05-242-7/+103
| | | | |
| * | | | tor-config: Tests for support functions for tracking ignored keysIan Jackson2022-05-241-0/+80
| | | | |
| * | | | tor-config: Support functions for tracking ignored config keysIan Jackson2022-05-243-0/+181
| | | | | | | | | | | | | | | | | | | | This turns out to need quite a complicated algorithm.
| * | | | Split TorClientConfig out of ArtiConfig, and Resolvable traitIan Jackson2022-05-248-51/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 ConfigReolveErrorIan Jackson2022-05-242-5/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | arti tests: Introduce bld_tor variableIan Jackson2022-05-241-22/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are going to reorganise ArtiConfig to not contain a TorClientConfig. This test case's calls to bld.tor() will all need to change. Do this in advance to make that future commit more readable.
| * | | | tor-config: Add a lint allowIan Jackson2022-05-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't understand why this isn't tripping all the time. Maybe because this is in a macro. Anyway, I am going to add a new invocation of this macro from within a test where, empirically, it trips.
* | | | | Merge branch 'no-disable-arti-permission-checks' into 'main'Nick Mathewson2022-05-253-7/+19
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | ci: don't disable arti permission checks, fix resulting issues See merge request tpo/core/arti!530
| * | | | fs-mistrust: allow symlinks to have any permissionsAlex Xu (Hello71)2022-05-252-1/+12
| | | | |
| * | | | ci: set umask instead of ARTI_FS_DISABLE_PERMISSION_CHECKSAlex Xu (Hello71)2022-05-251-6/+7
| | | | |
* | | | | Merge branch 'no_nochange' into 'main'Ian Jackson2022-05-253-72/+25
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | DirMgr:: Remove Error::NoChange as redundant. Closes #484 See merge request tpo/core/arti!532
| * | | | DirMgr:: Remove Error::NoChange as redundant.Nick Mathewson2022-05-253-72/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that the relevant functions now report changed/not-changed status via a boolean out-parameter (see !527), there's no reason to have a separate NoChanged error case. Closes #484.
* | | | | Merge branch 'bug475' into 'main'Ian Jackson2022-05-251-1/+6
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ConfigurationSources: Allow config files to be world-readable. Closes #475 See merge request tpo/core/arti!528
| * | | | | ConfigurationSources: Allow config files to be world-readable.Nick Mathewson2022-05-241-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | Fixes #475.
* | | | | | Merge branch 'support_policy' into 'main'Nick Mathewson2022-05-251-0/+172
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Initial draft policy on supported platforms and dependencies Closes #379 See merge request tpo/core/arti!513
| * | | | | | Apply clarifications to SupportPolicy.md from @DizietIan Jackson2022-05-241-3/+3
| | | | | | |
| * | | | | | Clarify which releases get advisories.Nick Mathewson2022-05-201-0/+2
| | | | | | |
| * | | | | | SupportPolicy: Add tiers, clarify what "support" means.Nick Mathewson2022-05-201-18/+102
| | | | | | |
| * | | | | | Initial draft policy on supported platforms and dependenciesNick Mathewson2022-05-181-0/+86
| | | | | | |
* | | | | | | Merge branch 'socks-errors' into 'main'Nick Mathewson2022-05-251-48/+67
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | reply socks error on more codepath Closes #258 See merge request tpo/core/arti!531
| * | | | | | reply socks error on more codepathtrinity-1686a2022-05-251-48/+67
| | |_|_|/ / | |/| | | |
* / | | | | Typo fixes (automated with "typos").Nick Mathewson2022-05-242-5/+5
|/ / / / /
* | | | | Merge branch 'isolation-example' into 'main'Nick Mathewson2022-05-241-0/+198
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add example for Isolation Closes #414 See merge request tpo/core/arti!524
| * | | | | mention isolation is an advanced topic and most usage don't require ↵trinity-1686a2022-05-241-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | implementing the trait
| * | | | | add example for Isolationtrinity-1686a2022-05-231-0/+195
| | | | | |
* | | | | | Merge branch 'bug482' into 'main'eta2022-05-242-29/+133
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DirMgr: Stop load-from-cache process when there is no change. Closes #482 See merge request tpo/core/arti!527
| * | | | | | DirMgr: Stop load-from-cache process when there is no change.Nick Mathewson2022-05-242-29/+133
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously in !511 I had introduced a bug where, if there was an error more serious than "no change", that error would keep us from noticing that we had no change, and we'd loop until the safety counter ran out. Then we'd panic. This commit fixes the bug by reintroducing the `changed` boolean -- this time as an outparam for the add_from_* methods. Fixes #482.
* | | | | | Allow use of RetainMut, since we do not require Rust 1.61 yet.Nick Mathewson2022-05-241-0/+2
| | | | | |
* | | | | | Run "cargo update" in preparation for the next release.Nick Mathewson2022-05-241-62/+48
| | | | | |
* | | | | | Start work on a changelog for 0.4.0Nick Mathewson2022-05-241-0/+81
| |_|_|/ / |/| | | |
* | | | | Merge branch 'configurable-mistrust' into 'main'Nick Mathewson2022-05-2422-228/+887
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | Make file permissions configurable via regular config mechanisms Closes #465 See merge request tpo/core/arti!515