Stripe metrics include ids in the group names about newrelic-ruby-agent

[workato-integration]

https://new-relic.atlassian.net/browse/NR-273271

[tannalynn]

Thanks for the feedback on the strip instrumentation! That is a good point for us to look into, I can definitely see how that could lead to a large number of metric names. Currently we use this to create the name

        def metric_name(event)
          "Stripe#{event.path}/#{event.method}"
        end

So we'll need to take a second look here to see the best way to extract some of the more useful information out of the path while not including those ids, since removing event.path completely would leave metric names that simply look like "Stripe/get".

[jdelStrother]

Yeah, agreed. All the stripe ids I know of are 2-3 letters (eg cus, sub, in), then an underscore, then a bunch of letters/numbers - I wondered if you might be able to do some regex magic to scrub those.
(yes yes, and then you have two problems 😛)

[kford-newrelic]

@jdelStrother thank you for giving us this feedback - our engineers agreed with you that our current instrumentation behavior is not desirable - we plan to take this up during the next quarter.

However, it was also agreed that, though not desirable, the current behavior isn't a bug per se, so we'll be treating this as an enhancement (which likely won't matter to you)

[kford-newrelic]

Overruled by engineers. Re-classifying as a bug