MeteredClient.execute()， The counter indicator increases by 2 each time

[flysun3344]

"sample.stop(timer)" in the finally code block should be placed in "if(time == null)"

[flysun3344]

The above problem is in feign-micrometer version 11.6

[vitalijr2]

No, it isn't.

If the sample.stop(timer) will be placed in this block we stop to call this method for any exception as there aren't any case when timer is null in finally block:

• we create timer in the end of happy path
• we create timer for any exception for all three cases including the highest level Exception, see MeteredClient, last catch of Exception

Of course we can leave if (timer == null) to work with Error and Throwable.
But here should be only one call of thestop: we should remove it from main block try-catch.