Allow tagsBasedOnJoinPoint to override extraTags with CountedAspect

[kinddevil]

When overwriting the extraTags in Aspect, the behavior is different - extraTags can be replaced in the TimedAspect but in CountedAspect as below.

I have a function with extra tags like

@Timed(extraTags = {"extra", "A"})
@Counted(extraTags = {"extra", "B"})
public void recordMetrics() {
...
}

When I overwrite the extra tag in TimedAspect

@Bean
  public TimedAspect timedAspect(MeterRegistry registry) {
    return new TimedAspect(registry, p -> Arrays.asList(Tag.of("extra", "AA"));
  }

The extra changed to AA successfully.
But when I use some similar code in CountedAspect it failed.

  @Bean
  public CountedAspect countedAspect(MeterRegistry registry) {
    return new CountedAspect(registry,  p -> Arrays.asList(Tag.of("extra", "BB"));
  }

The value is still B.

It is because of the different orders when applying the ProceedingJoinPoint tags

micrometer/micrometer-core/src/main/java/io/micrometer/core/aop/CountedAspect.java

Line 148 in 939e0e4

counter(pjp, counted)

and

micrometer/micrometer-core/src/main/java/io/micrometer/core/aop/TimedAspect.java

Line 192 in 939e0e4

.tags(tagsBasedOnJoinPoint.apply(pjp))

.

Can we make them in the same order?

[kinddevil]

There is a fix for the issue #2462, can someone take a look? Thank you!

[kinddevil]

Hi @shakuzen, do you have any suggestions for this issue?