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

strip Request mimetype #553

Merged
merged 2 commits into from
Apr 11, 2023
Merged

strip Request mimetype #553

merged 2 commits into from
Apr 11, 2023

Conversation

jitka
Copy link
Contributor

@jitka jitka commented Apr 11, 2023

Strip valid commends to prevent

openapi_core.validation.response.exceptions.DataValidationError: DataValidationError: Content for the following mimetype not found: application/json; charset=utf-8. Valid mimetypes: ['application/json']"

@codecov
Copy link

codecov bot commented Apr 11, 2023

Codecov Report

Merging #553 (a6f5082) into master (7456c4b) will not change coverage.
The diff coverage is n/a.

❗ Current head a6f5082 differs from pull request most recent head 218d07f. Consider uploading reports for the commit 218d07f to get more accurate results

@@           Coverage Diff           @@
##           master     #553   +/-   ##
=======================================
  Coverage   90.30%   90.30%           
=======================================
  Files         124      124           
  Lines        3681     3681           
  Branches      442      442           
=======================================
  Hits         3324     3324           
  Misses        302      302           
  Partials       55       55           
Impacted Files Coverage Δ
openapi_core/contrib/requests/requests.py 88.23% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@p1c2u p1c2u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added test. LGTM

@p1c2u
Copy link
Collaborator

p1c2u commented Apr 11, 2023

@jitka thank you for the contribution

@p1c2u p1c2u merged commit d5f9c30 into python-openapi:master Apr 11, 2023
7 checks passed
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.

None yet

2 participants