[GitHub] [trafficserver] ezelkow1 commented on pull request #8335: stats_over_http c++/remap/ipmap updates

[GitBox <gi...@apache.org>]

ezelkow1 commented on pull request #8335:
URL: https://github.com/apache/trafficserver/pull/8335#issuecomment-984723609


   Just some notes for myself. After some discussions we want to:
   
   - Remove the default of `_stats` when being used as a remap plugin, so that the remap itself becomes the endpoint
   - Still allow setting of the path via config file when running as a remap
   
   This would let you do 
   `map http://mystats.com http://whatever.com @plugin=stats_over_http.so`
   and just be able to hit stats without the _stats endpoint on a remap. It would still be optionally available via the config file though for things like ATC which would require this.
   
   We still want to leave the global usage as is, allowing a config or if the file doesnt exist use the param as the path, or default to _stats just because global usage needs its own path. On a remap though it will be config file based or nothing at all


-- 
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