envoyproxy / envoy-openssl

Envoy OpenSSL extensions
Apache License 2.0
46 stars 44 forks source link

enable async mode if the used OpenSSL engine allows it #4

Closed rojkov closed 4 years ago

rojkov commented 5 years ago

This patch allows the current OpenSSL engine used for a connection to work in asynchronous mode if the engine supports it. Otherwise the code falls back to synchronous mode (this is the default).

rojkov commented 5 years ago

@bdecoste ping

rojkov commented 5 years ago

@ipuustin Thanks! I've addressed your comments.

rojkov commented 5 years ago

Updated to follow the development of the upstream TLS (replaced booleans with an enum SocketState after https://github.com/envoyproxy/envoy/pull/6326/commits/5db871ccb8bfc4486ff478dd86d4d6c379ea0cfe).

@bdecoste please take a look.

rojkov commented 4 years ago

rebased to the latest master and tested async handling actually works with Envoy v1.12.0 + OpenSSL

rojkov commented 4 years ago

@bdecoste could you please have a look?

ipuustin commented 4 years ago

Looks good to me now.