maskit commented on PR #11902: URL: https://github.com/apache/trafficserver/pull/11902#issuecomment-2524219915
I thought the implementation should be changed instead because the documentation is what users see and new users who didn't use ATS 9 would be required to update their settings. But, unfortunately, convert2yaml.py disagrees. https://github.com/apache/trafficserver/blob/dc59c4c1fc8243f60601127442ea6835c434ffdd/tools/records/convert2yaml.py#L48-L49 I think the best way is to support the both names (`.value` and `.enabled`) for now, and introduce new name (`.mode`) to address the inconsistency between session_cache and origin_session_cache. -- 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-unsubscr...@trafficserver.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
