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.

Sign up for GitHub

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

Jump to bottom

do not check body in pyramid context #185

Merged

inkhey merged 1 commit into develop from fix/184_do_not_load_body_for_check_in_pyramid_context

Jun 7, 2019

Contributor

inkhey commented Jun 6, 2019

related to #184


          do not check body in pyramid context

7b93177

Member

lebouquetin commented Jun 6, 2019 •

edited by inkhey

Loading

TODO @inkhey :

create hapic issue for HTTP 400 in case of empty body : Raise 400 error in case of content_type json but empty body #186
create tracim frontend issue for default value to be {} for JSON requests instead of empty body : Frontend should return '{}' by default instead of '' to avoid invalid json. tracim/tracim#1871
reference each issue to the other
merge this (its approved for now)

lebouquetin self-requested a review

June 6, 2019 15:44

lebouquetin approved these changes

View reviewed changes

inkhey merged commit 0e05b37 into develop

inkhey deleted the fix/184_do_not_load_body_for_check_in_pyramid_context branch

July 21, 2020 07:13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet