You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The noise_handshake code from the Chromium folks used an abstraction layer around the crypto operations to allow use of Ring crypto, BoringSSL Rust bindings, or Rust crypto. For the initial PR, all but the Rust crypto were deleted, but they were not inlined into the files calling these APIs.
These APIs are Ring-centric and should be cleaned up to be more generic and simpler to use, if we want to support bindings besides Rust crypto. If we do want to support other bindings, we may want to widen this layer so it can be used elsewhere, such as the Oak Restricted Kernel, etc. If not, these APIs should be inlined into the two files that use them.
The text was updated successfully, but these errors were encountered:
The noise_handshake code from the Chromium folks used an abstraction layer around the crypto operations to allow use of Ring crypto, BoringSSL Rust bindings, or Rust crypto. For the initial PR, all but the Rust crypto were deleted, but they were not inlined into the files calling these APIs.
These APIs are Ring-centric and should be cleaned up to be more generic and simpler to use, if we want to support bindings besides Rust crypto. If we do want to support other bindings, we may want to widen this layer so it can be used elsewhere, such as the Oak Restricted Kernel, etc. If not, these APIs should be inlined into the two files that use them.
The text was updated successfully, but these errors were encountered: