| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| | * | | Resolve a dead-code warning. | Nick Mathewson | 2022-11-15 | 1 | -0/+1 | |
| | | | | ||||||
| | * | | Allow a subset-only ChanMethod match when looking up a BridgeConfig | Nick Mathewson | 2022-11-15 | 2 | -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 Mathewson | 2022-11-15 | 1 | -1/+0 | |
| | | | | ||||||
| * | | | Merge branch 'cfg-doc' into 'main' | Ian Jackson | 2022-11-15 | 1 | -3/+48 | |
| |\ \ \ | |/ / |/| / | |/ | | | arti cfg test: Add more documentation See merge request tpo/core/arti!856 | |||||
| | * | 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. | |||||
| * | | Merge branch 'fix_parse' into 'main' | eta | 2022-11-15 | 4 | -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 Mathewson | 2022-11-15 | 2 | -3/+34 | |
| | | | | ||||||
| | * | | tor-netdoc: Add a new ErrorKind for EmptyLine. | Nick Mathewson | 2022-11-15 | 3 | -2/+8 | |
| | | | | | | | | | | | | | | | | Empty lines were previously reported as BadKeyword, which is confusing. | |||||
| * | | | Merge branch 'implement_usage' into 'main' | Nick Mathewson | 2022-11-15 | 1 | -1/+22 | |
| |\ \ \ | |/ / |/| | | | | | | | | Implement more cases for `DirSpecificTarget`. See merge request tpo/core/arti!855 | |||||
| | * | | Implement more cases for `DirSpecificTarget`. | Nick Mathewson | 2022-11-15 | 1 | -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 Mathewson | 2022-11-15 | 2 | -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 Weak | Ian Jackson | 2022-11-15 | 1 | -0/+5 | |
| | | | ||||||
| | * | arti-client: Retain a strong reference to the inner Arc<BridgeDescMgr> | Ian Jackson | 2022-11-15 | 1 | -5/+7 | |
| |/ | | | | Fixes #636. | |||||
| * | Merge branch 'bdm-plumb' into 'main' | Ian Jackson | 2022-11-15 | 2 | -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 bootstrap | Ian Jackson | 2022-11-15 | 2 | -0/+72 | |
| | | | ||||||
| | * | arti-client: Store a handle to the DirMgrStore in TorClient (fmt) | Ian Jackson | 2022-11-15 | 1 | -1/+6 | |
| | | | | | | | | | rustfmt only. | |||||
| | * | arti-client: Store a handle to the DirMgrStore in TorClient | Ian Jackson | 2022-11-15 | 1 | -1/+4 | |
| | | | | | | | | | We'll need this shortly. | |||||
| | * | arti-client: Store a handle to the guardmgr in TorClient | Ian Jackson | 2022-11-15 | 1 | -1/+5 | |
| |/ | | | | We'll need this shortly. | |||||
| * | Merge branch 'make_bridges_work' into 'main' | Nick Mathewson | 2022-11-14 | 1 | -0/+1 | |
| |\ | | | | | | | | | Bugfix: preserve active_set when loading guards See merge request tpo/core/arti!848 | |||||
| | * | Preserve active_set when loading guards. | Nick Mathewson | 2022-11-14 | 1 | -0/+1 | |
| | | | ||||||
| * | | Merge branch 'report_missing_descs' into 'main' | Nick Mathewson | 2022-11-14 | 3 | -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 Mathewson | 2022-11-14 | 2 | -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 Mathewson | 2022-11-14 | 1 | -0/+37 | |
| | | | ||||||
| * | | Merge branch 'guardmgr' into 'main' | Ian Jackson | 2022-11-14 | 5 | -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 Jackson | 2022-11-14 | 1 | -4/+8 | |
| | | | | | | | | | | | | | rustfmt only, broken out to ease review. | |||||
| | * | | Move GuardMgr::new call to arti_client::TorClient | Ian Jackson | 2022-11-14 | 4 | -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 DirMgr | Ian Jackson | 2022-11-11 | 2 | -0/+3 | |
| | | | | ||||||
| | * | | cfg-ify an import | Ian Jackson | 2022-11-11 | 1 | -0/+1 | |
| | | | | ||||||
| * | | | Merge branch 'store3' into 'main' | Ian Jackson | 2022-11-14 | 7 | -165/+76 | |
| |\ \ \ | | | | | | | | | | | | | | | | | dirmgr: Expopse DirMgrStore See merge request tpo/core/arti!851 | |||||
| | * | | | dirmgr Store: Have constructors take DirMgrStore | Ian Jackson | 2022-11-14 | 4 | -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 Jackson | 2022-11-14 | 1 | -8/+1 | |
| | | | | | | | | | | | | | | | | | rustfmt only, broken out to ease review. | |||||
| | * | | | dirmgr Store: Introduce DirMgrStore and use for BridgeDescMgr | Ian Jackson | 2022-11-14 | 2 | -6/+33 | |
| | | | | | | | | | | | | | | | | | We are going to change DirMgr's constructors to also take a DirMgrStore. | |||||
| | * | | | dirmgr Store: Abolish redundant trait impl | Ian Jackson | 2022-11-14 | 2 | -109/+1 | |
| | | | | | ||||||
| | * | | | dirmgr Store: Use `**` syntax and deref to trait objects | Ian Jackson | 2022-11-14 | 2 | -34/+20 | |
| | | | | | | | | | | | | | | | | | This will allow us to get rid of a needless trait impl. | |||||
| | * | | | dirmgr Store: Move Send and 'static bounds to Store trait | Ian Jackson | 2022-11-14 | 1 | -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 Jackson | 2022-11-14 | 2 | -0/+51 | |
| |\| | | | | | | | | | | | | | | bridge descriptor dormancy: Define an enum and semantics See merge request tpo/core/arti!845 | |||||
| | * | | bridge descriptor dormancy: Accept the dormancy value | Ian Jackson | 2022-11-11 | 2 | -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 enum | Ian Jackson | 2022-11-11 | 1 | -0/+2 | |
| | | | | ||||||
| | * | | bridge descriptor dormancy: Define an enum and semantics | Ian Jackson | 2022-11-10 | 1 | -0/+39 | |
| | | | | | | | | | | | | | This is part of #630. | |||||
| * | | | Merge branch 'arnabanimesh-main-patch-72156' into 'main' | Nick Mathewson | 2022-11-13 | 1 | -1/+1 | |
| |\ \ \ | | | | | | | | | | | | | | | | | Fix Windows shortcut target See merge request tpo/core/arti!849 | |||||
| | * | | | Fix Windows shortcut target | arnabanimesh | 2022-11-13 | 1 | -1/+1 | |
| |/ / / | ||||||
| * | | | Merge branch 'workaround_545' into 'main' | eta | 2022-11-11 | 1 | -0/+5 | |
| |\ \ \ | |_|/ |/| | | | | | | | | Temporarily disable it_cancels_delayed_firings. See merge request tpo/core/arti!846 | |||||
| | * | | Temporarily disable it_cancels_delayed_firings. | Nick Mathewson | 2022-11-10 | 1 | -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 Mathewson | 2022-11-10 | 8 | -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 Mathewson | 2022-11-10 | 2 | -10/+12 | |
| | | | | ||||||
| | * | | GuardMgr: Downgrade a pt-client TODO about an internal API | Nick Mathewson | 2022-11-10 | 1 | -2/+4 | |
| | | | | ||||||
| | * | | GuardMgr Document why a BridgeSet has trivial weight thresholds. | Nick Mathewson | 2022-11-10 | 2 | -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 Mathewson | 2022-11-10 | 1 | -3/+25 | |
| | | | | | | | | | | | | | | | | Have the check only apply to non-bridge universes, and explain why in a bunch more comments. | |||||
