You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/05/08 23:37:36 UTC

[GitHub] [trafficserver] maskit commented on pull request #8830: Change ip-allow debug tag to ip_allow.

maskit commented on PR #8830:
URL: https://github.com/apache/trafficserver/pull/8830#issuecomment-1120509113

   I'm fine with making this change on 10.0, but have a little concern on backporting it to 9.x. Although it's just a debug tag name change, I think it's technically an incompatible change. I personally don't mind, but thought it's better to make a consensus.
   
   I found  several other tag names that don't follow the new rule. I guess those have to be changed as well when we use the new debug interface. If we are going to change ip-allow we should probably change the others.
   
   Here's the incomplete list:
   - log-file
   - log-filter-compare
   - log-agg
   - log-logbuffer
   - log-config
   - log-config-transfer
   - log-api-mutex
   - Trie::Insert


-- 
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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org