summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | tor-persist: state_dir: Introduce way to get at underlying lock guardIan Jackson2024-02-013-3/+40
| | | | | | | | | | | | | | | | | | 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 SyncIan Jackson2024-02-011-2/+5
|/ /
* | Merge branch 'clippy' into 'main'Nick Mathewson2024-01-318-8/+8
|\ \ | | | | | | | | | | | | Fix nightly clippy warnings See merge request tpo/core/arti!1942
| * | gsoc2023: download-manager: Truncate the output fileIan Jackson2024-01-311-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this we might end up with a mixture of new file and previous wreckage. If this were released code I would think about treating this as a security issue. Prompted by clippy.
| * | clippy: Use Result::cloned in several placesIan Jackson2024-01-313-3/+3
| | |
| * | tor-dirmgr: Simplify a clone call (fmt)Ian Jackson2024-01-311-3/+1
| | |
| * | tor-dirmgr: Simplify a clone callIan Jackson2024-01-311-2/+1
| | | | | | | | | | | | Prompted by clippy.
| * | arti-rpcserver: Add an allowIan Jackson2024-01-311-0/+1
| | |
| * | tor-rpcbase: Add an allowIan Jackson2024-01-311-0/+1
| | |
| * | tor-error: Backtrace: Remove a Captured(...) from a DisplayIan Jackson2024-01-311-1/+1
| | | | | | | | | | | | | | | Prompted by clippy complaining that the content wasn't ever read other than by the autogenerated Debug impl.
* | | Merge branch 'ci3' into 'main'Nick Mathewson2024-01-311-0/+2
|\ \ \ | |_|/ |/| | | | | | | | CI: Run coverage test on bigger runners See merge request tpo/core/arti!1944
| * | CI: Run coverage test on bigger runnersIan Jackson2024-01-311-0/+2
| |/ | | | | | | | | | | | | | | | | | | | | This is starting to get SIGKILL during linking. Lack of memory, presumably. @anarcat suggests we should run this on bigger runners, and that this could be achieved by applying the `tpa` tag. CC @trinity-1686a See also !1943.
* | Merge branch 'ipt_parameters' into 'main'Nick Mathewson2024-01-317-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 Mathewson2024-01-312-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 tuningNick Mathewson2024-01-251-3/+7
| | | | | | | | | | | | See #1210 "question 4".
| * | Base INTRO_POINT_EXPIRY_SLOP on OVERALL_UPLOAD_TIMEOUT.Nick Mathewson2024-01-253-4/+7
| | | | | | | | | | | | See discussion at #1210, "question 5".
| * | Document allowed numbers of intro points, and make them match CNick Mathewson2024-01-251-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 netparamsNick Mathewson2024-01-252-12/+14
| | | | | | | | | | | | | | | This makes us more spec-conformant. See discussion at #1210 "question 3".
| * | Rename our aliases for hs_intro_*_lifetime.Nick Mathewson2024-01-252-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 Mathewson2024-01-251-3/+0
| | |
| * | Use hs_intro_num_extra param for max_n_intro_relays.Nick Mathewson2024-01-251-3/+3
| | | | | | | | | | | | See #1210 "question 2" for more discussion here.
| * | hss: Use 3 hours for IPT_PUBLISH_*.Nick Mathewson2024-01-251-6/+9
| | | | | | | | | | | | See discussion and rationale on #1210 "question 1".
* | | Merge branch 'ci' into 'main'gabi-2502024-01-311-0/+2
|\ \ \ | | | | | | | | | | | | | | | | CI: Run some tests a bit earlier See merge request tpo/core/arti!1940
| * | | CI: blocking-todos: Run quicklyIan Jackson2024-01-311-0/+1
| | | | | | | | | | | | | | | | | | | | This is a very fast check. It's last in the list so that nothing gets blocked *by* it, but we want it to run early.
| * | | CI: minimal-versions: Block only on rust-checksIan Jackson2024-01-311-0/+1
| | | | | | | | | | | | | | | | | | | | In particular, let's not wait for all the other builds, to make iterating minimal-versions through CI less painful.
* | | | Merge branch 'ci2' into 'main'Nick Mathewson2024-01-311-0/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | CI: Run coverage test only in scheduled pipelines See merge request tpo/core/arti!1943
| * | | | CI: Run coverage test only in scheduled pipelinesIan Jackson2024-01-311-0/+2
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | This is starting to get SIGKILL during linking. Lack of memory, presumably. @nickm suggests running it only daily since apparently we don't look at the output that often anyway.
* | | | Merge branch 'lowercase-slugs' into 'main'Ian Jackson2024-01-316-16/+12
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 Jackson2024-01-311-6/+4
| | | | |
| * | | | tor-persist: Reject slugs that contain uppercase characters.Gabriela Moldovan2024-01-311-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 Moldovan2024-01-315-15/+8
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | The role is a slug, and slugs are not allowed to contain uppercase characters. Closes #1195
* | | | Merge branch 'rustdoc_fix' into 'main'Ian Jackson2024-01-311-1/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | rtcompat: Fix a rustdoc link. See merge request tpo/core/arti!1938
| * | | rtcompat: Fix a rustdoc link.Nick Mathewson2024-01-301-1/+1
| | | |
* | | | Merge branch 'state' into 'main'Ian Jackson2024-01-3112-52/+493
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | Implement most of state_dir See merge request tpo/core/arti!1935
| * | | tor-persist: state_dir: Fix a docs referenceIan Jackson2024-01-311-1/+1
| | | |
| * | | tor-persist: state_dir: Fix a leftover reference to `delete`Ian Jackson2024-01-311-1/+1
| | | | | | | | | | | | | | | | This is now InstanceStateHandle::purge.
| * | | tor-persist: Add a test for Resource::InstanceStateIan Jackson2024-01-311-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.4Ian Jackson2024-01-317-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | Apparently we need https://github.com/dbrgn/tracing-test/pull/17 too.
| * | | Require tracing-test 0.2.2Ian Jackson2024-01-317-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 errorIan Jackson2024-01-311-1/+1
| | | |
| * | | tor-hsservice: state_dir: Fix a wrong error stringIan Jackson2024-01-311-1/+1
| | | | | | | | | | | | | | | | We use `/` to separate, not `+`.
| * | | tor-hsservice: state_dir: Fix a wrong commentgabi-2502024-01-311-1/+1
| | | |
| * | | tor-hsservice: state_dir: Fix typogabi-2502024-01-311-1/+1
| | | |
| * | | state_dir: Deferred fmt churnIan Jackson2024-01-311-3/+1
| | | |
| * | | state_dir: When comparing path, use official separatorsIan Jackson2024-01-311-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 + notationIan Jackson2024-01-311-8/+9
| | | | | | | | | | | | | | | | As previously decided and documented.
| * | | state_dir: Break out make_secure_directory closureIan Jackson2024-01-311-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 aliasIan Jackson2024-01-311-2/+3
| | | | | | | | | | | | | | | | We're going to want this some more.
| * | | tor-persist: sort Cargo.tomlIan Jackson2024-01-311-1/+1
| | | |
| * | | tor-persist: state_dir: Remove three unused allowsIan Jackson2024-01-311-3/+0
| | | |