summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | netdoc: Test more accessors for tokenized documents.Nick Mathewson2022-06-072-4/+20
| | | | | | |
| * | | | | | Netdoc: add accessor to error for ParseErrorKindNick Mathewson2022-06-071-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The kind type was already public, but we didn't give the user any way to get it.
| * | | | | | netdoc: tests for parsing UnvalidatedEdCert.Nick Mathewson2022-06-071-1/+47
| | | | | | |
| * | | | | | tor-cert: Make more types Clone and Debug.Nick Mathewson2022-06-071-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (Not sure how we missed this before. This is part of making more of tor-netdoc tested.)
| * | | | | | Tests for netdoc RsaPublic type.Nick Mathewson2022-06-071-2/+37
| | | | | | |
| * | | | | | Fix a copy-paste error in Ed25519 parsing tests.Nick Mathewson2022-06-011-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We accidentally tried to validate (invalid) ed25519 keys as curve25519 keys.
| * | | | | | Add test for parsing bad tor version with too many elements.Nick Mathewson2022-06-011-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a coverage-driven test.
| * | | | | | Add test for mutability of RelayFamily.Nick Mathewson2022-06-011-0/+9
| | | | | | |
| * | | | | | Tests for tor_netdoc::util::internNick Mathewson2022-06-011-0/+33
| | |_|/ / / | |/| | | |
* | | | | | Merge branch 'ticket463_01-cell' into 'main'Ian Jackson2022-06-076-1/+599
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | cell: Implement UDP cell ABI See merge request tpo/core/arti!542
| * | | | | | udp: New AddressPort used in cellsDavid Goulet2022-06-072-42/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | | | udp: Allow empty hostname and no nul byteIan Jackson2022-06-073-34/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After changes to the prop339, the domain name in an Address can only be 255 bytes max and can NOT contain nul byte(s). Unit tests had to be modified to accept this change: - Centralise msg_ip_address - Add currently-passing tests for address length - Test counted address length longer than type wants Related to #463 Signed-off-by: David Goulet <[email protected]>
| * | | | | | cell: Don't use NUL terminated string in CONNECT_UDPDavid Goulet2022-06-073-43/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: David Goulet <[email protected]>
| * | | | | | cell: Move UDP to its own module and feature gate itDavid Goulet2022-06-076-314/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to #463 Signed-off-by: David Goulet <[email protected]>
| * | | | | | cell: Implement DATAGRAM cell from prop339David Goulet2022-06-072-4/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decoding and encoding of the DATAGRAM cell from proposal 339. Related to #463. Signed-off-by: David Goulet <[email protected]>
| * | | | | | cell: Implement CONNECTED_UDP cell from prop339David Goulet2022-06-072-5/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decoding and encoding implemented according to proposal 339. Related to #463
| * | | | | | cell: Implement CONNECT_UDP cell from prop339David Goulet2022-06-075-4/+622
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decoding and encoding is implemented according to proposal 339 specifications. Related to #463 Signed-off-by: David Goulet <[email protected]>
* | | | | | Merge branch 'llcrypto-openssl' into 'main'Nick Mathewson2022-06-074-0/+144
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add support for openssl backend in llcrypto Closes #442 See merge request tpo/core/arti!550
| * | | | | | use the right feature name for llcrypto openssltrinity-1686a2022-05-312-5/+5
| | | | | | |
| * | | | | | add support for openssl backend in llcryptotrinity-1686a2022-05-304-0/+144
| | | | | | |
* | | | | | | Merge branch 'updates-20220607' into 'main'Ian Jackson2022-06-076-21/+23
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | Update a few dependencies to more recent versions See merge request tpo/core/arti!570
| * | | | | | Upgrade serial_test dev-dependency to 0.7.0Nick Mathewson2022-06-072-6/+7
| | | | | | |
| * | | | | | Upgrade float_eq dev-dependency to 1.0.0Nick Mathewson2022-06-074-5/+5
| | | | | | |
| * | | | | | Upgrade tls-api to latest versions in arti-hyper.Nick Mathewson2022-06-072-10/+11
|/ / / / / /
* | | | | | Merge branch 'typos' into 'main'Nick Mathewson2022-06-053-6/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix typos See merge request tpo/core/arti!567
| * | | | | | Fix typosDimitris Apostolou2022-06-053-6/+6
|/ / / / / /
* | | | | | Merge branch 'testing-rng' into 'main'Nick Mathewson2022-06-024-0/+344
|\ \ \ \ \ \ | |_|_|/ / / |/| | | / / | | |_|/ / | |/| | | New facility for deterministic and reproducible test PRNGs. See merge request tpo/core/arti!561
| * | | | New facility for deterministic and reproducible test PRNGs.Nick Mathewson2022-06-024-0/+344
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new `testing_rng()` function is meant as a replacement for thread_rng() for use in unit tests. By default, it uses a randomly seeded RNG, but prints the seed before the test so that you can reproduce any failures that occur. You can override this via the environment to use a previous seed, or by using a deterministic seed for all your tests. Backend for #486.
* | | | | Merge branch 'check-license-fixes' into 'main'Ian Jackson2022-06-022-1/+20
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update check_licenses and add it to CI. Closes #462 See merge request tpo/core/arti!559
| * | | | | Add check_licenses to CI.Nick Mathewson2022-05-311-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The artifact-caching logic is taken from other examples in the .gitlab-ci.yml.
| * | | | | check_licenses: list new crates with manual license verificationNick Mathewson2022-05-311-1/+17
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #462. Note that the license on `ring` is slightly problematic for some users, including as it does old the openssl license[^1], with advertising clause and all. That's not a blocker for us now, since `ring` is not a required dependency. But we wouldn't want `ring` to become a mandatory dependency because of this. [^1]: To make the situation even more complicated, modern openssl has relicensed under apache-2.0, but that doesn't necessarily help us, since ring took its code from boringssll, which forked from an older version of openssl.
* | | | | Merge branch 'better-fs-mistrust-badpermissions-string' into 'main'Ian Jackson2022-06-024-29/+43
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | fs-mistrust: Improve BadPermission string See merge request tpo/core/arti!554
| * | | | | fs-mistrust: Improve BadPermission stringAlex Xu (Hello71)2022-05-313-17/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To me, "Incorrect permissions on file or directory /path: g=w o=w" implies that the current permissions on /path are 022. Change the message to "Incorrect permissions: /path is u=rwx,g=rwx,o=rwx; need g-w,o-w", which is closer to chmod syntax and is more useful in non-interactive environments such as CI and support.
| * | | | | fs-mistrust: BadPermission(_, _) -> BadPermission(..)Alex Xu (Hello71)2022-05-313-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | Next commit adds another parameter to Error::BadPermission.
* | | | | | Merge branch 'doc' into 'main'Nick Mathewson2022-06-011-1/+1
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | tor-config: Fix a doc link See merge request tpo/core/arti!563
| * | | | | tor-config: Fix a doc linkIan Jackson2022-06-011-1/+1
|/ / / / / | | | | | | | | | | | | | | | Nightly cargo doc complaints about this.
* | | | | Merge branch 'lint-followup' into 'main'Nick Mathewson2022-06-011-17/+35
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve maint/add_warning Closes #494 See merge request tpo/core/arti!560
| * | | | | maint/add_warning: Convert remaining assertsIan Jackson2022-05-311-4/+8
| | | | | |
| * | | | | maint/add_warning: Introduce ImproperFile and convert one assertIan Jackson2022-05-311-2/+18
| | | | | |
| * | | | | maint/add_warning: Use python3 argparseIan Jackson2022-05-311-11/+9
| | | | | |
* | | | | | Merge branch 'lint' into 'main'Ian Jackson2022-05-3139-46/+211
|\| | | | | | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | lints: Make lint blocks consistent and ensure they stay that way Closes #469 See merge request tpo/core/arti!557
| * | | | maint/add_warning: Add "how to fix" to error messageIan Jackson2022-05-311-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/557#note_2808879
| * | | | maint/add_warning: Run with --check in CIIan Jackson2022-05-311-0/+1
| | | | |
| * | | | maint/add_warning: Provide --check modeIan Jackson2022-05-311-2/+22
| | | | |
| * | | | maint/add_warning: Change shebang to python3Ian Jackson2022-05-311-1/+1
| | | | | | | | | | | | | | | | | | | | "python" sometimes means Python 2.
| * | | | maint/add_warning: Use .tmp~ for temp filename, not .bakIan Jackson2022-05-311-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | "bak" ought to be a backup file, with the old contents, which this most definitely isn't.
| * | | | lints: Make lint blocks consistentIan Jackson2022-05-316-6/+7
| | | | | | | | | | | | | | | | | | | | The remaining consequences of running add_warning
| * | | | lints: arti-testing: Move some allows outside the auto blockIan Jackson2022-05-311-2/+3
| | | | |
| * | | | lints: tor-protover: Move two allows outside the auto blockIan Jackson2022-05-311-2/+3
| | | | | | | | | | | | | | | | | | | | These need to survive.
| * | | | lints: Add let_unit_value allow to all cratesIan Jackson2022-05-3135-1/+35
| | | | | | | | | | | | | | | | | | | | | | | | | From running add_warning, with manual picking of the right hunks/lines.