New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Respect transaction_ignore_urls
and transaction_ignore_user_agents
when creating transactions in the spring webflux instrumentation
#2515
Conversation
…` when creating transactions in the spring webflux instrumentation
👋 @tobiasstadler Thanks a lot for your contribution! It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it. Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it. |
@@ -110,6 +112,22 @@ public static boolean isServletTransaction(ServerWebExchange exchange) { | |||
return Boolean.TRUE == exchange.getAttributes().get(SERVLET_TRANSACTION); | |||
} | |||
|
|||
public static boolean isExcluded(WebConfiguration webConfiguration, RequestPath requestPath, @Nullable String userAgentHeader) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You/We should probably consolidate the isExcluded
variants in a follow up PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have just opened #2524 for that, once this is merged I'll remove the duplication.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank You!
@elasticmachine run elasticsearch-ci/docs |
Thank You! |
What does this PR do?
Fixes #2514
Checklist