search

strongloop / strong-remoting

Communicate between objects in servers, mobile apps, and other servers.
www.strongloop.com
Other
105 stars 93 forks source link

update check for turning off sloppy coercion when body request is application/json #266

Closed ebourmalo closed 8 years ago

ebourmalo commented 8 years ago

Check and fix if the content-type header is an application/json request type. Allow now the charset definition in the content-type (i.e 'application/json;charset=UTF-8').

Connect to #267

slnode commented 8 years ago

Can one of the admins verify this patch? To accept patch and trigger a build add comment ".ok\W+to\W+test."

ghislainf commented 8 years ago

:+1:

rmg commented 8 years ago

@slnode ok to test

bajtos commented 8 years ago

Hello @ebourmalo, sorry for taking so long to respond. Could you please rebase your patch on top of the current master and add a unit-test to demonstrate your code works as intended and prevent regressions in the future?

slnode commented 8 years ago

Can one of the admins verify this patch? To accept patch and trigger a build add comment ".ok\W+to\W+test."

bajtos commented 8 years ago

@slnode ok to test

slnode commented 8 years ago

Can one of the admins verify this patch?

slnode commented 8 years ago

Can one of the admins verify this patch?

slnode commented 8 years ago

Can one of the admins verify this patch?

bajtos commented 8 years ago

Closing in favour of https://github.com/strongloop/strong-remoting/pull/350