| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | cargo: Update `arti-*` and `tor-*` to `0.38.0` | Clara Engler | 2026-01-12 | 1 | -13/+13 |
| | | | | | | | | | | Done using the following: ```bash for crate in $(./maint/list_crates | rg '^(tor|arti-)'); do cargo set-version -p $crate 0.38.0 done ``` | ||||
| * | tor-netdoc: authcert: Fix wrong link | Ian Jackson | 2026-01-06 | 1 | -1/+1 |
| | | |||||
| * | tor-netdoc: authcert: Add TODO about constructors | Ian Jackson | 2026-01-06 | 1 | -0/+2 |
| | | |||||
| * | tor-netdoc: authcert: rustfmt | Ian Jackson | 2026-01-06 | 1 | -5/+5 |
| | | |||||
| * | tor-netdoc: authcert: Delete syntax info from rustdocs | Ian Jackson | 2026-01-06 | 1 | -37/+2 |
| | | | | | | This is in the spec. I don't think it is a great idea to duplicate it here. | ||||
| * | tor-netdoc: authcert: Fix remaining old names/paths in docs | Ian Jackson | 2026-01-06 | 1 | -8/+8 |
| | | | | | | These names came from tmp, and we renamed things as we went, but didn't change the docs everywhere. | ||||
| * | tor-netdoc: authcert: Improve docs for CrossCert | Ian Jackson | 2026-01-06 | 1 | -13/+12 |
| | | | | | Fix the type names while we're here. | ||||
| * | tor-netdoc: authcert: rename tmp test module | Ian Jackson | 2026-01-06 | 1 | -1/+1 |
| | | | | | This module is now the "proper" tests for the parse2 impl on AuthCert. | ||||
| * | tor-netdoc: authcert: abolish tmp module (reindent) | Ian Jackson | 2026-01-06 | 1 | -50/+50 |
| | | |||||
| * | tor-netdoc: authcert: abolish tmp module | Ian Jackson | 2026-01-06 | 1 | -140/+14 |
| | | | | | | | | | | | | tmp's version of AuthCert is gone. Each of its fields is already in AuthCert, including the docs links. I've decided not to transfer the syntax snippets. The remaining function in that module is now an inherent method on AuthCert, not on tmp's version. This needs reformatting since verify_self_signed is now at the wrong level! | ||||
| * | tor-netdoc: authcert: derive PartialEq and Eq in tests | Ian Jackson | 2026-01-06 | 1 | -0/+1 |
| | | | | | | It's not clear that we want to expose these impls, but our existing tests (in test::tmp) want them. | ||||
| * | tor-netdoc: authcert: Transfer spec links into AuthCert from tmp | Ian Jackson | 2026-01-06 | 1 | -28/+14 |
| | | |||||
| * | tor-netdoc: authcert: derive parse2 | Ian Jackson | 2026-01-06 | 2 | -1/+13 |
| | | |||||
| * | tor-netdoc: authcert: Add CrossCert to AuthCert | Ian Jackson | 2026-01-06 | 2 | -2/+19 |
| | | | | | This will enable parse2 to process it. | ||||
| * | tor-netdoc: authcert: Move signatures out of tmp (fmt) | Ian Jackson | 2026-01-06 | 1 | -2/+2 |
| | | |||||
| * | tor-netdoc: authcert: Move signatures out of tmp | Ian Jackson | 2026-01-06 | 1 | -68/+79 |
| | | |||||
| * | tor-netdoc: authcert: Move CrossCert out of tmp | Ian Jackson | 2026-01-06 | 2 | -86/+99 |
| | | | | | | | | | We can call this type authcert::CrossCert. The names in the docs are getting rather wrong, and right now the docs build produces warnings. We'll tidy all that at the end after we're done moving and renaming. | ||||
| * | tor-netdoc: authcert: Rename verify_selfcert to verify_self_signed | Ian Jackson | 2026-01-06 | 1 | -14/+14 |
| | | | | | | | The distinction is: `verify_selfcert` sounds like it only verifies the self certificate. `veriify_self_signed` completely verifies a document, albeit one that is expected to be self-signed. | ||||
| * | tor-netdoc: authcert: Move AuthCertVersion out of tmp | Ian Jackson | 2026-01-06 | 2 | -20/+31 |
| | | | | | | Let's keep the existing name `AuthCert` in authcert.rs, so we want to rename this from DirKeyCertificateVersion, while we move it. | ||||
| * | tor-netdoc: authcert: Make all fields pub | Ian Jackson | 2026-01-06 | 1 | -7/+7 |
| | | | | | This is our new style. Now there's no known-dead code. | ||||
| * | tor-netdoc: authcert: Use Iso8601TimeSp for time fields | Ian Jackson | 2026-01-06 | 2 | -11/+11 |
| | | | | | This will let the parse2 derive work properly. | ||||
| * | tor-netdoc: authcert: Shuffle fingerprint values about | Ian Jackson | 2026-01-06 | 2 | -19/+11 |
| | | | | | | | | | | Make AuthCert look like the network document. This means removing its stored copy of H(KP_auth_sign_rsa), which it previously had via the embedded AuthCertKeyIds. We reculculate it as needed in AuthCert::key_ids(). | ||||
| * | tor-netdoc: authcert: Abolish AuthCert::sk_fingerprint | Ian Jackson | 2026-01-06 | 2 | -6/+2 |
| | | | | | There are no in-tree callers. | ||||
| * | tor-netdoc: authcert: Make key_ids return owned AuthCertKeyIds | Ian Jackson | 2026-01-06 | 3 | -5/+6 |
| | | | | | | AuthCert is about to lose its copy of H(KP_auth_sign_rsa) so it needs to return an owned value. | ||||
| * | tor-netdoc: RSA: check exponent and min size in parse2 | Ian Jackson | 2026-01-06 | 2 | -1/+24 |
| | | | | | | | | | | | | | | | | According to the spec we always use this fixed exponent, and we have a minimum size of 1024. This is checked adhoc in the old parser with some slight assistance from what is now `RsaPublicParse1Helper`. It's not clear to me that checking the exponent is actually a good idea. I think checking the size is probably a bad idea, and if it is a good idea then 1024 is clearly too short. But rather than revisit these questions, let's reproduce the old behaviour in parse2. In parse2 these checks should be features of the type. | ||||
| * | tor-netdoc: RSA: rename helper type to RsaPublicParse1Helper | Ian Jackson | 2026-01-06 | 4 | -15/+18 |
| | | | | | | | This is used by the old parsing code, apparently as a thing to hang the checking methods off. It is confusing to have so many different RSA types! Let's at least rename this one. | ||||
| * | tor-netdoc: authcert: Improve docs a bit | Ian Jackson | 2026-01-06 | 1 | -5/+12 |
| | | | | | Spec links from the tmp module will be added later. | ||||
| * | tor-netdoc: authcert: Rename fields in AuthCert to match the spec | Ian Jackson | 2026-01-06 | 2 | -39/+39 |
| | | |||||
| * | tor-netdoc: authcert: Reorder fields in AuthCert to match the spec | Ian Jackson | 2026-01-06 | 1 | -4/+4 |
| | | |||||
| * | tor-netdoc: parse_netdoc_multiple_with_offsets: Add comment about offsets | Ian Jackson | 2025-12-18 | 1 | -0/+5 |
| | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3554#note_3313768 | ||||
| * | tor-netdoc: test2: Make t_ok (which takes no boundaries) take &D not &[D] | Ian Jackson | 2025-12-18 | 1 | -16/+16 |
| | | |||||
| * | tor-netdoc: test2: Test parse_netdoc_multiple_with_offsets | Ian Jackson | 2025-12-18 | 1 | -2/+25 |
| | | | | | | | * Make a fancy version of t_ok, t_ok_multi, that takes the expected boundary byte offsets. * t_ok is now just for single-document files and is implemented in terms of t_ok_multi. * Use t_ok_multi for the one test case with multiple documents. | ||||
| * | tor-netdoc: test2: Add a test file with more than one document | Ian Jackson | 2025-12-18 | 1 | -0/+17 |
| | | |||||
| * | tor-netdoc: Provide parse2::parse_netdoc_multiple_with_offsets | Ian Jackson | 2025-12-18 | 2 | -0/+22 |
| | | |||||
| * | tor-netdoc: Break out and expose ItemStream::byte_position | Ian Jackson | 2025-12-18 | 2 | -3/+11 |
| | | |||||
| * | tor-netdoc: Fix doc comment for parse_netdoc_multiple | Ian Jackson | 2025-12-18 | 1 | -1/+1 |
| | | |||||
| * | tor-netdoc: netstatus preamble: Reorder fields to match spec order | Ian Jackson | 2025-12-18 | 1 | -21/+21 |
| | | | | | | As suggested here https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3550#note_3313757 | ||||
| * | tor-netdoc: netstatus preamble: Add a lot of blank lines | Ian Jackson | 2025-12-18 | 1 | -0/+11 |
| | | | | | | I'm going to reorder things and this will make that much less error-prone, at least for me. | ||||
| * | tor-netdoc: netstatus preamble: Clarify TODO for known-flags | Ian Jackson | 2025-12-18 | 1 | -1/+1 |
| | | |||||
| * | tor-netdoc: netstatus: improve TODO comments | Ian Jackson | 2025-12-18 | 1 | -1/+10 |
| | | | | | | | | | This is the result of reading through the spec and comparing it to the struct. Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3550#note_3312672 | ||||
| * | tor-netdoc: Clarify the wrong field bug example | Ian Jackson | 2025-12-18 | 1 | -1/+4 |
| | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3550#note_3312670 | ||||
| * | tor-netdoc: Fix typo in comment | Ian Jackson | 2025-12-18 | 1 | -1/+1 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3550#note_3312669 | ||||
| * | tor-netdoc: rustfmt | Ian Jackson | 2025-12-18 | 1 | -2/+1 |
| | | |||||
| * | tor-netdoc: Preamble: Parse published in votes | Ian Jackson | 2025-12-18 | 3 | -0/+5 |
| | | |||||
| * | tor-netdoc: Preamble: Parse consensus-methods in votes | Ian Jackson | 2025-12-18 | 4 | -1/+5 |
| | | |||||
| * | tor-netdoc: Preamble: Fix consensus-method parsing in votes | Ian Jackson | 2025-12-18 | 1 | -1/+1 |
| | | |||||
| * | tor-netdoc: impl NetdocParseableFields for Preamble | Ian Jackson | 2025-12-18 | 1 | -0/+12 |
| | | |||||
| * | tor-netdoc: impl ItemValueParseable for NetParams<i32> | Ian Jackson | 2025-12-18 | 1 | -1/+11 |
| | | |||||
| * | tor-netdoc: impl NetdocParseableFields for ProtoStatuses | Ian Jackson | 2025-12-18 | 1 | -1/+63 |
| | | |||||
| * | tor-netdoc: Use $P for prelude in one place | Ian Jackson | 2025-12-18 | 2 | -2/+2 |
| | | | | | This was missed, apparently. | ||||
