trafficserver icon indicating copy to clipboard operation
trafficserver copied to clipboard

Published 20 hours ago • verified publisher icon apache

Should we remove support for NPN?

Open shinrich opened this issue 3 years ago • 7 comments

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.

shinrich avatar Jun 23 '21 16:06 shinrich

👍

maskit avatar Jun 23 '21 23:06 maskit

yes!

bryancall avatar Jun 28 '21 23:06 bryancall

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.

github-actions[bot] avatar Mar 15 '23 01:03 github-actions[bot]

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 avatar Mar 15 '23 06:03 maskit

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

maskit avatar Feb 07 '24 17:02 maskit

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

zwoop avatar Feb 07 '24 17:02 zwoop

https://github.com/apache/trafficserver/pull/11045

maskit avatar Feb 07 '24 18:02 maskit