Copilot commented on code in PR #13121:
URL: https://github.com/apache/trafficserver/pull/13121#discussion_r3184880752
##########
tools/hrw4u/src/tables.py:
##########
@@ -47,7 +47,6 @@
"inbound.resp.": MapParams(target=HeaderOperations.OPERATIONS, add=True,
validate=Validator.http_header_name(), sections={SectionType.READ_RESPONSE,
SectionType.SEND_RESPONSE}),
"inbound.status.reason": MapParams(target="set-status-reason",
validate=Validator.quoted_or_simple(), sections=HTTP_SECTIONS),
"inbound.status": MapParams(target="set-status",
validate=Validator.range(0, 999), sections=HTTP_SECTIONS),
Review Comment:
This removes `inbound.url.*` from the assignment map, but the updated
fixtures only cover the new `outbound.url.*` spelling. There still isn't a
negative test that `inbound.url.host = ...` now fails, so the core behavior
this PR is meant to enforce could regress without any test catching it.
--
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]
