Skip to content

updated _form_response function#685

Merged
markstory merged 1 commit intogetsentry:masterfrom
beliaev-maksim:mbeliaev/rework_form_response
Oct 23, 2023
Merged

updated _form_response function#685
markstory merged 1 commit intogetsentry:masterfrom
beliaev-maksim:mbeliaev/rework_form_response

Conversation

@beliaev-maksim
Copy link
Collaborator

updated _form_response function with more native handling by utilizing the correct HTTPResponse from http rather than from urllib3

…ing the correct `HTTPResponse` from `http` rather than from `urllib3`
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Files Coverage Δ
responses/__init__.py 100.00% <100.00%> (ø)

📢 Thoughts on this report? Let us know!.

@markstory markstory merged commit 87db9df into getsentry:master Oct 23, 2023
@beliaev-maksim beliaev-maksim deleted the mbeliaev/rework_form_response branch October 23, 2023 14:45
beliaev-maksim added a commit to beliaev-maksim/responses that referenced this pull request Nov 14, 2023
* Added comment why this is the behaviour we have
* Added run on the test to fix coverage
markstory pushed a commit that referenced this pull request Nov 14, 2023
* Revert socket change (#685)
* Added comment why this is the behaviour we have
* Added run on the test to fix coverage
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.

2 participants