rutgerclaes opened a new issue, #2888:
URL: https://github.com/apache/iceberg-rust/issues/2888
### Is your feature request related to a problem or challenge?
reqwest is pinned with default-features = false, so iceberg-catalog-rest has
no TLS backend and https:// catalog URIs fail (e.g. against Lakekeeper). The
only workaround is a direct reqwest dependency with a TLS feature in the
consumer's manifest. Then rust's feature unification solves the issue for you.
But this is a very implicit and brittle solution.
### Describe the solution you'd like
Passthrough features named after reqwest's, with the OS trust store
variant on
by default; `default-features = false` still gives a TLS-free build:
```toml
[features]
default = ["rustls-tls-native-roots"]
rustls-tls = ["reqwest/rustls-tls"]
rustls-tls-native-roots = ["reqwest/rustls-tls-native-roots"]
```
Implemented here, with docs and a --no-default-features CI check:
https://github.com/rutgerclaes/iceberg-rust/tree/tls-support-reqwest
### Willingness to contribute
I would be willing to contribute to this feature with guidance from the
Iceberg Rust community
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
