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

Support newer httpx versions #866

Merged
merged 8 commits into from
Feb 4, 2022

Conversation

nozik
Copy link
Contributor

@nozik nozik commented Jan 18, 2022

Description

Support the latest versions of httpx, currently only versions lower than 0.19.0 are supported.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

Adapting the existing tests of the previous versions of httpx.

Does This PR Require a Core Repo Change?

  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@nozik nozik requested a review from a team January 18, 2022 06:06
@nozik nozik force-pushed the support_newer_httpx_versions branch from a7b4175 to 30831a8 Compare January 18, 2022 06:29
@lzchen
Copy link
Contributor

lzchen commented Jan 18, 2022

@jomasti
Is it alright if you were to be added as a maintainer for this instrumentation? This was what I was adhering to in this comment

@jomasti
Copy link
Contributor

jomasti commented Jan 19, 2022

@lzchen I don't have the time to dedicate to any maintenance. It looks like this is being handled okay, and I wouldn't be providing anything extra.

@nozik nozik force-pushed the support_newer_httpx_versions branch from 30831a8 to e041297 Compare January 19, 2022 05:56
@owais owais enabled auto-merge (squash) January 28, 2022 09:29
auto-merge was automatically disabled January 28, 2022 12:18

Head branch was pushed to by a user without write access

@nozik nozik force-pushed the support_newer_httpx_versions branch from 0703336 to a03173f Compare January 28, 2022 12:18
@owais owais enabled auto-merge (squash) January 28, 2022 16:08
auto-merge was automatically disabled February 2, 2022 07:49

Head branch was pushed to by a user without write access

@nozik nozik force-pushed the support_newer_httpx_versions branch 3 times, most recently from fb685b0 to 37642d6 Compare February 4, 2022 12:42
@nozik nozik force-pushed the support_newer_httpx_versions branch from 37642d6 to fb0e0fb Compare February 4, 2022 13:11
@srikanthccv
Copy link
Member

@nozik to fix the docs issue please try adding an exception httpx.Response here https://github.com/open-telemetry/opentelemetry-python-contrib/blob/main/docs/nitpick-exceptions.ini#L22-L27.

@nozik
Copy link
Contributor Author

nozik commented Feb 4, 2022

@lonewolf3739 - that did the trick, thank you!
@ocelotl @owais PR is ready for merge. Thanks!

@lzchen lzchen merged commit 0b9e96d into open-telemetry:main Feb 4, 2022
@nozik nozik deleted the support_newer_httpx_versions branch February 4, 2022 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants