mirrors/arti.git - mirror of https://gitlab.torproject.org/tpo/core/arti

	Commit message (Collapse)	Author	Age	Files	Lines
...
\| * \| \|	tor-persist: state_dir: Enable rustfmt	Ian Jackson	2024-01-31	1	-1/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The formatting will be normalised roughly as we go.
\| * \| \|	tor-persist: load_store: Add some tracing	Ian Jackson	2024-01-31	1	-3/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Now we get log messages about what we're reading/writing.
\| * \| \|	tor-persist: Fix a doc comment	Ian Jackson	2024-01-31	1	-1/+1
\|/ / /
* \| \|	Merge branch 'arti-path-slugs' into 'main'	gabi-250	2024-01-31	16	-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 Moldovan	2024-01-31	1	-1/+1
\| \| \| \|
\| * \| \|	tor-hsclient: Rename HsClientSpecifier to HsClientNickname (fmt).	Gabriela Moldovan	2024-01-31	2	-4/+4
\| \| \| \|
\| * \| \|	tor-hsclient: Rename HsClientSpecifier to HsClientNickname.	Gabriela Moldovan	2024-01-31	4	-13/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Service nicknames are represented by `HsNickname`, so let's rename `HsClientSpecifier` to `HsClientNickname`.
\| * \| \|	tor-persist: Make all Slugs non-empty.	Gabriela Moldovan	2024-01-31	5	-26/+26
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This also removes a TODO regarding adding a `Nickname` type for representing `HsClientSpecifier` and `HsNickname` (we don't need it if `Slug`s are non-empty). Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1931#note_2990227
\| * \| \|	tor-keymgr: Add roundrip test for HsId -> Slug -> HsId conversion.	Gabriela Moldovan	2024-01-31	1	-1/+22
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This also fixes a bug in `HsId::from_component`.
\| * \| \|	tor-keymgr: Allow empty denotators in ArtiPath (fmt).	Gabriela Moldovan	2024-01-31	1	-8/+2
\| \| \| \|
\| * \| \|	tor-keymgr: Allow empty denotators in ArtiPath.	Gabriela Moldovan	2024-01-31	2	-11/+7
\| \| \| \|
\| * \| \|	tor-hsservice: Fix corner case in HsNickname FromStr impl.	Gabriela Moldovan	2024-01-31	1	-3/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It was previously possible to construct an `HsNickname` out of an empty string using the `FromStr` impl.
\| * \| \|	tor-persist: Forbid slugs from starting with a hyphen.	Gabriela Moldovan	2024-01-31	3	-5/+44
\| \| \| \|
\| * \| \|	tor-keymgr: Add a TODO about removing ValidatedString.	Gabriela Moldovan	2024-01-31	1	-0/+2
\| \| \| \|
\| * \| \|	tor-keymgr: Remove the Slug tests (fmt).	Gabriela Moldovan	2024-01-31	2	-8/+3
\| \| \| \|
\| * \| \|	tor-keymgr: Remove the Slug tests.	Gabriela Moldovan	2024-01-31	1	-34/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	These tests belong in tor_persist now.
\| * \| \|	tor-keymgr: Abolish ArtiPathComponent.	Gabriela Moldovan	2024-01-31	7	-104/+57
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`ArtiPathComponent`s are really just `Slugs`. Part of #1193, #1092
\| * \| \|	tor-hsclient: Use Slug for HsClientSpecifier (fmt).	Gabriela Moldovan	2024-01-31	1	-11/+3
\| \| \| \|
\| * \| \|	tor-hsclient: Use Slug for HsClientSpecifier.	Gabriela Moldovan	2024-01-31	2	-4/+21
\| \| \| \|
\| * \| \|	tor-keymgr: Make ArtiPathComponent a Slug internally (fmt).	Gabriela Moldovan	2024-01-31	3	-36/+57
\| \| \| \|
\| * \| \|	tor-keymgr: Make ArtiPathComponent a Slug internally.	Gabriela Moldovan	2024-01-31	3	-104/+70
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This will make it easier to replace `ArtiPathComponent` with `Slug` later down the line. Note this changes the syntax rules of `ArtiPathComponent`: * previously `ArtiPathComponent`s could be unicode strings, (now they are lowercase ASCII alphanumerics, plus `-`, `_`) * previously `ArtiPathComponent`s couldn't start with `-` or `_`, but now they can Part of #1193, #1092
\| * \| \|	tor-keymgr: Strip .onion suffix from HsId before building Slug.	Gabriela Moldovan	2024-01-31	3	-5/+30
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We are about to replace `ArtiPathComponent` with `Slug`, but `Slug`s don't support `.`, so let's strip the `.onion` suffix before encoding it in the `ArtiPath`.
\| * \| \|	tor-hsservice: Make Slug the underlying repr of HsNickname.	Gabriela Moldovan	2024-01-31	1	-19/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We represent `HsNickname` as a `Slug` instead of an `ArtiPathComponent` (we are about to remove the latter). Part of #1092
\| * \| \|	tor_persist::slug: Allow hyphens in Slugs.	Gabriela Moldovan	2024-01-31	1	-2/+3
\|/ / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The internal representation of `HsNickname` will soon be `Slug`, and nicknames can contain hyphens, so let's add `-` to the allowed charset. Part of #1092
* \| \|	Merge branch 'new-strum' into 'main'	gabi-250	2024-01-30	11	-23/+45
\|\ \ \ \| \|/ / \|/\| \| \| \| \| \| \| \|	Upgrade to strum 0.26 See merge request tpo/core/arti!1937
\| * \|	Upgrade to strum 0.26	Nick Mathewson	2024-01-30	11	-23/+45
\|/ /
* \|	Merge branch 'empty-slug' into 'main'	Ian Jackson	2024-01-29	1	-0/+12
\|\ \ \| \| \| \| \| \| \| \| \| \| \| \|	slug: Provide BadSlug::EmptySlugNotAllowed See merge request tpo/core/arti!1933
\| * \|	slug: Provide BadSlug::EmptySlugNotAllowed	Ian Jackson	2024-01-29	1	-0/+12
\| \| \|
* \| \|	Merge branch 'make-secure-dir' into 'main'	Ian Jackson	2024-01-29	2	-0/+19
\|\ \ \ \| \|/ / \|/\| \| \| \| \| \| \| \|	fs_mistrust: Provide CheckedDir::make_secure_dir See merge request tpo/core/arti!1927
\| * \|	fs_mistrust: Provide CheckedDir::make_secure_dir	Ian Jackson	2024-01-29	2	-0/+19
\|/ / \| \| \| \| \| \| \| \| \| \|	state_dir wants this, to descend into subdirectories. I think the implementation could be improved - see the TODO.
* \|	Merge branch 'publisher-tasks' into 'main'	gabi-250	2024-01-26	1	-3/+67
\|\ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tor-hsservice: Abort any pending desc uploads if shutting down. Closes #1253 See merge request tpo/core/arti!1921
\| * \|	tor-hsservice: Downgrade a debug message to trace.	Gabriela Moldovan	2024-01-26	1	-1/+4
\| \| \|
\| * \|	tor-hsservice: Abort any pending desc uploads if shutting down.	Gabriela Moldovan	2024-01-26	1	-4/+46
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`Reactor::upload_for_time_period` now exits as soon as the Reactor is dropped (its upload tasks check if `shutdown_tx` was dropped before proceeding). Closes #1253
\| * \|	tor-hsservice: Add an UploadError type for interruption due to shutdown.	Gabriela Moldovan	2024-01-26	1	-0/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This error will be returned if an upload is aborted because the reactor shutting down.
\| * \|	tor-hsservice: Document how the publisher tasks are shut down.	Gabriela Moldovan	2024-01-26	1	-0/+7
\|/ / \| \| \| \| \| \|	Part of #1253
* \|	Merge branch 'set_faulty_eventually' into 'main'	Nick Mathewson	2024-01-26	2	-6/+24
\|\ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	HSS: Mark an Ipt as faulty if it fails continuously for 15 minutes. Closes #1248 See merge request tpo/core/arti!1915
\| * \|	HSS: Mark an Ipt as faulty if it fails continuously for 15 minutes.	Nick Mathewson	2024-01-25	2	-6/+24
\| \| \| \| \| \| \| \| \| \| \| \|	Closes #1248.
* \| \|	Merge branch 'verifier' into 'main'	Nick Mathewson	2024-01-26	2	-2/+3
\|\ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	fs_mistrust: Expose CheckedDir::verifier See merge request tpo/core/arti!1928
\| * \| \|	fs_mistrust: Expose CheckedDir::verifier	Ian Jackson	2024-01-25	2	-2/+3
\| \|/ / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I'm not sure why this isn't public. It seems like an obviously reasonable transformation. If I as author of !1927 had been an external contributor, I could have worked around the lack of .make_secure_directory() if .verifier() had existed. So I think exposing it would be useful.
* \| \|	Merge branch 'slug2' into 'main'	Ian Jackson	2024-01-25	2	-94/+62
\|\ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	TryIntoSlug See merge request tpo/core/arti!1922
\| * \| \|	state_dir: Remove a now-unused allow	Ian Jackson	2024-01-25	1	-1/+0
\| \| \| \|
\| * \| \|	state_dir: Replace InstanceIdString with Slug and SlugRef	Ian Jackson	2024-01-25	1	-44/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	And tidy up some of the docs to refer to the slug module.
\| * \| \|	state_dir: Abolish now-obsolete Slug trait rename	Ian Jackson	2024-01-25	1	-5/+3
\| \| \| \|
\| * \| \|	slugs: Provide TryIntoSlug, and replace state_dir::Slug with it	Ian Jackson	2024-01-25	2	-47/+48
\| \| \| \|
* \| \| \|	Merge branch 'tempdir' into 'main'	Ian Jackson	2024-01-25	10	-70/+153
\|\ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	test-temp-dir: New crate See merge request tpo/core/arti!1925
\| * \| \| \|	test-temp-dir: Plan when we think we'll do 1.x	Ian Jackson	2024-01-25	1	-0/+1
\| \| \| \| \|
\| * \| \| \|	test-temp-dir: Drop a TODO/allow	Ian Jackson	2024-01-25	1	-3/+0
\| \| \| \| \|
\| * \| \| \|	test-temp-dir: Update docs for it being a crate	Ian Jackson	2024-01-25	1	-5/+5
\| \| \| \| \|
\| * \| \| \|	test-temp-dir: Improve the docs and fix broken links	Ian Jackson	2024-01-25	1	-9/+22
\| \| \| \| \|
\| * \| \| \|	test-temp-dir: New crate	Ian Jackson	2024-01-25	10	-67/+139
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Introduce the crate, move the code motion, and make minimal necessary changes.