Skip to content

add support mtls #207

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
nwomib opened this issue Jan 20, 2025 · 0 comments
Open

add support mtls #207

nwomib opened this issue Jan 20, 2025 · 0 comments
Labels
feature A new functionality

Comments

@nwomib
Copy link

nwomib commented Jan 20, 2025

Please add support mTLS certs

@themilchenko themilchenko added the feature A new functionality label Jan 20, 2025
olegrok added a commit that referenced this issue Jun 6, 2025
Different libraries can use FFI OpenSSL bindings.
E.g. cartridge already uses it:
https://github.com/tarantool/cartridge/blob/master/cartridge/sslsocket.lua

Right now there is no any significant difference in cdef. But in
future it can be different (e.g. after implementation of #207).
Since we use "struct SSL_METHOD {} SSL_METHOD" definition
it can cause errors like 'attempt to
redefine 'SSL_METHOD'" that lead to the case then pcall silently
handle an error but some symbols won't be defined.

This patch fixes such case.
That's only relevant for struct definitions. Functions and typedefs can be redefined.
olegrok added a commit that referenced this issue Jun 9, 2025
Different libraries can use FFI OpenSSL bindings.
E.g. cartridge already uses it:
https://github.com/tarantool/cartridge/blob/master/cartridge/sslsocket.lua

Right now there is no any significant difference in cdef. But in
future it can be different (e.g. after implementation of #207).
Since we use "struct SSL_METHOD {} SSL_METHOD" definition
it can cause errors like 'attempt to
redefine 'SSL_METHOD'" that lead to the case then pcall silently
handle an error but some symbols won't be defined.

This patch fixes such case.
That's only relevant for struct definitions. Functions and typedefs can be redefined.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new functionality
Projects
None yet
Development

No branches or pull requests

2 participants