Skip to content

Added DTLS and ALPN client support. - #446

Closed
denandz wants to merge 2 commits into
nmap:masterfrom
denandz:master
Closed

Added DTLS and ALPN client support.#446
denandz wants to merge 2 commits into
nmap:masterfrom
denandz:master

Conversation

@denandz

@denandz denandz commented Jul 5, 2016

Copy link
Copy Markdown

This change adds support Datagram Transport Layer Security (DTLS) when creating UDP connections with ncat. Application-Layer Protocol Negotiation (ALPN) support has also been added, allowing the user to specify an ALPN string when connecting the SSL services. For example, connecting to an HTTP2 compatible web server and supplying the 'h2' ALPN string.

@dmiller-nmap

Copy link
Copy Markdown

@denandz Thanks for this contribution! It looks like a fun and exciting addition to our program. Before we can review it, please ensure that it passes our build test. Specifically, you should guard sections of code that depend on OpenSSL with #ifdef HAVE_OPENSSL.

@denandz

denandz commented Jul 6, 2016

Copy link
Copy Markdown
Author

@dmiller-nmap I double checked the code and it looks like all the relevant guards are there. The build test seems to have a problem locating DTLS_client_method(). What version of openssl is the build using? SSL_CTX_set_alpn_protos() is also only available in more recent versions of openssl. I guess I could attempt to get some voodoo working to detect the openssl version and add in additional guards? Please advise. Roger that, added in checks for the openssl version.

Also, in the travis-ci non-ssl builds are being built with openssl. The './configure $SSL_FLAG' doesn't seem to work as expected. On my system:

$ export SSL_FLAG="--without-ssl"
$ ./configure $SSL_FLAG
{snip}
Configured with: ndiff zenmap nping openssl lua ncat
Configured without: localdirs nmap-update
Type make (or gmake on some *BSD machines) to compile.
$

Similar is reflected in the travis-ci build logs. (EG: https://travis-ci.org/nmap/nmap/jobs/142393838)

@dmiller-nmap

Copy link
Copy Markdown

@denandz Thanks for pointing that out about the travis builds! I just pushed a change to correctly configure without OpenSSL. Hope to be able to review this pull soon.

@denandz

denandz commented Aug 12, 2016

Copy link
Copy Markdown
Author

@dmiller-nmap any update on the pull ?

@denandz

denandz commented May 8, 2017

Copy link
Copy Markdown
Author

McFly?

@denandz

denandz commented May 31, 2017

Copy link
Copy Markdown
Author

@denandz

denandz commented Jul 7, 2017

Copy link
Copy Markdown
Author

image

@denandz

denandz commented Jul 27, 2017

Copy link
Copy Markdown
Author

@dmiller-nmap

dmiller-nmap commented Jul 29, 2017 via email

Copy link
Copy Markdown

@nmap-bot nmap-bot closed this in fdfc367 Jul 29, 2017
@denandz

denandz commented Jul 30, 2017

Copy link
Copy Markdown
Author

Thanks @dmiller-nmap! You can use Denis Andzakovic as the credit name in the change log. Thanks for merging these features ^_^

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants