zrhoffman commented on code in PR #7645:
URL: https://github.com/apache/trafficcontrol/pull/7645#discussion_r1262842529
##########
.github/actions/to-integration-tests/cdn.json:
##########
@@ -1,8 +1,15 @@
{
+ "client_certificate_authentication" : {
+ "root_certificates_directory" : "/etc/pki/tls/traffic_ops/"
+ },
"disable_auto_cert_deletion": false,
"use_ims": true,
"role_based_permissions": true,
"traffic_ops_golang": {
+ "tls_config": {
+ "MinVersion": 769,
Review Comment:
This was suggested before in
https://github.com/apache/trafficcontrol/pull/4573#discussion_r400346222:
> 769, 770, etc are not very intuitive -- would it be better to use a string
or float that we translate into `tls.VersionTLS11` from the `tls` package in
the stdlib? If 770 is basically required in order for TO to even communicate w/
Riak, should we just make 770 the default, allowing people to change it if they
know what they're doing?
--
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: issues-unsubscr...@trafficcontrol.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
