summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Resolve a dead-code warning.Nick Mathewson2022-11-151-0/+1
| | |
| * | Allow a subset-only ChanMethod match when looking up a BridgeConfigNick Mathewson2022-11-152-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | If we have a bridge guard that is using Direct connection and it knows multiple addresses, our code to match it with a BridgeConfig is wrong, because the BridgeConfig has only one address, and our code looks for an exact match. Fixes #642.
| * | Remove a spurious log that I committed by mistake. Whoops.Nick Mathewson2022-11-151-1/+0
| | |
* | | Merge branch 'cfg-doc' into 'main'Ian Jackson2022-11-151-3/+48
|\ \ \ | |/ / |/| / | |/ | | arti cfg test: Add more documentation See merge request tpo/core/arti!856
| * 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.
* | Merge branch 'fix_parse' into 'main'eta2022-11-154-5/+42
|\ \ | | | | | | | | | | | | | | | | | | netdoc: Allow empty lines at the end of router descriptors Closes #637 See merge request tpo/core/arti!857
| * | netdoc: Allow empty lines at the end of a router descriptor.Nick Mathewson2022-11-152-3/+34
| | |
| * | tor-netdoc: Add a new ErrorKind for EmptyLine.Nick Mathewson2022-11-153-2/+8
| | | | | | | | | | | | | | | Empty lines were previously reported as BadKeyword, which is confusing.
* | | Merge branch 'implement_usage' into 'main'Nick Mathewson2022-11-151-1/+22
|\ \ \ | |/ / |/| | | | | | | | Implement more cases for `DirSpecificTarget`. See merge request tpo/core/arti!855
| * | Implement more cases for `DirSpecificTarget`.Nick Mathewson2022-11-151-1/+22
|/ / | | | | | | | | | | Without this fix, any attempt to actually use a circuit for a `DirSpecificTarget` will fail, since supports() will say false, and `restrict_mut()` will say `NotSupported`.
* | Merge branch 'bdm-arc' into 'main'Nick Mathewson2022-11-152-5/+12
|\ \ | |/ |/| | | | | | | | | arti-client: Retain a strong reference to the inner Arc<BridgeDescMgr> Closes #636 See merge request tpo/core/arti!854
| * guardmgr: Document that guardmgr only holds a WeakIan Jackson2022-11-151-0/+5
| |
| * arti-client: Retain a strong reference to the inner Arc<BridgeDescMgr>Ian Jackson2022-11-151-5/+7
|/ | | | Fixes #636.
* Merge branch 'bdm-plumb' into 'main'Ian Jackson2022-11-152-2/+86
|\ | | | | | | | | | | | | Create and install a BridgeDescMgr when we bootstrap Closes #631 See merge request tpo/core/arti!852
| * bridges: Create and install a BridgeDescMgr when we bootstrapIan Jackson2022-11-152-0/+72
| |
| * arti-client: Store a handle to the DirMgrStore in TorClient (fmt)Ian Jackson2022-11-151-1/+6
| | | | | | | | rustfmt only.
| * arti-client: Store a handle to the DirMgrStore in TorClientIan Jackson2022-11-151-1/+4
| | | | | | | | We'll need this shortly.
| * arti-client: Store a handle to the guardmgr in TorClientIan Jackson2022-11-151-1/+5
|/ | | | We'll need this shortly.
* Merge branch 'make_bridges_work' into 'main'Nick Mathewson2022-11-141-0/+1
|\ | | | | | | | | Bugfix: preserve active_set when loading guards See merge request tpo/core/arti!848
| * Preserve active_set when loading guards.Nick Mathewson2022-11-141-0/+1
| |
* | Merge branch 'report_missing_descs' into 'main'Nick Mathewson2022-11-143-1/+66
|\| | | | | | | | | | | | | GuardMgr: Update desired bridge descs from 'update' function. Closes #622 See merge request tpo/core/arti!847
| * GuardMgr: Update desired bridge descs from 'update' function.Nick Mathewson2022-11-142-1/+29
| | | | | | | | | | There are some new TODOs here for us to think about, but I think this will give us something to test.
| * GuardSet: Add a method to report which bridge descs we want.Nick Mathewson2022-11-141-0/+37
| |
* | Merge branch 'guardmgr' into 'main'Ian Jackson2022-11-145-4/+23
|\ \ | | | | | | | | | | | | Move guardmgr creation to arti-client, rather than within circmgr See merge request tpo/core/arti!850
| * | Move GuardMgr::new call to arti_client::TorClient (fmt)Ian Jackson2022-11-141-4/+8
| | | | | | | | | | | | rustfmt only, broken out to ease review.
| * | Move GuardMgr::new call to arti_client::TorClientIan Jackson2022-11-144-2/+13
| | | | | | | | | | | | | | | | | | | | | Having this done within circmgr was irregular - most of our other key buildup functions are done in TorClient::create_inner. It is also inconvenient, as it buries the guardmgr within the circmgr.
| * | Document Clone semantics of GuardMgr and DirMgrIan Jackson2022-11-112-0/+3
| | |
| * | cfg-ify an importIan Jackson2022-11-111-0/+1
| | |
* | | Merge branch 'store3' into 'main'Ian Jackson2022-11-147-165/+76
|\ \ \ | | | | | | | | | | | | | | | | dirmgr: Expopse DirMgrStore See merge request tpo/core/arti!851
| * | | dirmgr Store: Have constructors take DirMgrStoreIan Jackson2022-11-144-10/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Now the Store is constructed by arti_client, solving the problem described here https://gitlab.torproject.org/tpo/core/arti/-/issues/631#note_2853665 but in a different way.
| * | | dirmgr Store: Introduce DirMgrStore and use for BridgeDescMgr (fmt)Ian Jackson2022-11-141-8/+1
| | | | | | | | | | | | | | | | rustfmt only, broken out to ease review.
| * | | dirmgr Store: Introduce DirMgrStore and use for BridgeDescMgrIan Jackson2022-11-142-6/+33
| | | | | | | | | | | | | | | | We are going to change DirMgr's constructors to also take a DirMgrStore.
| * | | dirmgr Store: Abolish redundant trait implIan Jackson2022-11-142-109/+1
| | | |
| * | | dirmgr Store: Use `**` syntax and deref to trait objectsIan Jackson2022-11-142-34/+20
| | | | | | | | | | | | | | | | This will allow us to get rid of a needless trait impl.
| * | | dirmgr Store: Move Send and 'static bounds to Store traitIan Jackson2022-11-141-2/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | This seems tidier to me. (It saved a lot of typing when I was trying to make some complicated pub supertrait with sealed private supertrait, but apparently not now.)
* | | Merge branch 'bdm-dormant' into 'main'Ian Jackson2022-11-142-0/+51
|\| | | | | | | | | | | | | | bridge descriptor dormancy: Define an enum and semantics See merge request tpo/core/arti!845
| * | bridge descriptor dormancy: Accept the dormancy valueIan Jackson2022-11-112-1/+11
| | | | | | | | | | | | But right now, don't do anything with it. That will come in a future MR.
| * | bridge descriptor dormancy: Add a TODO for this enumIan Jackson2022-11-111-0/+2
| | |
| * | bridge descriptor dormancy: Define an enum and semanticsIan Jackson2022-11-101-0/+39
| | | | | | | | | | | | This is part of #630.
* | | Merge branch 'arnabanimesh-main-patch-72156' into 'main'Nick Mathewson2022-11-131-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Fix Windows shortcut target See merge request tpo/core/arti!849
| * | | Fix Windows shortcut targetarnabanimesh2022-11-131-1/+1
|/ / /
* | | Merge branch 'workaround_545' into 'main'eta2022-11-111-0/+5
|\ \ \ | |_|/ |/| | | | | | | | Temporarily disable it_cancels_delayed_firings. See merge request tpo/core/arti!846
| * | Temporarily disable it_cancels_delayed_firings.Nick Mathewson2022-11-101-0/+5
| |/ | | | | | | | | This test is timing-dependent in a way that seems to fail on heavily loaded CI machines. See #545.
* | Merge branch 'guardmgr-todo-misc' into 'main'Nick Mathewson2022-11-108-75/+142
|\ \ | | | | | | | | | | | | Resolve most "TODO pt-client" comments in GuardMgr See merge request tpo/core/arti!840
| * | GuardMgr: Explain why we timestamp BridgeSets as we do.Nick Mathewson2022-11-102-10/+12
| | |
| * | GuardMgr: Downgrade a pt-client TODO about an internal APINick Mathewson2022-11-101-2/+4
| | |
| * | GuardMgr Document why a BridgeSet has trivial weight thresholds.Nick Mathewson2022-11-102-3/+4
| | | | | | | | | | | | | | | | | | This is the only way I could find in which parameter interpretation differs between bridge guards and relay guards; with it documented, I can remove a TODO about identifying such ways.
| * | GuardMgr: once more, fix our n_primary_dir_info check.Nick Mathewson2022-11-101-3/+25
| | | | | | | | | | | | | | | Have the check only apply to non-bridge universes, and explain why in a bunch more comments.