maskit commented on code in PR #11673:
URL: https://github.com/apache/trafficserver/pull/11673#discussion_r1715626770
##########
src/traffic_server/traffic_server.cc:
##########
@@ -2215,6 +2215,21 @@ main(int /* argc ATS_UNUSED */, const char **argv)
REC_ReadConfigInteger(num_of_udp_threads, "proxy.config.udp.threads");
}
+ ats_scoped_str server_ports(255);
+ *server_ports = '\0';
+ REC_ReadConfigString(server_ports, "proxy.config.http.server_ports", 255);
+ bool quic_server_port = strstr(server_ports, "/quic/") != server_ports;
+
+#if TS_USE_QUIC == 0
+ if (quic_server_port) {
+ Fatal("proxy.config.http.server_ports listening for quic but
traffic_server wasn't built with quic support.");
Review Comment:
This check could be done as a part of this block, if we want to have a
specialized message for QUIC (the block already has code for unrecognized
options).
https://github.com/apache/trafficserver/blob/fc5918e5a8bb1eb88b98689c463c3574a64e8bef/src/records/RecHttp.cc#L367-L463
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
