Hi all,
First, the monitor extension is awesome! If you are unfamiliar with it, I recommend giving it a look.
Second, I want to acknowledge that my questions (potential changes) may be pointing to things which would negatively impact existing implementations. As such, I'm fine with a blanket "No, we really shouldn't do that..." as a response.
That said, I've opened PR here: https://github.com/geoserver/geoserver/pull/4503 to show the questions/changes I'm curious about.
As I implemented a quick implementation of RequestDataListener, I noticed that successful OWS requests fired 4 messages through (failures sent 5). Of those messages, I think only two of them are useful and my changes would remove some of the (potentially) redundant calls.
Thanks in advance for any feedback,
Jim