Skip to content
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

SSL support for Windows when compiling libpcap? #1299

Closed
daluu opened this issue Mar 24, 2024 · 2 comments
Closed

SSL support for Windows when compiling libpcap? #1299

daluu opened this issue Mar 24, 2024 · 2 comments

Comments

@daluu
Copy link

daluu commented Mar 24, 2024

I was just looking through the logic for SSL detection for libpcap build process under https://github.com/the-tcpdump-group/libpcap/blob/master/configure.ac#L1740-L1893, to see how the process goes about detecting SSL support so that the compiled libpcap and rpcapd would contain SSL/TLS support.

I didn't observe any explicit logic for SSL detection there that seems specific to Windows. So I'm just wondering is there SSL detection and support for building libpcap with SSL/TLS on Windows? If not natively by default, what might be the extra steps needed to build for Windows with SSL support?

If it's already there to support Windows, can you point me to the section where detection on Windows would apply? Maybe I overlooked things.

In any case, it might be good to mention this on the README for building on Windows - whether currently supported or not, and how to ensure support for it, if supported.

@guyharris
Copy link
Member

If not natively by default, what might be the extra steps needed to build for Windows with SSL support?

  1. Either get an OpenSSL/LibreSSL library for Windows, complete with an SDK (headers and .libs) and add stuff (similar to what's done for finding the packet32.dll headers and library) to specify where to find the SDK, or find some other TLS library, including the one that I think Microsoft provides, and add code to support that.
  2. Proceed from there.

infrastation added a commit that referenced this issue Apr 22, 2024
@infrastation
Copy link
Member

Documentation updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants