garydgregory commented on issue #129: Rename all classes prefixed with "H2..." to "Http2...". URL: https://github.com/apache/httpcomponents-core/pull/129#issuecomment-500118041 Hi @ok2c , TBH, I had not looked at HTTP/2 close enough to learn that "The string "h2" identifies the protocol where HTTP/2 uses Transport Layer Security (TLS) [TLS12]." according to https://http2.github.io/http2-spec/ Are you saying all classes that start with "H2" have to do with TLS? If yes, then we should not rename. If not, then we should rename to at least distinguish "h2" vs. HTTP/2 classes. If none of the classes are specific to TLS then we can rename them all. Please advise. If we have classes called "H2", should we have some called "H2C" to distinguish anything related to clear text? The timing of my PR is of coincidental timing, we are all busy folks. Thank you, Gary
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
