tree d79466920cc1cbf3e61ef0ed1e376e1ccdccc523
parent 724985ffec9e284477cad3e3e2eace3a0a3da195
author Ed Tanous <etanous@nvidia.com> 1722024684 -0700
committer Ed Tanous <ed@tanous.net> 1722364419 +0000

Upgrade select alpn callback

nghttp2 has added a new callback[1] which solves the issue we were
seeing previously with const.  This cleans up some NOLINT markers, but
should have no functional changes

[1] https://nghttp2.org/documentation/nghttp2_select_alpn.html#c.nghttp2_select_alpn

Tested:

Curl shows HTTP2 upgrade works properly.

* ALPN, offering h2
* ALPN, offering http/1.1
* ALPN, server accepted to use h2
* Using HTTP2, server supports multiplexing
* Connection state changed (HTTP/2 confirmed)

Change-Id: Ic41754c58e6166aac22ed5afca5371661db959da
Signed-off-by: Ed Tanous <etanous@nvidia.com>