oatpp / oatpp-openssl

OpenSSL adaptor for Oat++ applications
https://oatpp.io/
Apache License 2.0
13 stars 22 forks source link

Added getter for open ssl context #11

Closed MHaselmaier closed 2 years ago

MHaselmaier commented 2 years ago

Implements #10

lganzzzo commented 2 years ago

Hey @MHaselmaier ,

Even though I don't mind merging this PR, it won't resolve the #10, and the fix mentioned in #10 won't work for all cases.

The Request::getConnection() is not always the same object as was created by the connection provider. If you add ex.: connection monitor - Request::getConnection() will return a different proxy object instead of the original connection issues by connection provider.

Fix mentioned in #10 is more a hack than a way to fix it.

We have to think of another way better way to fix that.