ipfs-shipyard / ipfs-shipyard/py-ipfs-http-client

swarm.connect() returns failed to parse multiaddr "": empty multiaddr

Open
#246 2 comments 0 reactions 0 assignees View on GitHub
bug help wanted P2 - Medium
Dominant language
Python
Stars
685
Forks
200
PR merge metrics
No merged PRs in 30d

Description

py-ipfs-http-client = 0.6.1
ipfs = 0.6.0

It appears the arguments are not allocated properly.

client/swarm.py:

args = (str(address), *(str(a) for a in address))
return self._client.request('/swarm/disconnect', args, decoder='json', **kwargs)

This builds an URL like: http+ip4://127.0.0.1:5001/api/v0/swarm/connect?stream-channels=true&arg=%2Fip4%2FXX.XX.XX.XX%2Ftcp%2F4001%2Fipfs%2FPEERADDRESS&arg=%2F&arg=i&arg=p&arg=4&arg=%2F&arg=X&arg=X&arg=.&arg=X&arg=X&arg=.&arg=X&arg=.&arg=X&arg=X&arg=%2F&arg=t&arg=c&arg=p&arg=%2F&arg=4&arg=0&arg=0&arg=1&arg=%2F&arg=i&arg=p&arg=f&arg=s&arg=%2F&arg=P&arg=E&arg=E&arg=R&arg=A&arg=D&arg=D&arg=R&arg=E&arg=S&arg=S

Are all this extra arguments needed?

Please let me know so I can submit a PR to fix this.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.