Should we remove support for NPN?

[shinrich]

While looking through the ALPN logic, I notice that we still fallback to NPN if ALPN is not offered. Should we get rid of that support? It seems that the world has moved on.

I also noticed that our internal data structures are all named around npn. May want to due a name update to avoid confusion in future maintenance.

[maskit]

👍

[bryancall]

yes!

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

[maskit]

Can we do this on 10.0 release? Should we deprecate it first and remove it on 11.0 release?

https://curl.se/mail/lib-2022-02/0110.html

[maskit]

ATS 10 Hackathon: OpenSSL 3 still supports NPN. We may need to keep the support for a while for old Android.

[zwoop]

Maybe mark it deprecated though, which opens the door for removal for future ATS releases.