fix: update axios and fix default value for 'data'

jhpedemonte commented 2 years ago

Update axios to latest, in order to pick up vulnerability fix
Don't default to null for POST data, since axios will JSON.stringify that as "null" (instead of sending an empty payload, as before)

I verified locally with my app. When only updating axios to 0.21.4 (using yarn's resolutions feature), my app was broken as one of my POST requests was sending "null" as the payload body. With this PR, it's working as expected: an empty payload instead.

Fixes https://github.com/infinitered/apisauce/issues/276

chakrihacker commented 2 years ago

Looks good to me. Thanks for the pr

infinitered-circleci commented 2 years ago

:tada: This PR is included in version 2.1.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

infinitered / apisauce

fix: update axios and fix default value for 'data' #277