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 exclude urls feature to HTTPX instrumentation #539

Open
jomasti opened this issue Jun 8, 2021 · 5 comments · May be fixed by #1900
Open

Add exclude urls feature to HTTPX instrumentation #539

jomasti opened this issue Jun 8, 2021 · 5 comments · May be fixed by #1900
Assignees
Labels
feature-request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@jomasti
Copy link
Contributor

jomasti commented Jun 8, 2021

Is your feature request related to a problem?
This is to add the exclude urls functionality to the HTTPX instrumentation. This is a follow-up for #461.

Describe the solution you'd like
The solution should allow for skipping instrumentation for a list of excluded URLs similar to the current solutions.

@srikanthccv srikanthccv added help wanted Extra attention is needed good first issue Good for newcomers labels Apr 21, 2022
@zqumei0
Copy link

zqumei0 commented Aug 8, 2022

Hi, can you assign this issue to me? Thanks in advance!

@srikanthccv
Copy link
Member

Feel free to ask questions here or on Slack channel/DM if you need any help.

@macieyng
Copy link
Contributor

macieyng commented Aug 2, 2023

Hi @zqumei0 👋 Is there any progress? Did you worked on it? Do you need any support? I'm willing to work on this too.

@zqumei0
Copy link

zqumei0 commented Aug 2, 2023

Hey @macieyng! Yes I have made some progress with this, but I can sync up with you on Slack with what I currently have and see how to move forward if you're good with that.

@macieyng
Copy link
Contributor

macieyng commented Aug 3, 2023

Sure. My nick on slack is the same as here.

Meanwhile, please create a pull request (it can be a draft) with what you've got so far and link it with this issue.

@zqumei0 zqumei0 linked a pull request Aug 4, 2023 that will close this issue
11 tasks
zqumei0 added a commit to zqumei0/opentelemetry-python-contrib that referenced this issue Aug 8, 2023
zqumei0 added a commit to zqumei0/opentelemetry-python-contrib that referenced this issue Aug 26, 2023
zqumei0 added a commit to zqumei0/opentelemetry-python-contrib that referenced this issue Sep 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants