search

acquia / waterwheel.js

A generic JavaScript helper library to query and manipulate Drupal 8 via core REST and JSON API
https://github.com/acquia/waterwheel.js
232 stars 26 forks source link

Setting content type is case sensitive prior to axios@0.13 #30

Closed brendan-hurley closed 8 years ago

brendan-hurley commented 8 years ago

Hi,

Axios dependencies listed at 0.11. Prior to this commit in axios project header type 'Content-Type' is case sensitive as preceeding, but is currently set as 'Content-type'.

I noticed that the Content-Type request header value was being duplicated on 'POST' and 'PATCH' requests, like this: 'application/json, application/json' and this was causing my Drupal instance to fall over. This patch fixes that.

Suggest either updating the module or taking in this patch if appropriate.

Thanks, Brendan.

prestonso commented 8 years ago

Thanks for the pull request @creativebench! Could you reopen this PR against the 0.7.0 branch instead? That's where we're tracking current progress :)

mattgrill commented 8 years ago

@creativebench Additionally can you bump the minor version in package.json Thanks.