| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | | | Add new entries to changelog. | Nick Mathewson | 2022-05-26 | 1 | -2/+15 | |
| |/ / / | ||||||
| * | | | Merge branch 'tests-20220525' into 'main' | Nick Mathewson | 2022-05-26 | 3 | -4/+150 | |
| |\ \ \ | |/ / |/| | | | | | | | | Add a few coverage-driven tests See merge request tpo/core/arti!536 | |||||
| | * | | Remove some stray dbg!() calls. | Nick Mathewson | 2022-05-26 | 1 | -2/+1 | |
| | | | | ||||||
| | * | | Test a few more corner cases in tor-persist | Nick Mathewson | 2022-05-25 | 2 | -4/+111 | |
| | | | | | | | | | | | | | | | | Also, if we get an IoError from a call to fs-mistrust, report it as an IoError. | |||||
| | * | | More tests for ClockSkew. | Nick Mathewson | 2022-05-25 | 1 | -0/+40 | |
| | | | | ||||||
| * | | | Merge branch 'link-to-coverage' into 'main' | Nick Mathewson | 2022-05-26 | 1 | -1/+2 | |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | add link to coverage reports Closes #251 See merge request tpo/core/arti!537 | |||||
| | * | | | add link to coverage reports | trinity-1686a | 2022-05-26 | 1 | -1/+2 | |
| | | | | | ||||||
| * | | | | Merge branch 'standardise-shell' into 'main' | Nick Mathewson | 2022-05-26 | 21 | -61/+76 | |
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Standardise shell Closes #425 See merge request tpo/core/arti!533 | |||||
| | * | | | | fix docker_reproducible_build not passing its parameters properly | trinity-1686a | 2022-05-26 | 1 | -1/+1 | |
| | | | | | | ||||||
| | * | | | | update comments in coverage_fuzz_corpora | trinity-1686a | 2022-05-25 | 1 | -25/+15 | |
| | | | | | | ||||||
| | * | | | | update readmes | trinity-1686a | 2022-05-25 | 3 | -8/+20 | |
| | | | | | | ||||||
| | * | | | | make almost all scripts use bash and set euo pipefail | trinity-1686a | 2022-05-25 | 18 | -29/+42 | |
| | |/ / / | ||||||
| * | | | | Merge branch 'config-split' into 'main' | Ian Jackson | 2022-05-26 | 11 | -85/+862 | |
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Break TorClientConfig out of ArtiConfig and warn on unknown config keys Closes #459 and #417 See merge request tpo/core/arti!529 | |||||
| | * | | | | 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. | |||||
| | * | | | | arti: Better doc for ArtiCombinedConfig | Ian Jackson | 2022-05-25 | 1 | -1/+3 | |
| | | | | | | ||||||
| | * | | | | 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 | 2 | -1/+57 | |
| | | | | | | ||||||
| | * | | | | 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 | 2 | -2/+3 | |
| | | | | | | | | | | | | | | | | | | | | | Split into its own commit to avoid churn in the rename commits. | |||||
| | * | | | | tor-config: Rename resolve_return_unrecognized, ..._ignore_... | Ian Jackson | 2022-05-25 | 3 | -6/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 3 | -48/+48 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | As per review comments https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/529#note_2807076 | |||||
| | * | | | | semvar_status: Document config changes | Ian Jackson | 2022-05-24 | 1 | -0/+6 | |
| | | | | | | ||||||
| | * | | | | tor-config: Example for load module | Ian Jackson | 2022-05-24 | 3 | -0/+48 | |
| | | | | | | ||||||
| | * | | | | tor-config: Tests for ignored config key handling | Ian Jackson | 2022-05-24 | 2 | -8/+60 | |
| | | | | | | ||||||
| | * | | | | tor-config: Track and (by default) warn on ignored config keys | Ian Jackson | 2022-05-24 | 2 | -7/+103 | |
| | | | | | | ||||||
| | * | | | | 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 | 3 | -0/+181 | |
| | | | | | | | | | | | | | | | | | | | | | This turns out to need quite a complicated algorithm. | |||||
| | * | | | | Split TorClientConfig out of ArtiConfig, and Resolvable trait | Ian Jackson | 2022-05-24 | 8 | -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 ConfigReolveError | Ian Jackson | 2022-05-24 | 2 | -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 variable | Ian Jackson | 2022-05-24 | 1 | -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 allow | Ian Jackson | 2022-05-24 | 1 | -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 Mathewson | 2022-05-25 | 3 | -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 permissions | Alex Xu (Hello71) | 2022-05-25 | 2 | -1/+12 | |
| | | | | | | ||||||
| | * | | | | ci: set umask instead of ARTI_FS_DISABLE_PERMISSION_CHECKS | Alex Xu (Hello71) | 2022-05-25 | 1 | -6/+7 | |
| | | | | | | ||||||
| * | | | | | Merge branch 'no_nochange' into 'main' | Ian Jackson | 2022-05-25 | 3 | -72/+25 | |
| |\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | DirMgr:: Remove Error::NoChange as redundant. Closes #484 See merge request tpo/core/arti!532 | |||||
 |
index : mirrors/arti.git | |
| mirror of https://gitlab.torproject.org/tpo/core/arti |
| summaryrefslogtreecommitdiff |