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

[core] add raise_for_status to HttpResponse #16399

Merged
merged 4 commits into from
Jan 29, 2021

Conversation

iscai-msft
Copy link
Contributor

fixes #16336

Calling response.raise_for_status() will raise and HttpResponseError if status code >= 400, do nothing if not.

Adding this code now for the low level client, where we want to make a network request without automatically raising the error ourself, and instead allow users to raise the error themselves

Purposely does not include error model deserialization

@xiangyan99
Copy link
Member

Please update core version to 1.11.0.

@iscai-msft
Copy link
Contributor Author

/azp run python - storage - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@iscai-msft iscai-msft merged commit 94c2735 into Azure:master Jan 29, 2021
@iscai-msft iscai-msft deleted the raise_for_status branch January 29, 2021 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add "raise_for_status" in HttpResponse
3 participants