summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | chanmgr: Add a comment/TODO about channel expiryDavid Goulet2026-03-181-0/+16
| | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | chanmgr: Support reconfigure of unauthenticated channelsDavid Goulet2026-03-181-5/+8
| | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | chanmgr: Use the unauth_channels listDavid Goulet2026-03-181-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Put channel without relay identities in the unauth_channels list. Signed-off-by: David Goulet <[email protected]>
| * | | | chanmgr: Add a list of unauthenticated channelsDavid Goulet2026-03-181-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Needed to handle client/bridge -> relay channels which are not authenticated meaning they don't have a RelayIds so they can't be put in the ListByRelayIds<>. Feature gate it to "relay" as a client will never have those kind of channels. Signed-off-by: David Goulet <[email protected]>
* | | | | Merge branch 'arti-extra-everything' into 'main'Jim Newsome2026-03-181-13/+25
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | arti-extra build: enable everything in full and experimental See merge request tpo/core/arti!3759
| * | | | arti-extra: don't include flowctl-cc or counter-galois-onionJim Newsome2026-03-181-8/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Including these would introduce flakiness, until arti#2400 is resolved. This also adds a general mechanism for omitting specific features in the arti-extra build.
| * | | | arti-extra build: enable everything in full and experimentalJim Newsome2026-03-181-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we enabled features piecemeal to avoid enabling `rpc`, which resulted in breakage when running under shadow. Experimentally, this appears to no longer be the case. Enabling everything in `full` and `experimental` gives us at least a little bit of testing of those features (that they don't break the build or other usage), even if we don't specifically exercise them (yet) in the CI tests.
* | | | | Merge branch 'http-connect-outbound-rebased' into 'main'Nick Mathewson2026-03-188-75/+638
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Http connect outbound rebased See merge request tpo/core/arti!3789
| * | | | cargo fmtNihal2026-03-181-6/+9
| | | | |
| * | | | tor-chanmgr: add loopback validation, startup warning and httparseNihal2026-03-186-176/+334
| | | | |
| * | | | tor-chanmgr: split handshake into helper functionsNihal2026-03-181-14/+47
| | | | |
| * | | | tor-chanmgr: fix type inference in handshakeNihal2026-03-182-6/+6
| | | | |
| * | | | tor-chanmgr: refactor handshake into helper functionsNihal2026-03-181-30/+53
| | | | |
| * | | | tor-chanmgr: add base64ct dependency and docsNihal2026-03-184-8/+19
| | | | |
| * | | | tor-chanmgr: implement HTTP CONNECT proxy handshake and wire transportNihal2026-03-182-20/+186
| | | | |
| * | | | tor-chanmgr: add ProxyProtocol::HttpConnect variant and URI parsingNihal2026-03-181-36/+205
| | | | |
* | | | | Merge branch 'chutney-arti-extra' into 'main'Jim Newsome2026-03-186-47/+157
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | chutney test: include arti-extra bin See merge request tpo/core/arti!3772
| * | | | | chutney test: include arti-extra binJim Newsome2026-03-115-20/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test the arti-extra bin, which includes most of `full` and `experimental` features, too.
| * | | | | network.py: break up construction of configs listJim Newsome2026-03-111-47/+46
| | | | | |
| * | | | | chutney test: don't override CHUTNEY_START_TIMEJim Newsome2026-03-111-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default has been increased to 300, so there's no need to override it here.
| * | | | | chutney ci: convert chutney-script to a normal python moduleJim Newsome2026-03-115-44/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will give us a bit more flexibility, e.g. to pass additional arguments. Now that the python linters understand it, we fix some linting issues. We preserve the ability to run the test on other networks by making the Config.network optional. The new default of `None` means to use the built-in module to create the network. The functionality of the test itself is the same.
* | | | | | Merge branch 'extra-info' into 'main'Clara Engler2026-03-183-1/+138
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-dirclient: Support for extra-info requests See merge request tpo/core/arti!3764
| * | | | | | tor-netdoc: Add TODO wrt double definitionClara Engler2026-03-181-0/+1
| | | | | | |
| * | | | | | tor-dirclient: Add TODO wrt Default for requestsClara Engler2026-03-181-0/+2
| | | | | | |
| * | | | | | tor-netdoc: Properly export ExtraInfoDigestClara Engler2026-03-121-0/+3
| | | | | | |
| * | | | | | tor-dirclient: Add TODO for renamingClara Engler2026-03-121-0/+1
| | | | | | |
| * | | | | | tor-dirclient: No ref semantic in ExtraInfoClara Engler2026-03-121-6/+6
| | | | | | |
| * | | | | | tor-netdoc: Add ExtraInfoDigest type aliasClara Engler2026-03-121-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Accidentally left out in the previous commit.
| * | | | | | tor-dirclient: Small semantic fixupsClara Engler2026-03-121-5/+15
| | | | | | |
| * | | | | | tor-dirclient: Support for extra-info requestsClara Engler2026-03-111-1/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit implements support for extra-info requests in tor-dirclient. It is gated behind the already existing `routerdesc` feature flag, because entities building upon router descriptors most likely also need extra-info, namely directory mirrors and directory authorities. The implementation is largely based upon the existing one for router descriptors, as they are of a very similar request format, most likely due to simply being closely related anyways.
* | | | | | | Merge branch 'ticket2404_01' into 'main'David Goulet2026-03-175-180/+515
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | relay: Crypto task rewrite Closes #2404 See merge request tpo/core/arti!3782
| * | | | | | | relay: Pass now() instead of the runtime in the crypto taskDavid Goulet2026-03-171-37/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This way we get the whole task job to be aligned on one single now value to avoid potential inconsistencies between expiry and key generation. Fixes #2404 Signed-off-by: David Goulet <[email protected]>
| * | | | | | | relay: Add crypto tasks unit testDavid Goulet2026-03-173-1/+245
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tests the key rotation code. Signed-off-by: David Goulet <[email protected]>
| * | | | | | | relay: Use same wallclock() time when generating keysDavid Goulet2026-03-171-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | | | | relay: Make the crypto tasks use the runtime wallclockDavid Goulet2026-03-172-25/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This way we can unit tests properly. Signed-off-by: David Goulet <[email protected]>
| * | | | | | | relay: Rewrite the rotation key logic in the crypto taskDavid Goulet2026-03-172-163/+252
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is almost a full rewrite of the crypto task which was needed in order to support our relay signing certificate to be put in the keystore which will be needed for the offline key feature. Instead of having rotate_key() do all the things, we now instead do two pass: 1. Remove all expired keys and certs. 2. Generate any missing keys. This still results in using the minimum valid_until of all our keys for the task sleep time. We can know cleanup the local trait used for this gymnastic and trade it for some more KeyMgr gymnastic. Fixes #2404 Signed-off-by: David Goulet <[email protected]>
| * | | | | | | relay: Don't log warn if key already exists when generating oneDavid Goulet2026-03-171-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | | | | relay: Remove leftover comment from previous code iterationDavid Goulet2026-03-171-4/+0
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
* | | | | | | Merge branch 'expiry_hours' into 'main'David Goulet2026-03-174-34/+86
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cert: Fix minor rounding bug in systemtime-to-expiry conversion Closes #2407 See merge request tpo/core/arti!3787
| * | | | | | cert: Fix minor but annoying bug in cert expiry calculationNick Mathewson2026-03-171-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We documented our SystemTime-to-expiry conversion as always rounding _up_, but we did not account for fractional seconds when doing so. Therefore, if the requested expiration was set partway through the first second of an hour, the conversion would round down. This patch fixes that, and adds a regression test. I've confirmed that the test fails without this patch. Closes #2407
| * | | | | | cert: Deduplicate expiry-in-hours logicNick Mathewson2026-03-174-34/+70
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These certificates use a weird expiration format: counting hours since the unix epoch. Previously we had it implemented in two different places. This patch centralizes it, since we are about to become slightly more complicated.
* | | | | | Merge branch 'keymgr-cert-fixes2' into 'main'gabi-2502026-03-172-21/+115
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | keymgr: Use the keypair specifier when generating keys. See merge request tpo/core/arti!3785
| * | | | | keymgr: Remove now-addressed XXXGabriela Moldovan2026-03-171-3/+0
| | | | | |
| * | | | | keymgr: Fix feature-gating in testsGabriela Moldovan2026-03-171-4/+3
| | | | | |
| * | | | | keymgr: Lower get() logic into get_from_store()Gabriela Moldovan2026-03-171-13/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This moves the logic for retrieving a public key from its corresponding keypair into `get_from_store()`. Fixes a bug which made it impossible to retrieve a public key using the key specifier of its keypair type with any function other than `KeyMgr::get()`.
| * | | | | keymgr: Add more tests to reveal the get_*() bug for public keysGabriela Moldovan2026-03-171-1/+65
| | | | | | | | | | | | | | | | | | | | | | | | This will be fixed in the next commit.
| * | | | | keymgr: Pass the item type to entry_descriptor() (fmt)Gabriela Moldovan2026-03-171-1/+5
| | | | | |
| * | | | | keymgr: Pass the item type to entry_descriptor()Gabriela Moldovan2026-03-171-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | I am about to repurpose this test helper for other item types too.
| * | | | | keymgr: Preserve item metadata when converting to TestPublicKey (fmt)Gabriela Moldovan2026-03-171-1/+4
| | | | | |
| * | | | | keymgr: Preserve item metadata when converting to TestPublicKeyGabriela Moldovan2026-03-171-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | This will enable us to test the provenance of public keys.