| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | cfg compilation: Add two allows | Ian Jackson | 2022-11-18 | 1 | -0/+1 | |
| | | ||||||
| * | Merge branch 'fix_nightly_20221118' into 'main' | Ian Jackson | 2022-11-18 | 1 | -3/+3 | |
| |\ | | | | | | | | | Fix warnings and breakage in nightly (2022-11-18 edition) See merge request tpo/core/arti!872 | |||||
| | * | Fix a bunch of "needless borrow" warnings on nightly | Nick Mathewson | 2022-11-18 | 1 | -3/+3 | |
| | | | | | | | | | | | It looks like, despite a few false starts, they've got this warning right; there weren't any false positives. | |||||
| * | | arti: Mark bridge and pt support non-experimental, and on-by-default | Nick Mathewson | 2022-11-18 | 2 | -4/+14 | |
| | | | ||||||
| * | | arti: Split long lines in Cargo.toml | Nick Mathewson | 2022-11-18 | 1 | -1/+10 | |
| |/ | | | | This will make the next commit easier to review. | |||||
| * | pt config, arti cfg test: Expect bridges.transports to be missing | Ian Jackson | 2022-11-16 | 1 | -2/+9 | |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/823#note_2854365 | |||||
| * | arti cfg test: Coalesce expect_missing | Ian Jackson | 2022-11-16 | 1 | -0/+22 | |
| | | | | | | | If there are subkeys which are covered by also expected_missing superkeys, delete them, since we expect the superkey to be missing, so the subkey won't show up. | |||||
| * | tor-ptmgr: make configuration use builders, plumb into arti-client | eta | 2022-11-16 | 1 | -0/+16 | |
| | | | | | | | | This brings the draft configuration mechanisms in tor-ptmgr in line with the config in other crates, using builders. It also plumbs the config type through into the main `arti-client` config, and adds some example lines to `arti-example-config.toml`. | |||||
| * | arti cfg test: Document exhaustive_1 and comment exhaustive | Ian Jackson | 2022-11-15 | 1 | -0/+39 | |
| | | ||||||
| * | arti cfg test: Break out CONFIG_KEYS_EXPECT_NO_EXAMPLE | Ian Jackson | 2022-11-15 | 1 | -2/+9 | |
| | | | | | Giving this a name will allow us to refer to it docs in a moment. | |||||
| * | arti cfg test: Drop a redundant allow | Ian Jackson | 2022-11-15 | 1 | -1/+0 | |
| | | | | | | This allow is also present in the standard lint block at the start of the test inline module. | |||||
| * | Fix Windows shortcut target | arnabanimesh | 2022-11-13 | 1 | -1/+1 | |
| | | ||||||
| * | bump rust-version to 1.60 in every crate. | Nick Mathewson | 2022-11-10 | 1 | -1/+1 | |
| | | ||||||
| * | Check more targets | trinity-1686a | 2022-11-10 | 2 | -2/+6 | |
| | | ||||||
| * | Merge branch 'clap-v3' into 'main' | Ian Jackson | 2022-11-08 | 2 | -35/+43 | |
| |\ | | | | | | | | | | | | | Upgrade to clap 3 Closes #616 See merge request tpo/core/arti!830 | |||||
| | * | Upgrade to clap 3: replace deprecated functions | Gabriel de Perthuis | 2022-11-04 | 2 | -29/+37 | |
| | | | | | | | | | | | | | | | | | | | | | | | Also replace Arg::takes_value with Arg::action to configure flags that don't take parameters vs options that do. This makes it easy to upgrade to clap 4. Tested deprecations with: cargo check --workspace --all-features --features clap/deprecated | |||||
| | * | Upgrade to clap 3 with minimal changes | Gabriel de Perthuis | 2022-11-04 | 2 | -7/+7 | |
| | | | | | | | | | | | | | | | | | This removes the last cargo audit override (for the unmaintained ansi_term). Don't mark options as required when they have default values: see <https://github.com/clap-rs/clap/pull/3793>. | |||||
| * | | Spelling fixes and normalizations on some high-level crates | Nick Mathewson | 2022-11-07 | 2 | -4/+4 | |
| | | | ||||||
| * | | Fix typos | Dimitris Apostolou | 2022-11-06 | 1 | -3/+3 | |
| |/ | ||||||
| * | Run add_warnings. | Nick Mathewson | 2022-11-03 | 2 | -0/+2 | |
| | | ||||||
| * | Merge branch 'notify' into 'main' | Ian Jackson | 2022-11-01 | 1 | -123/+137 | |
| |\ | | | | | | | | | | | | | refactor config reload triggering Closes #562 See merge request tpo/core/arti!819 | |||||
| | * | debounce reload cfg events | trinity-1686a | 2022-11-01 | 1 | -0/+7 | |
| | | | ||||||
| | * | rename unused field | trinity-1686a | 2022-10-30 | 1 | -3/+2 | |
| | | | ||||||
| | * | update some strings | trinity-1686a | 2022-10-30 | 1 | -10/+10 | |
| | | | ||||||
| | * | use iife so we can try! instead of custom error handling | trinity-1686a | 2022-10-30 | 1 | -64/+56 | |
| | | | ||||||
| | * | refactor so events always indicate reload | trinity-1686a | 2022-10-30 | 1 | -71/+74 | |
| | | | ||||||
| | * | depends less on notify types | trinity-1686a | 2022-10-30 | 1 | -22/+35 | |
| | | | ||||||
| * | | remove unused dependancies | trinity-1686a | 2022-10-29 | 1 | -3/+0 | |
| |/ | ||||||
| * | cargo fmt to remove blank lines | Ian Jackson | 2022-10-12 | 1 | -1/+0 | |
| | | | | | | | | Apparently cargo fmt doesn't like these, which my perl rune didn't delete. This commit is precisely the result of `cargo fmt`. | |||||
| * | Replace all README copies in src/lib.rs with includes | Ian Jackson | 2022-10-12 | 1 | -186/+1 | |
| | | | | | | | | | The feature we want is `#[doc = include_str!("README.md")]`, which is stable since 1.54 and our MSRV is now 1.56. This commit is precisely the result of the following Perl rune: perl -i~ -0777 -pe 's{(^//!(?!.*\@\@).*\n)+}{#![doc = include_str!("../README.md")]\n}m' crates/*/src/lib.rs | |||||
| * | README doctests: fix arti-client | Ian Jackson | 2022-10-12 | 1 | -5/+5 | |
| | | | | | Add ,ignore to ignore three shell runes. | |||||
| * | cfg tests: bridges: Document test case assumptions | Ian Jackson | 2022-10-12 | 1 | -0/+19 | |
| | | ||||||
| * | cfg tests: bridges: Remove now-redundant block { } | Ian Jackson | 2022-10-12 | 1 | -13/+11 | |
| | | ||||||
| * | cfg test: bridges: Test all three feature cases | Ian Jackson | 2022-10-12 | 1 | -4/+31 | |
| | | | | | | | | | | | | | | This demonstrates that: * !bridge-client: uncommenting nondefault bridge config generates urecognized config key warnings (but the config is still accepted)( * bridge-client, !pt-client: uncommenting nondefault bridges generates error due to attempting to use a PT. If that's filtered out, everything is fine. * pt-client: Everything is good (as before). | |||||
| * | cfg tests: bridges: Prepare for more comprehensive testing | Ian Jackson | 2022-10-12 | 1 | -8/+20 | |
| | | | | | | | | | | | | | | * Introduce filter_examples and resolve_examples helpers, which will become more complex in a moment. * Move the API test into a { } block to minimise subsequent diff. It's going to become conditional. * In subsequent comparisons, use the parsed version, since the API built one might not exist. No overall functional change. | |||||
| * | cfg tests: Make ExampleSectionLines::resolve fallible | Ian Jackson | 2022-10-12 | 1 | -5/+5 | |
| | | ||||||
| * | Fix comment typo. | Nick Mathewson | 2022-10-12 | 1 | -1/+1 | |
| | | ||||||
| * | bridge config: Clarify that examples are nonfunctional | Ian Jackson | 2022-10-12 | 1 | -0/+2 | |
| | | ||||||
| * | bridges: Test configuration | Ian Jackson | 2022-10-12 | 1 | -0/+121 | |
| | | ||||||
| * | pt and bridges: Parse configuration | Ian Jackson | 2022-10-12 | 1 | -3/+1 | |
| | | ||||||
| * | config exhaust checking: Feature-limit some of the tests | Ian Jackson | 2022-10-12 | 1 | -11/+17 | |
| | | ||||||
| * | Add pt-client and bridge-client features to arti and arti-client | Ian Jackson | 2022-10-12 | 3 | -1/+7 | |
| | | ||||||
| * | bridges config example: Add much more example syntax | Ian Jackson | 2022-10-12 | 1 | -4/+14 | |
| | | ||||||
| * | bridges config example: Add bridges section to example config | Ian Jackson | 2022-10-12 | 2 | -0/+24 | |
| | | | | | | | | This is precisely the text from the original version of !744. There is no implementation yet, so we must add a entry to the exception list in the tests. | |||||
| * | config exhaust checking: Allow for whole sections | Ian Jackson | 2022-10-12 | 1 | -2/+17 | |
| | | | | | | | | | Section headings appear uncommented in the file, so if we have a whole section which is completely unrecognized (ie, an entry with no `.`, it will be spotted when we parse the not-uncommented file too. Right now there aren't any but there will be in a moment. | |||||
| * | config exhaust checking: Allow for options only unrecognized in new | Ian Jackson | 2022-10-12 | 1 | -4/+10 | |
| | | ||||||
| * | config exhaust checking: Move unrecognized lists earlier | Ian Jackson | 2022-10-12 | 1 | -3/+3 | |
| | | | | | It turns out that we will need these even for uncommented parsing. | |||||
| * | arti config exhaust checking: Improve message, add comment | Ian Jackson | 2022-10-12 | 1 | -2/+4 | |
| | | | | | I had a failure that was confusing to me, and I wrote it... | |||||
| * | Allow "clippy::single_char_pattern" in tests. | Ian Jackson | 2022-10-12 | 1 | -0/+1 | |
| | | | | | | | | This lint exists for perf reasons, and this is rarely relevant in tests. Using double quoted str is generally cognitively less burdensome. | |||||
| * | Make cargo-sort stop complaining | eta | 2022-10-10 | 1 | -1/+1 | |
| | | ||||||
