summaryrefslogtreecommitdiff
path: root/crates/arti
Commit message (Collapse)AuthorAgeFilesLines
...
* cfg compilation: Add two allowsIan Jackson2022-11-181-0/+1
|
* Merge branch 'fix_nightly_20221118' into 'main'Ian Jackson2022-11-181-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 nightlyNick Mathewson2022-11-181-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-defaultNick Mathewson2022-11-182-4/+14
| |
* | arti: Split long lines in Cargo.tomlNick Mathewson2022-11-181-1/+10
|/ | | | This will make the next commit easier to review.
* pt config, arti cfg test: Expect bridges.transports to be missingIan Jackson2022-11-161-2/+9
| | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/823#note_2854365
* arti cfg test: Coalesce expect_missingIan Jackson2022-11-161-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-clienteta2022-11-161-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 exhaustiveIan Jackson2022-11-151-0/+39
|
* arti cfg test: Break out CONFIG_KEYS_EXPECT_NO_EXAMPLEIan Jackson2022-11-151-2/+9
| | | | Giving this a name will allow us to refer to it docs in a moment.
* arti cfg test: Drop a redundant allowIan Jackson2022-11-151-1/+0
| | | | | This allow is also present in the standard lint block at the start of the test inline module.
* Fix Windows shortcut targetarnabanimesh2022-11-131-1/+1
|
* bump rust-version to 1.60 in every crate.Nick Mathewson2022-11-101-1/+1
|
* Check more targetstrinity-1686a2022-11-102-2/+6
|
* Merge branch 'clap-v3' into 'main'Ian Jackson2022-11-082-35/+43
|\ | | | | | | | | | | | | Upgrade to clap 3 Closes #616 See merge request tpo/core/arti!830
| * Upgrade to clap 3: replace deprecated functionsGabriel de Perthuis2022-11-042-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 changesGabriel de Perthuis2022-11-042-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 cratesNick Mathewson2022-11-072-4/+4
| |
* | Fix typosDimitris Apostolou2022-11-061-3/+3
|/
* Run add_warnings.Nick Mathewson2022-11-032-0/+2
|
* Merge branch 'notify' into 'main'Ian Jackson2022-11-011-123/+137
|\ | | | | | | | | | | | | refactor config reload triggering Closes #562 See merge request tpo/core/arti!819
| * debounce reload cfg eventstrinity-1686a2022-11-011-0/+7
| |
| * rename unused fieldtrinity-1686a2022-10-301-3/+2
| |
| * update some stringstrinity-1686a2022-10-301-10/+10
| |
| * use iife so we can try! instead of custom error handlingtrinity-1686a2022-10-301-64/+56
| |
| * refactor so events always indicate reloadtrinity-1686a2022-10-301-71/+74
| |
| * depends less on notify typestrinity-1686a2022-10-301-22/+35
| |
* | remove unused dependanciestrinity-1686a2022-10-291-3/+0
|/
* cargo fmt to remove blank linesIan Jackson2022-10-121-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 includesIan Jackson2022-10-121-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-clientIan Jackson2022-10-121-5/+5
| | | | Add ,ignore to ignore three shell runes.
* cfg tests: bridges: Document test case assumptionsIan Jackson2022-10-121-0/+19
|
* cfg tests: bridges: Remove now-redundant block { }Ian Jackson2022-10-121-13/+11
|
* cfg test: bridges: Test all three feature casesIan Jackson2022-10-121-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 testingIan Jackson2022-10-121-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 fallibleIan Jackson2022-10-121-5/+5
|
* Fix comment typo.Nick Mathewson2022-10-121-1/+1
|
* bridge config: Clarify that examples are nonfunctionalIan Jackson2022-10-121-0/+2
|
* bridges: Test configurationIan Jackson2022-10-121-0/+121
|
* pt and bridges: Parse configurationIan Jackson2022-10-121-3/+1
|
* config exhaust checking: Feature-limit some of the testsIan Jackson2022-10-121-11/+17
|
* Add pt-client and bridge-client features to arti and arti-clientIan Jackson2022-10-123-1/+7
|
* bridges config example: Add much more example syntaxIan Jackson2022-10-121-4/+14
|
* bridges config example: Add bridges section to example configIan Jackson2022-10-122-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 sectionsIan Jackson2022-10-121-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 newIan Jackson2022-10-121-4/+10
|
* config exhaust checking: Move unrecognized lists earlierIan Jackson2022-10-121-3/+3
| | | | It turns out that we will need these even for uncommented parsing.
* arti config exhaust checking: Improve message, add commentIan Jackson2022-10-121-2/+4
| | | | I had a failure that was confusing to me, and I wrote it...
* Allow "clippy::single_char_pattern" in tests.Ian Jackson2022-10-121-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 complainingeta2022-10-101-1/+1
|