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 missing HTTP status codes #1920

Merged
merged 5 commits into from
May 26, 2017
Merged

Add missing HTTP status codes #1920

merged 5 commits into from
May 26, 2017

Conversation

0xd669
Copy link
Contributor

@0xd669 0xd669 commented May 25, 2017

What do these changes do?

Extend list of web_exceptions, add HTTPUnprocessableEntity, HTTPFailedDependency, HTTPInsufficientStorage status codes

Are there changes in behavior for the user?

Use HTTPUnprocessableEntity, HTTPFailedDependency, HTTPInsufficientStorage to respond with appropriate status codes.

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt
  • Add a new entry to CHANGES.rst

@codecov-io
Copy link

codecov-io commented May 25, 2017

Codecov Report

Merging #1920 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1920      +/-   ##
==========================================
+ Coverage   97.05%   97.05%   +<.01%     
==========================================
  Files          37       37              
  Lines        7604     7610       +6     
  Branches     1328     1328              
==========================================
+ Hits         7380     7386       +6     
  Misses        101      101              
  Partials      123      123
Impacted Files Coverage Δ
aiohttp/web_exceptions.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f64de97...eeff02e. Read the comment docs.

@asvetlov asvetlov merged commit ec9dd3c into aio-libs:master May 26, 2017
@asvetlov
Copy link
Member

Thanks for contribution!

@lock
Copy link

lock bot commented Oct 29, 2019

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.

@lock lock bot added the outdated label Oct 29, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants