| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | 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 | 1 | -0/+21 | |
| | | ||||||
| * | tor-netdoc: Break out and expose ItemStream::byte_position | Ian Jackson | 2025-12-18 | 1 | -3/+10 | |
| | | ||||||
| * | 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. | |||||
| * | tor-netdoc encode: Fix typos in multiplicity check method docs | Ian Jackson | 2025-12-10 | 1 | -4/+4 | |
| | | ||||||
| * | tor-netdoc encode: Enhance explanation of DeterminedMultiplicitySelector | Ian Jackson | 2025-12-10 | 1 | -0/+3 | |
| | | ||||||
| * | tor-netdoc encode multiplicity: Fix wrong trait ref | Ian Jackson | 2025-12-10 | 1 | -1/+1 | |
| | | ||||||
| * | tor-netdoc encode multiplicity: Add a note about omission of intro items | Ian Jackson | 2025-12-10 | 1 | -1/+1 | |
| | | ||||||
| * | tor-netdoc Linkify three more types | Ian Jackson | 2025-12-10 | 1 | -1/+1 | |
| | | ||||||
| * | tor-netdoc Fix docs typo | Ian Jackson | 2025-12-10 | 1 | -1/+1 | |
| | | ||||||
| * | tor-netdoc: Add two TODOs re trait naming | Ian Jackson | 2025-12-10 | 2 | -0/+4 | |
| | | ||||||
| * | tor-netdoc: Apply encoding derive to SharedRandStatus | Ian Jackson | 2025-12-10 | 1 | -2/+3 | |
| | | | | | | | | | I feel that before merging all this, I ought at least to demonstrate that it can be applied to at least one real type! We can unconditionally `#[derive(Deftly)]`. That is simpler, and harmless if no actual deftly derives are applied. | |||||
| * | tor-netdoc: test2: check round-trip encoding | Ian Jackson | 2025-12-10 | 1 | -29/+162 | |
| | | | | | | | | | | | | | | The test case documents are often not in canonical form. We don't want to change them to be in canonical form because we want to test parsing edge cases. So, we provide a mangling scheme with in-document notes, and annotate all the documents. Apply suitable annotations to all the documents. Test that the encoded forms are as expected. This reverts commit 05e7b8b60061a7e8a8d004a822cc213d3ea22d47. | |||||
| * | tor-netdoc: test2: implement encoding traits for all test types | Ian Jackson | 2025-12-10 | 2 | -20/+59 | |
| | | | | | | | | Make this whole module conditional on the encode feature. Apply the derives. Provide the necessary manual implementations for the types where we use `with`. | |||||
| * | tor-netdoc: derive netdoc encoding traits | Ian Jackson | 2025-12-10 | 4 | -0/+487 | |
| | | | | | | Signature encoding is not yet supported. We need to consider how this will work. | |||||
| * | tor-netdoc: debug: Add a placeholder debug print to all the other derives | Ian Jackson | 2025-12-10 | 2 | -0/+36 | |
| | | | | | | | | | We want to recognise `#[deftly(netdoc(debug))]` because otherwise we have to make lots of these attributes conditional. So, implement it for *all* the derives, providing a nugatory debug statement. | |||||
| * | tor-netdoc: parse2: debug: Explain why we use io::stderr() | Ian Jackson | 2025-12-10 | 1 | -0/+2 | |
| | | ||||||
| * | tor-netdoc: parse2: debug: Fix an error message | Ian Jackson | 2025-12-10 | 1 | -1/+1 | |
| | | | | | What is being written to here is StderrLock, not a String. | |||||
| * | tor-netdoc: parse2: debug: Fix scoping of netdoc_parseable_derive_debug | Ian Jackson | 2025-12-10 | 2 | -1/+2 | |
| | | | | | | | While testing I discovered that the `#[deftly(netdoc(debug))]` feature (which dumps parsing progress to stderr) didn't compile if the derive was invoked from elsewhere. | |||||
| * | tor-netdoc: derive: F_KEYWORD_REPORT: put concat on the outside | Ian Jackson | 2025-12-10 | 1 | -2/+2 | |
| | | | | | This makes it possible to use $F_KEYWORD_REPORT within another concat. | |||||
| * | tor-netdoc: derive_common: Break out docs for attrs in flatten | Ian Jackson | 2025-12-10 | 2 | -8/+13 | |
| | | | | | | | Code motion fromk NetdocParseableFields to NetdocFieldsDeriveCommon. This uses the derive-deftly imported doc comment feature. | |||||
| * | tor-netdoc: derive_common: Break out field ordering check | Ian Jackson | 2025-12-10 | 2 | -16/+18 | |
| | | | | | Code motion from NetdocParseable to NetdocEntireDeriveCommon. | |||||
| * | tor-netdoc: derive_common: Break out item field type definitions | Ian Jackson | 2025-12-10 | 2 | -6/+6 | |
| | | | | | Code motion from ItemValueParseable to NetdocItemDeriveCommon. | |||||
| * | tor-netdoc: derive_common: Break out main field type definitions | Ian Jackson | 2025-12-10 | 2 | -12/+11 | |
| | | | | | | | | | Code motion from NetdocParseable to NetdocEntireDeriveCommon. Code motion from NetdocParseableFields to NetdocFieldsDeriveCommon. Again, the encoder wants exactly this logic. | |||||
| * | tor-netdoc: derive_common: Break out F_KEYWORD_*, F_FLATTEN, F_NORMAL | Ian Jackson | 2025-12-10 | 2 | -34/+33 | |
| | | | | | | | | Code motion from NetdocSomeItemsParseableCommon to NetdocSomeItemsDeriveCommon. Encoding wants exactly the same logic. | |||||
| * | tor-netdoc: Introduce new derive_common Rust module with d-d modules | Ian Jackson | 2025-12-10 | 2 | -0/+68 | |
| | | | | | | | | There is nothing actually in any of these yet. But it is most convenient to lay out the structure now. We'll move code into these modules in forthcoming commits. | |||||
| * | tor-netdoc: parse2: Rename NetdocParseableCommon d-d module | Ian Jackson | 2025-12-10 | 1 | -3/+3 | |
| | | | | | | | | | This is only used for structs which contain items, not item value structs. We need a name that reflects this, especially as we are going to add a bunch more modules full of common macro elements. | |||||
| * | tor-netdoc: Break out netdoc_ordering_check into new derive_common.rs | Ian Jackson | 2025-12-10 | 3 | -34/+38 | |
| | | ||||||
