Closed davidporos92 closed 4 years ago
The response_errors also included the route label, which can lead to a lots of data. This is pretty similar to #65
response_errors
Removes the route from the response_errors. Also, I've updated the readme to align with the current responses.
Avoid creating an excessive number of metrics.
This part of the code is covered by test.
good catch, thanks @davidporos92
Will verify and merge asap
thanks again @davidporos92 !
Motivation
The
response_errors
also included the route label, which can lead to a lots of data. This is pretty similar to #65What
Removes the route from the
response_errors
. Also, I've updated the readme to align with the current responses.Why
Avoid creating an excessive number of metrics.
Verification Steps
This part of the code is covered by test.
Checklist:
Progress