[GitHub] ocket8888 commented on issue #3312: Disallow underscore in xmlId

[GitBox <gi...@apache.org>]

ocket8888 commented on issue #3312: Disallow underscore in xmlId
URL: https://github.com/apache/trafficcontrol/issues/3312#issuecomment-461886444
 
 
   To be maybe a bit more exhaustive, the TO API considers `aB&%=/#@$^*()[]!{}+\\|`~` to be a perfectly valid xml_id, and (I think) only the `a` ought to be allowed. Maybe `B` if we implement casefolding for SNI quirks in SSL implementations. Additionally, the TP validator catches none of that either, and will also let you submit a form with a `.` in it (though that fails because the API rejects it) but I do believe checks for `_`

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services