| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | | | | hs_ntor: fix comment typo INTROUDCE1 | Jim Newsome | 2023-11-15 | 1 | -1/+1 | |
| | |/ / / |/| | | | ||||||
| * | | | | Merge branch 'keymgr-docs2' into 'main' | gabi-250 | 2023-11-15 | 6 | -87/+142 | |
| |\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | tor-keymgr: Minor doc improvements Closes #1066 See merge request tpo/core/arti!1731 | |||||
| | * | | | tor-keymgr: Add missing backticks (fmt). | Gabriela Moldovan | 2023-11-13 | 3 | -4/+3 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Add missing backticks. | Gabriela Moldovan | 2023-11-13 | 1 | -2/+2 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Move module-level docs to KeyMgr. | Gabriela Moldovan | 2023-11-13 | 1 | -13/+13 | |
| | | | | | | | | | | | | | | | | | The module is private so it won't show up in the public docs. | |||||
| | * | | | tor-keymgr: Expand the ArtiNativeKeystore docs. | Gabriela Moldovan | 2023-11-13 | 1 | -1/+19 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Clarify how the default/secondary stores are used. | Gabriela Moldovan | 2023-11-13 | 1 | -2/+25 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Mention the custom SSH algorithm name for expanded ed keys. | Gabriela Moldovan | 2023-11-13 | 1 | -1/+2 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Clarify what ArtiPath are used for. | Gabriela Moldovan | 2023-11-13 | 2 | -2/+7 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Add some doctests. | Gabriela Moldovan | 2023-11-13 | 1 | -0/+35 | |
| | | | | | ||||||
| | * | | | tor-keymgr: Abolish KeyPathPatternSet. | Gabriela Moldovan | 2023-11-13 | 4 | -65/+39 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | We don't really need it. Closes #1066 | |||||
| | * | | | tor-keymgr: Downgrade a TODO HSS to TODO. | Gabriela Moldovan | 2023-11-13 | 1 | -1/+1 | |
| | | |/ | |/| | ||||||
| * | | | Merge branch 'circmgr-test-mock-runtime' into 'main' | Ian Jackson | 2023-11-14 | 2 | -33/+36 | |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | tor-circmgr: Port tests to MockRuntime Closes #1090 See merge request tpo/core/arti!1727 | |||||
| | * | | | tor-circmgr: partially port mgr tests to MockRuntime | Saksham Mittal | 2023-11-14 | 1 | -11/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | MockSleepRuntime wraps a MockRuntime instead of using test_with_one_runtime!(), so the tests should be more deterministic | |||||
| | * | | | tor-circmgr: partially port test_double_timeout() to MockRuntime | Saksham Mittal | 2023-11-14 | 1 | -6/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tests still use MockSleepRuntime for the more complicated tests where we forcefully make one or both futures time out, but MockSleepRuntime wraps a MockRuntime instead of using test_with_all_runtimes!(), so the tests should be more deterministic | |||||
| | * | | | tor-circmgr: fix build_*_timeout() for MockRuntime port | Saksham Mittal | 2023-11-14 | 1 | -8/+5 | |
| | | | | | | | | | | | | | | | | | | | | | The funky thing was getting rid of the Canceled error from the channel and returning the actual error from build_owned() properly | |||||
| | * | | | tor-circmgr: have FakeCirc derive Debug | Saksham Mittal | 2023-11-11 | 1 | -5/+2 | |
| | | | | | ||||||
| | * | | | tor-circmgr: Port some build_* tests to MockRuntime | Saksham Mittal | 2023-11-11 | 1 | -15/+27 | |
| | |/ / | | | | | | | | | | build_modest_timeout() and build_huge_timeout() don't pass yet | |||||
| * | | | tor-hsservice: Downgrade TODO HSS to TODO. | Gabriela Moldovan | 2023-11-13 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | tor-hsservice: Remove some `allow`s we don't need anymore. | Gabriela Moldovan | 2023-11-13 | 1 | -4/+0 | |
| |/ / | ||||||
| * | | Merge branch 'send_msg_but_no_handler' into 'main' | Nick Mathewson | 2023-11-09 | 1 | -2/+3 | |
| |\ \ | | | | | | | | | | | | | | | | | | | Make conversation::send_message work without an error. Closes #1085 See merge request tpo/core/arti!1726 | |||||
| | * | | circuit: On SendMsgAndInstallHandler, tolerate None handler | Nick Mathewson | 2023-11-02 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously it was possible for `handler` to be None only when `msg` was also None, which would make SendMsgAndInstallHandler into a no-op. Now, if `msg` is present but `handler` is absent, we use the previously installed handler, which I think was our intention. Without this patch, `Conversation::send_message` simply won't work. Fixes #1085. | |||||
| | * | | circuit::reactor: Remember that meta_handler is Send. | Nick Mathewson | 2023-11-02 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | (We already require that it is Send when the client gives it to us in circuit.rs, but we had previously forgotten that when we stored it in the Reactor.) | |||||
| * | | | Merge branch 'time-store' into 'main' | Ian Jackson | 2023-11-02 | 5 | -7/+430 | |
| |\ \ \ | | | | | | | | | | | | | | | | | Provide module for handling storage of times on disk See merge request tpo/core/arti!1723 | |||||
| | * | | | tor-hsservice: time_store: Add TODO HSS re serialisations | Ian Jackson | 2023-11-02 | 1 | -0/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1723#note_2960861 https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1723#note_2961013 | |||||
| | * | | | tor-hsservice: time_store: Slight structural tidying | Ian Jackson | 2023-11-02 | 1 | -3/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1723#note_2960877 and following some IRC discussion. | |||||
| | * | | | HSS time_store: clarify by removing a comma | gabi-250 | 2023-11-02 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | tor-hsservice: New time_store module (provide accessors) | Ian Jackson | 2023-11-01 | 1 | -2/+43 | |
| | | | | | | | | | | | | | | | | | These are the logically necessary accessors. | |||||
| | * | | | tor-hsservice: New time_store module | Ian Jackson | 2023-11-01 | 2 | -0/+372 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I found time handling in IPT persistence very confusing to think about, so I propose to deal with all of the hard questions in a module. Again we have a doctest so use the technique we used for having timeout_track not be semver-exposed. | |||||
| | * | | | tor-rtmock: MockRuntimeBuilder: new sleep_provider() setter | Ian Jackson | 2023-11-01 | 2 | -1/+11 | |
| | | | | | ||||||
| | * | | | tor-rtmock: simplify/reorganise MockRuntimeBuilder::build | Ian Jackson | 2023-11-01 | 1 | -4/+3 | |
| | | | | | | | | | | | | | | | | | | | | | Always reify a default SimpleMockTimeProvider, and then apply the starting_wallclock to it. This will make the next change fit. | |||||
| | * | | | tor-hsservice: timeout_track: Remove some unnecessary allows | Ian Jackson | 2023-11-01 | 1 | -3/+1 | |
| | | | | | | | | | | | | | | | | | Retain the TODO for making this properly pub somewhere. | |||||
| * | | | | tor-keymgr: Add TODO about adding helper for implementing KeyDenotator. | Gabriela Moldovan | 2023-11-02 | 1 | -0/+2 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Map all errors to KeystoreCorruptionError (fmt). | Gabriela Moldovan | 2023-11-02 | 1 | -4/+2 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Map all errors to KeystoreCorruptionError. | Gabriela Moldovan | 2023-11-02 | 1 | -2/+2 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Use KeystoreCorruptionError instead of internal!. | Gabriela Moldovan | 2023-11-02 | 1 | -3/+3 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Add an error type for keystore corruption (fmt). | Gabriela Moldovan | 2023-11-02 | 2 | -4/+4 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Add an error type for keystore corruption. | Gabriela Moldovan | 2023-11-02 | 2 | -1/+30 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Use ArrayVec to avoid an unnecessary allocation. | Gabriela Moldovan | 2023-11-02 | 2 | -2/+4 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Use parse() instead of from_str(). | Gabriela Moldovan | 2023-11-02 | 1 | -5/+3 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Add TODO about supporting multiple key denotators. | Gabriela Moldovan | 2023-11-02 | 1 | -0/+5 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Add test for the KeyDenotator impl for TimePeriod. | Gabriela Moldovan | 2023-11-02 | 1 | -0/+11 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Remove unused KeyDenotator::glob() function. | Gabriela Moldovan | 2023-11-02 | 1 | -11/+0 | |
| | | | | | | | | | | | | | | | | | | | | | We no longer need this function now that we always match the entire denotator rather than its individual components, | |||||
| * | | | | tor-keymgr: Add test for ArtiPath denotator validation. | Gabriela Moldovan | 2023-11-02 | 1 | -0/+22 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Update the test for paths with `+` as a disallowed char. | Gabriela Moldovan | 2023-11-02 | 1 | -1/+9 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | `'+'` is now allowed to occur once within an `ArtiPath` (but **not** within an `ArtiPathComponent`), so we need to special case the `"c++"` test. | |||||
| * | | | | tor-hsservice: Use KeyDenotator::decode instead of manually decoding the ↵ | Gabriela Moldovan | 2023-11-02 | 1 | -12/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | denotator. Note that instead of matching each individual component of a key denotator (e.g. using a glob pattern like `*_*_*`), we now match the entire denotator (using the `*` pattern) and let `KeyDenotator::decode` handle its parsing. In the long run, this approach should reduce code duplication (as the decoding logic for each type is centralized in its `KeyDenotator` implementation) and the need for complex glob patterns. Closes #1070 | |||||
| * | | | | tor-keymgr: Use the new denotator separator instead of underscore. | Gabriela Moldovan | 2023-11-02 | 2 | -11/+24 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Underscores are used within `ArtiPath`s as visual separators. To reduce ambiguity, we now use a different character for separating the key denotators (e.g. time periods) from the rest of the `ArtiPath`. Closes #1063 | |||||
| * | | | | tor-keymgr: Add a constant for the denotator separator. | Gabriela Moldovan | 2023-11-02 | 2 | -1/+9 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Remove now-outdated TODOs. | Gabriela Moldovan | 2023-11-02 | 1 | -4/+0 | |
| | | | | | ||||||
| * | | | | tor-keymgr: Rename KeyDenotator::display() to KeyDenotator::encode(). | Gabriela Moldovan | 2023-11-02 | 1 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | This renames the serialization function to `encode()` for consistency with its `KeyDenotator::decode()` counterpart. | |||||
