Fix reserved consistentHashQueryParams entries causing internal server error instead of client error

[ocket8888]

This PR fixes #6385 by causing Traffic Ops to return the correct error code (400 Bad Request) and an alert describing what went wrong when a client attempts to create or modify a Delivery Service such that it would an entry in its Consistent Hashing Query string Parameters that is reserved for use by Traffic Router.

---

Which Traffic Control components are affected by this PR?

• Traffic Ops

What is the best way to verify this PR?

topost -kp deliveryservices '{       
        "active": false,
        "cdnId": 2,
        "consistentHashQueryParams": ["format"],
        "displayName": "test",
        "dnsBypassCname": "test",
        "dscp": 1,
        "geoLimit": 1,
        "geoLimitCountries": "US,CA",
        "geoProvider": -7,
        "initialDispersion": 1,
        "ipv6RoutingEnabled": true,
        "logsEnabled": true,
        "missLat": 0,
        "missLong": 0,
        "multiSiteOrigin": false,
        "orgServerFqdn": "test.quest",
        "protocol": 7,
        "qstringIgnore": 5,
        "rangeRequestHandling": 0,
        "regionalGeoBlocking": false,
        "tenantId": 1,
        "typeId": 1,
        "xmlId": "test"
}'

If this is a bugfix, which Traffic Control versions contained the bug

• not sure

PR submission checklist

• This PR has tests
• This PR has documentation
• This PR has a CHANGELOG.md entry
• This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

[codecov]

Codecov Report

Merging #7405 (03197b7) into master (904e240) will decrease coverage by 0.01%.
The diff coverage is 11.11%.

@@             Coverage Diff              @@
##             master    #7405      +/-   ##
============================================
- Coverage     27.12%   27.12%   -0.01%     
  Complexity       98       98              
============================================
  Files           685      685              
  Lines         77414    77421       +7     
  Branches         90       90              
============================================
- Hits          20999    20998       -1     
- Misses        54423    54431       +8     
  Partials       1992     1992              

Flag	Coverage Δ
golib_unit	48.40% <ø> (-0.01%)	⬇️
grove_unit	4.60% <ø> (ø)
t3c_unit	5.32% <ø> (ø)
traffic_monitor_unit	21.28% <ø> (ø)
traffic_ops_integration	69.42% <ø> (ø)
traffic_ops_unit	22.18% <11.11%> (-0.01%)	⬇️
traffic_stats_unit	10.14% <ø> (ø)
unit_tests	23.69% <11.11%> (-0.01%)	⬇️
v3	57.79% <ø> (ø)
v4	79.18% <ø> (ø)
v5	78.63% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
lib/go-tc/deliveryservices.go	86.94% <ø> (ø)
...fic_ops_golang/deliveryservice/deliveryservices.go	17.67% <11.11%> (-0.06%)	⬇️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more