request-api allow the http method to be specified

[kumavis]

via options.method = 'GET'

[kumavis]

not sure if test failure is related...

  1 failing
  1) .files callback API .add .createAddStream stream of valid files and dirs:
     Timeout of 80000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

[daviddias]

What was the issue you were having? CORS related?

[kumavis]

@diasdavid Parity only supports GET for block/get per ipfs/http-api-spec
https://github.com/ipfs/http-api-spec/blob/master/apiary.apib#L807

[daviddias]

ah, I see. Well, would it be fine for parity to support just method * for now on that one? I feel that it is easier to make it mimic what go-ipfs does than having to remember everyone to set the method for this specific call.

[kumavis]

can we either:

1. change the spec or
2. adhere to the spec

[kumavis]

I feel that it is easier to make it mimic what go-ipfs does

go-ipfs and js-ipfs both handle this method when made as a GET request

[kumavis]

https://github.com/ipfs/http-api-spec/issues/124

[kumavis]

since parity only implements the single route (block/get) I'll just hit that manually and drop js-ipfs-api in the datastore adapter

[daviddias]

on #551 (comment)

We've a lot to do in that land - https://github.com/ipfs/http-api-spec/issues/116