maskit commented on PR #12198: URL: https://github.com/apache/trafficserver/pull/12198#issuecomment-3005587412
I'm wondering how this kind of lists should be on ATS config. `proxy_protocol_allowlist` is just a comma-separated string (it's probably because the previous config format could not have a real list structure). And `per_client.connection.exempt_list` is a string too but a filename. I do see the benefit of having the list in a separate file, but I also think inconsistency should be avoided. Most settings that have filenames are ones that had different formats, and now we are trying to make all of them YAML. It may be a time to think about what should be separated from records.yaml. -- 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
