| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | tor-hsservice: ipt_mgr: Move storage handle to state | Ian Jackson | 2024-02-01 | 2 | -9/+11 | |
| | | | | | | | | | | | The new state_dir types require &mut for storing, which is correct, but that means we can't have it in Immutable. | |||||
| * | | tor-hsservice: impl state_dir::InstanceIdentity for HsNickname | Ian Jackson | 2024-02-01 | 1 | -0/+10 | |
| | | | ||||||
| * | | tor-hsservice: storage: Move Arcs into type aliases | Ian Jackson | 2024-02-01 | 4 | -12/+12 | |
| | | | | | | | | | | | | | | | These are here because that's what you get from the tor_persist singleton StageMgr API (for type erasure reasons). We're going to change these to tor_persist::state_dir types and those don't involve Arcs. | |||||
| * | | tor-hsservice: ipt_set: Make store method take &mut self | Ian Jackson | 2024-02-01 | 1 | -1/+1 | |
| | | | | | | | | | | | Only people who can mutate the state ought to be saving it. Otherwise there might be concurrent overwrites. | |||||
| * | | tor-hsservice: tests: Add a missing drop call | Ian Jackson | 2024-02-01 | 1 | -0/+2 | |
| | | | | | | | | | | | | | This doesn't actually change the behaviour with current Rust. But it avoids bugs and future changes. Relying on drop order for temporary directory lifetime seems bad. | |||||
| * | | tor-persist: state_dir: Add some missing Clone and Debug impls | Ian Jackson | 2024-02-01 | 1 | -5/+5 | |
| | | | ||||||
| * | | tor-persist: state_dir: Introduce way to get at underlying lock guard | Ian Jackson | 2024-02-01 | 2 | -3/+39 | |
| | | | | | | | | | | | | | And export the type, so callers don't need to depend directly on fslock_guard; many callers will need to own the returned value, not do anything else with it. | |||||
| * | | tor-persist: state_dir: Make the storage handle Send and Sync | Ian Jackson | 2024-02-01 | 1 | -2/+5 | |
| | | | ||||||
| * | | Merge branch 'clippy' into 'main' | Nick Mathewson | 2024-01-31 | 7 | -8/+7 | |
| |\ \ | |/ |/| | | | | | Fix nightly clippy warnings See merge request tpo/core/arti!1942 | |||||
| | * | clippy: Use Result::cloned in several places | Ian Jackson | 2024-01-31 | 3 | -3/+3 | |
| | | | ||||||
| | * | tor-dirmgr: Simplify a clone call (fmt) | Ian Jackson | 2024-01-31 | 1 | -3/+1 | |
| | | | ||||||
| | * | tor-dirmgr: Simplify a clone call | Ian Jackson | 2024-01-31 | 1 | -2/+1 | |
| | | | | | | | | | Prompted by clippy. | |||||
| | * | arti-rpcserver: Add an allow | Ian Jackson | 2024-01-31 | 1 | -0/+1 | |
| | | | ||||||
| | * | tor-rpcbase: Add an allow | Ian Jackson | 2024-01-31 | 1 | -0/+1 | |
| | | | ||||||
| | * | tor-error: Backtrace: Remove a Captured(...) from a Display | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | | | | | | | Prompted by clippy complaining that the content wasn't ever read other than by the autogenerated Debug impl. | |||||
| * | | Merge branch 'ipt_parameters' into 'main' | Nick Mathewson | 2024-01-31 | 7 | -42/+62 | |
| |\ \ | | | | | | | | | | | | | | | | | | | Clean up, tune, and correct various parameters related to introduction points. Closes #1210 See merge request tpo/core/arti!1924 | |||||
| | * | | Rename parameters for intro-point lifetime to match the spec. | Nick Mathewson | 2024-01-31 | 2 | -6/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (Also, correct the comments that describe them.) We may as well match the spec names when they aren't completely bogus. We are already renaming these parameters for this release, so it isn't an additional breaking change. | |||||
| | * | | hss: clean up comments on ipt establish time tuning | Nick Mathewson | 2024-01-25 | 1 | -3/+7 | |
| | | | | | | | | | | | | | See #1210 "question 4". | |||||
| | * | | Base INTRO_POINT_EXPIRY_SLOP on OVERALL_UPLOAD_TIMEOUT. | Nick Mathewson | 2024-01-25 | 3 | -4/+7 | |
| | | | | | | | | | | | | | See discussion at #1210, "question 5". | |||||
| | * | | Document allowed numbers of intro points, and make them match C | Nick Mathewson | 2024-01-25 | 1 | -7/+17 | |
| | | | | | | | | | | | | | | | | | | | | | | Part of #1210. These values are not necessarily the cleverest possible, but they match the C implementation. We can tune them if we need to. | |||||
| | * | | hss: Pick random intro-point lifetime based on netparams | Nick Mathewson | 2024-01-25 | 2 | -12/+14 | |
| | | | | | | | | | | | | | | | | This makes us more spec-conformant. See discussion at #1210 "question 3". | |||||
| | * | | Rename our aliases for hs_intro_*_lifetime. | Nick Mathewson | 2024-01-25 | 2 | -2/+3 | |
| | | | | | | | | | | | | | | | | | | | The old names were somewhat inaccurate. (I would have considered keeping and deprecating the old names, but we already have breaking changes in tor-netdir.) | |||||
| | * | | fixup! hss: Use 3 hours for IPT_PUBLISH_*. | Nick Mathewson | 2024-01-25 | 1 | -3/+0 | |
| | | | | ||||||
| | * | | Use hs_intro_num_extra param for max_n_intro_relays. | Nick Mathewson | 2024-01-25 | 1 | -3/+3 | |
| | | | | | | | | | | | | | See #1210 "question 2" for more discussion here. | |||||
| | * | | hss: Use 3 hours for IPT_PUBLISH_*. | Nick Mathewson | 2024-01-25 | 1 | -6/+9 | |
| | | | | | | | | | | | | | See discussion and rationale on #1210 "question 1". | |||||
| * | | | Merge branch 'lowercase-slugs' into 'main' | Ian Jackson | 2024-01-31 | 3 | -15/+11 | |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | tor-keymgr: Lowercase the role when building the ArtiPath. Closes #1195 See merge request tpo/core/arti!1934 | |||||
| | * | | | tor-persist: Reduce nesting in check_syntax(). | Ian Jackson | 2024-01-31 | 1 | -6/+4 | |
| | | | | | ||||||
| | * | | | tor-persist: Reject slugs that contain uppercase characters. | Gabriela Moldovan | 2024-01-31 | 1 | -1/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `slug` docs say slugs are not allowed to contain uppercase characters, so let's return an error if they do. Part of #1195 | |||||
| | * | | | tor-keymgr: Lowercase the role when building the ArtiPath. | Gabriela Moldovan | 2024-01-31 | 2 | -14/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The role is a slug, and slugs are not allowed to contain uppercase characters. Closes #1195 | |||||
| * | | | | Merge branch 'rustdoc_fix' into 'main' | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| |\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | rtcompat: Fix a rustdoc link. See merge request tpo/core/arti!1938 | |||||
| | * | | | rtcompat: Fix a rustdoc link. | Nick Mathewson | 2024-01-30 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | tor-persist: state_dir: Fix a docs reference | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | tor-persist: state_dir: Fix a leftover reference to `delete` | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | This is now InstanceStateHandle::purge. | |||||
| * | | | | tor-persist: Add a test for Resource::InstanceState | Ian Jackson | 2024-01-31 | 1 | -0/+38 | |
| | | | | | | | | | | | | | | | | | | | | | We should add more tests for these error representations but we want to rework these errors so let's not do all that now. | |||||
| * | | | | Require tracing-test 0.2.4 | Ian Jackson | 2024-01-31 | 7 | -7/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Apparently we need https://github.com/dbrgn/tracing-test/pull/17 too. | |||||
| * | | | | Require tracing-test 0.2.2 | Ian Jackson | 2024-01-31 | 7 | -7/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need this bugfix https://github.com/dbrgn/tracing-test/pull/15 for `#[traced_test]` when `Result` is redefined. Fixes the tests with minimal-versions. | |||||
| * | | | | tor-hsservice: state_dir: Quote some strings in an error | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | tor-hsservice: state_dir: Fix a wrong error string | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | We use `/` to separate, not `+`. | |||||
| * | | | | tor-hsservice: state_dir: Fix a wrong comment | gabi-250 | 2024-01-31 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | tor-hsservice: state_dir: Fix typo | gabi-250 | 2024-01-31 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | state_dir: Deferred fmt churn | Ian Jackson | 2024-01-31 | 1 | -3/+1 | |
| | | | | | ||||||
| * | | | | state_dir: When comparing path, use official separators | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | This would fix the tests on Windows I think ? (Tests which don't seem to run in routine CI AFAICT.) | |||||
| * | | | | state_dir: Use subdirectory rather than + notation | Ian Jackson | 2024-01-31 | 1 | -8/+9 | |
| | | | | | | | | | | | | | | | | | As previously decided and documented. | |||||
| * | | | | state_dir: Break out make_secure_directory closure | Ian Jackson | 2024-01-31 | 1 | -4/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're about to reuse this. This changes the Resource in the error to be more accurate, too. This will be more important as we add another call site. | |||||
| * | | | | state_dir: Introduce local PATH_SEPARATOR alias | Ian Jackson | 2024-01-31 | 1 | -2/+3 | |
| | | | | | | | | | | | | | | | | | We're going to want this some more. | |||||
| * | | | | tor-persist: sort Cargo.toml | Ian Jackson | 2024-01-31 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | tor-persist: state_dir: Remove three unused allows | Ian Jackson | 2024-01-31 | 1 | -3/+0 | |
| | | | | | ||||||
| * | | | | tor-persist: state_dir: Test functionality that exists now | Ian Jackson | 2024-01-31 | 2 | -0/+109 | |
| | | | | | ||||||
| * | | | | tor-persist: state_dir: Enable rustfmt (formatting) | Ian Jackson | 2024-01-31 | 1 | -15/+14 | |
| | | | | | | | | | | | | | | | | | Remaining formatting churn. | |||||
| * | | | | tor-persist: state_dir: Implement StateDirectory::new | Ian Jackson | 2024-01-31 | 1 | -1/+17 | |
| | | | | | ||||||
