Body may need to be an opaque type

[justinwoo]

It might be necessarily to use an opaque type for body content in requests, for supporting things like formData.

[Cmdv]

Ha I've just hit this issue when trying to do a GET with headers (for auth token) but get the error

Error: TypeError: Failed to execute 'fetch' on 'Window': Request with GET/HEAD method cannot have body.

I guess you could make:

type Options =
  ( method :: Method
  , body :: Maybe String -- make this a Maybe
  , headers :: Headers
  , credentials :: Credentials
  , follow :: Int
  , redirect :: Redirect
  )

But that would be quite the breaking change, also think this is related 😂

[justinwoo]

that's such a weird error, since it's noted in MDN and others that body is simply ignored

[Cmdv]

yeah seems that window.fetch wants no body 😢
whatwg/fetch#551

This is a shame, I tried making body a Maybe locally, but when doing body: Nothing it still seems to add it!

what do you think would be a port of call for me to investigate? ps is this related to original issue ¯\_(ツ)_/¯

[justinwoo]

i'd just do the filtering here

purescript-milkis/src/Milkis.js

Line 5 in cedf354

return fetchImpl(url, options).catch(function(e) {

, seems like it's inevitable to do it at some level anyway

[Cmdv]

cool got it working, want me to create a PR?

[justinwoo]

sure, sounds good

[i-am-the-slime]

I am just looking at sending formData with Milkis.
From reading this, I get the vibe that this isn't supported right now. I'd be fine to just unsafeCoerce if that's a workaround.
Do you think this would work?

[justinwoo]

yeah i use unsafeCoerce atm. if you want to prevent some accidental reusages you might wrap this function, but i haven't had any problems with coercing form data to this field.