Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor api, fix bugs #10

Merged
merged 3 commits into from
Jul 16, 2021
Merged

refactor api, fix bugs #10

merged 3 commits into from
Jul 16, 2021

Conversation

justprosh
Copy link
Member

  • now set_external_swarm_multiaddr change Addresses.Announce (set_external_api_multiaddr previously
  • set_external_swarm_multiaddr returns multiaddr with peer id
  • get_from doesn't connect to nodes, just use --api ipfs option
  • get added to retrieve files from local node
  • connect added to use separately

@justprosh justprosh requested a review from folex July 15, 2021 18:32
Co-authored-by: folex <0xdxdy@gmail.com>
@@ -121,7 +121,7 @@ pub fn get_peer_id(local_multiaddr: String, timeout_sec: u64) -> IpfsGetPeerIdRe
}

#[marine]
pub fn set_external_api_multiaddr(multiaddr: String, local_multiaddr: String, timeout_sec: u64) -> IpfsResult {
pub fn set_external_swarm_multiaddr(multiaddr: String, api_multiaddr: String, timeout_sec: u64) -> IpfsResult {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

api_multiaddr => swarm_multiaddr

@folex folex merged commit 65db814 into master Jul 16, 2021
@folex folex deleted the api_fixes branch July 16, 2021 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants