mirrors/arti.git/crates/tor-hsrproxy/src, branch arti-v1.7.0mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.7.02025-09-29T19:30:03ZRemove "doc_auto_cfg" incantation from all crates.2025-09-29T19:30:03ZNick Mathewsonnickm@torproject.org2025-09-29T18:30:14Zurn:sha1:808b0ec2e3735a6dbbebeb857a3d6040ae02c697
This feature has been removed from nightly, in favor of doc_cfg.
proto: Move the `stream` module under `client` (breaking).2025-08-18T16:09:04ZGabriela Moldovangabi@torproject.org2025-08-18T15:52:38Zurn:sha1:702e7276898d3f31e22033c5b64fea91703f3701
The `stream` module is client-specific, for the most part, so I am
moving it under `client`. Later on, we will factor out the parts that
can be shared with the relay implementation.
Note: this is a breaking change as the deleted `stream` module was
`pub`. We could've kept the module and reexported from it the public
types from `tor_proto::client::stream`, but I think it's better to have
this `client` namespacing, because it makes the separation between the
client and relay parts clearer.
Switch Cargo.toml files to edition 2024.2025-08-07T15:28:36ZNick Mathewsonnickm@torproject.org2025-08-06T01:19:58Zurn:sha1:77b0de43b8c67cdb81befe0680a3df43b6ad37bc
First, run
```
git grep -l "^edition =" |
xargs perl -i -pe 's/^edition *=.*/edition = "2024"/;'
```
Second, manually verify that all Cargo.toml files have changed,
and nothing else has changed.
Third, run cargo fmt again.
Temporarily suppress mismatched_lifetime_syntaxes.2025-07-07T15:50:04ZGabriela Moldovangabi@torproject.org2025-07-07T15:50:04Zurn:sha1:a11b611466a5a73f5e06a0aabd17284a57d12fe3
See #2060.
metrics: Discuss schema stability (currently, "no")2025-04-08T17:20:50ZIan Jacksonijackson@chiark.greenend.org.uk2025-04-08T12:17:17Zurn:sha1:63b1afba93c24519c20e95229e894d3e91cc658ator-hsrproxy: config: make ProxyActionDiscriminants private2025-04-08T17:20:50ZIan Jacksonijackson@chiark.greenend.org.uk2025-04-08T12:05:07Zurn:sha1:0818b110e24b573a217dac6c73e8ff1db31eb6fd
beta clippy pointed out that it shouldn't be exhaustive.
but `#[strum_discriminants(non_exhaustive)]` is rejected.
metrics: tor-hsrproxy: Use a bespoke type for counter outcomes2025-04-08T17:20:50ZIan Jacksonijackson@chiark.greenend.org.uk2025-04-08T11:50:17Zurn:sha1:e02da4600e7a66bee5ac3dc57b708c0942bda895
Prompted by
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2918#note_3185872
tor-hsrproxy: Add allow(unused) if features not all enabled2025-04-08T17:20:50ZIan Jacksonijackson@chiark.greenend.org.uk2025-04-07T15:44:40Zurn:sha1:a02715d099d386a67c7a7db7702c2499df50d636
This avoids warnings from failure to cfg-decorate imports.
tor-hsrproxy: Export a connection count metric (fmt)2025-04-08T17:20:50ZIan Jacksonijackson@chiark.greenend.org.uk2025-04-02T16:23:22Zurn:sha1:016bcdbe6e93d7a565e33640c1363d5a79c7472dtor-hsrproxy: Export a connection count metric2025-04-08T17:20:50ZIan Jacksonijackson@chiark.greenend.org.uk2025-04-02T16:21:39Zurn:sha1:e9ee2f4204b5139de9946d86c942b6e62427f258
This proves the concept. Many more metrics will need to be added
throughout the codebase.
With this compiled in and enabled, I see this with curl:
> ```
> # TYPE arti_hss_proxy_connections_failed_total counter
> arti_hss_proxy_connections_failed_total{nickname="ztest",action="destroy_circuit"} 0
> arti_hss_proxy_connections_failed_total{nickname="ztest",action="ignore_stream"} 0
> arti_hss_proxy_connections_failed_total{nickname="ztest",action="forward"} 0
> arti_hss_proxy_connections_failed_total{nickname="ztest",action="reject_stream"} 0
>
> # TYPE arti_hss_proxy_connections_total counter
> arti_hss_proxy_connections_total{nickname="ztest",action="reject_stream"} 0
> arti_hss_proxy_connections_total{nickname="ztest",action="forward"} 1
> arti_hss_proxy_connections_total{nickname="ztest",action="destroy_circuit"} 0
> arti_hss_proxy_connections_total{nickname="ztest",action="ignore_stream"} 0
>
> # TYPE arti_hss_proxy_connections_ok_total counter
> arti_hss_proxy_connections_ok_total{nickname="ztest",action="destroy_circuit"} 0
> arti_hss_proxy_connections_ok_total{nickname="ztest",action="reject_stream"} 0
> arti_hss_proxy_connections_ok_total{nickname="ztest",action="ignore_stream"} 0
> arti_hss_proxy_connections_ok_total{nickname="ztest",action="forward"} 1
> ```