faysou commented on issue #413:
URL: 
https://github.com/apache/arrow-rs-object-store/issues/413#issuecomment-3649153993

   https://github.com/apache/arrow-rs-object-store/pull/576
   
   Here is another try as a PR. Likely more elegant than before. Chatting with 
Gemini it told me that aws-lc-rs was designed to be ring compatible, and that 
reexporting ring or aws-lc-rs as a common library could allow to factor the 
implementations.
   
   The implementation looks elegant to me. I've also better configured reqwest 
so it doesn't pull rustls-native by default.
   
   ```rust
   #[cfg(feature = "ring")]
   #[allow(unused_imports)]
   pub(crate) use ring as crypto;
   
   #[cfg(feature = "aws-lc")]
   #[allow(unused_imports)]
   pub(crate) use aws_lc_rs as crypto;
   ```


-- 
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]

Reply via email to