summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | 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
| | | |
| * | | tor-persist: state_dir: Test functionality that exists nowIan Jackson2024-01-313-0/+111
| | | |
| * | | tor-persist: state_dir: Enable rustfmt (formatting)Ian Jackson2024-01-311-15/+14
| | | | | | | | | | | | | | | | Remaining formatting churn.
| * | | tor-persist: state_dir: Implement StateDirectory::newIan Jackson2024-01-311-1/+17
| | | |
| * | | tor-persist: state_dir: Implement acquire_instanceIan Jackson2024-01-312-7/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | | | And also instance_peek_storage, which needs to share some code, so determined the shape of this implementation. Make StateDirectory have the (one) field we need.
| * | | tor-persist: state_dir: Implement InstanceStateHandle and StorageHandleIan Jackson2024-01-311-8/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are the straightforward methods which, given a CheckedDir, perform the relevant operation. Make InstanceStateHandle have the necessary fields. Constructing an InstanceStateHandle is a bit complex and will come in a moment.
| * | | tor-persist: state_dir: Have instance id throw fmt::ErrorIan Jackson2024-01-311-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In practice, allowing Bug here was quite inconvenient for everyone. I think very few call sites will want to throw anything interesting. If they *do* want to cause an error, without panicking, they could write a syntactically invalid slug.
| * | | tor-persist: Error: Provide for handling LocalResourceAlreadyInUseIan Jackson2024-01-311-0/+5
| | | | | | | | | | | | | | | | | | | | state_dir doesn't support a "shared read-only" mode, so it needs an error.
| * | | tor-persist: Error: Provide for handling BadSlugIan Jackson2024-01-311-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce a Bug variant. Then, use it for conversions from BadSlug. See the comment for why I think this is appropriate.
| * | | tor-persist: state_dir: Two API fixesIan Jackson2024-01-311-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | You don't need to consume a Mistrust to get a CheckedDir - you use a Verifier and then something clones it for you. So StateDirectory::new should take &Mistrust. And instance_peek_storage needs T to be deserialisable!
| * | | tor-persist: state_dir: Rename InstanceStateHandle::purge (from delete)Ian Jackson2024-01-311-2/+2
| | | | | | | | | | | | | | | | | | | | This is rather more serious than many deletions - it wipes out the whole directory. And this name goes better with purge_instances.
| * | | tor-persist: state_dir: Enable rustfmtIan Jackson2024-01-311-1/+0
| | | | | | | | | | | | | | | | The formatting will be normalised roughly as we go.
| * | | tor-persist: load_store: Add some tracingIan Jackson2024-01-311-3/+18
| | | | | | | | | | | | | | | | Now we get log messages about what we're reading/writing.
| * | | tor-persist: Fix a doc commentIan Jackson2024-01-311-1/+1
|/ / /
* | | Merge branch 'arti-path-slugs' into 'main'gabi-2502024-01-3116-242/+250
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-keymgr: Replace ArtiPathComponent with Slug Closes #1193 and #1092 See merge request tpo/core/arti!1931
| * | | tor-persist: Document that slugs cannot start with '-'.Gabriela Moldovan2024-01-311-1/+1
| | | |
| * | | tor-hsclient: Rename HsClientSpecifier to HsClientNickname (fmt).Gabriela Moldovan2024-01-312-4/+4
| | | |
| * | | tor-hsclient: Rename HsClientSpecifier to HsClientNickname.Gabriela Moldovan2024-01-314-13/+12
| | | | | | | | | | | | | | | | | | | | Service nicknames are represented by `HsNickname`, so let's rename `HsClientSpecifier` to `HsClientNickname`.