I was matching the regex that is accepted by the ops api. Per our discussion however it seems that there is a risk of breaking existing testing suites etc by restricting the use of underscores. I will change the regex to accept underscores and leave the sub for adding additional checks on other fields.
[ Full content available at: https://github.com/apache/trafficcontrol/pull/2806 ] This message was relayed via gitbox.apache.org for [email protected]
