Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

fix: support POST-only HTTP API #38

Closed
wants to merge 1 commit into from
Closed

fix: support POST-only HTTP API #38

wants to merge 1 commit into from

Conversation

lidel
Copy link
Contributor

@lidel lidel commented Apr 4, 2020

Invalid. See ipfs/ipfs-webui#1430 instead

ipfs/kubo#7097 will block `GET` commands on
API port, switching everything to POST.

This breaks Files screen in ipfs-webui as noted in
ipfs/ipfs-webui#1429

ipfs-webui is using older version of js-ipfs-http-client, one before
huge refactor into async iterables, which means switching to the latest
version won't be a trivial task.

For now, we just apply simple patch on top of ipfs-http-client v39.0.2
to ensure it sends commands as POST.

Proper fix will land when ipfs-webui is refactored to work with
ipfs-http-client >41.x

Closes #1429
@lidel
Copy link
Contributor Author

lidel commented Apr 4, 2020

Hm.. patching does not work for transitive dependencies.
I think we may need to move this to ipfs-webui

@lidel lidel closed this Apr 4, 2020
@lidel lidel deleted the fix/api-post branch April 4, 2020 19:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant