duke8253 commented on pull request #7673: URL: https://github.com/apache/trafficserver/pull/7673#issuecomment-814265128
> Changing tag is probably ok, but changing `Warning` to `Debug` may be not. Warnings are printed regardless of `diags.debug.enabled` and `diags.debug.tags`, and those would be disappeared on prod. If those warnings are not really important on prod then I'm ok with making them debug log, but users watching the warnings would be surprised. We might want to mark this incompatible. I didn't check how `SSLError` works but I have the same concern on it as well. I thought about that too, but the message logged in Diags also seems reasonable to be in debug logs as well. We will have our ops team test this out for a few days and see what they prefers, I'll make adjustments according to their comments. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
