bneradt commented on PR #11273: URL: https://github.com/apache/trafficserver/pull/11273#issuecomment-2069892350
> I'd say relying on reason strings is a bad custom even for testing because such a test is not runnable on H2 and H3. I don't think we should open wide the door for making tests that won't work for H2/3. Agreed. My separate change made the plugin code handle an absent reason string: https://github.com/apache/trafficserver/pull/11272 That said, is there a problem having a reason string for 418? -- 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]
