Skip to content
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

Add allowUrls as a config option for web instrumentation #4899

Open
jairo-mendoza opened this issue Aug 2, 2024 · 1 comment · May be fixed by #4938
Open

Add allowUrls as a config option for web instrumentation #4899

jairo-mendoza opened this issue Aug 2, 2024 · 1 comment · May be fixed by #4938

Comments

@jairo-mendoza
Copy link

jairo-mendoza commented Aug 2, 2024

NB: Before opening a feature request against this repo, consider whether the feature should/could be implemented in the other OpenTelemetry client libraries. If so, please open an issue on opentelemetry-specification first.

Is your feature request related to a problem? Please describe.

Not a problem, but more of a nice to have that could help with handling restrictions without having to disallow specific urls explicitly.

Describe the solution you'd like

A config option holding an array of urls or a regex for both the FetchInstrumentation and XMLHttpRequestInstrumentation classes. Requests will then be checked against allowUrls, not sending a span if there isn’t a match. Similar solution can be implemented for DocumentLoadInstrumentation over on the contrib repo.

Copy link

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

@github-actions github-actions bot added the stale label Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant