search

tom-draper / api-analytics

Lightweight monitoring and analytics for API frameworks.
https://apianalytics.dev
MIT License
179 stars 26 forks source link

rust actix analytics library uses native-tls by default, unable to use rustls #5

Closed 1zun4secondary closed 1 year ago

1zun4secondary commented 1 year ago

Hello.

It is not possible to stop reqwest from using native-tls. reqwest = { version = "0.11", features = ["json", "blocking"] } defined here: https://github.com/tom-draper/api-analytics/blob/main/analytics/rust/actix/analytics/Cargo.toml

I prefer to use rustls-tls, which can be defined like this: reqwest = { version = "0.11", default-features = false, features = ["json", "blocking", "rustls-tls"] }

A possible solution to this issue would add features to the analytics crate like this:

reqwest = { version = "0.11", default-features = false, features = ["json", "blocking"] }

...

[features]
default = ["native-tls"]

rustls = ["reqwest/rustls-tls"]
native-tls = ["reqwest/default-tls"]