Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

feat: /api/v0/dns #1172

Merged
merged 7 commits into from
Jan 16, 2018
Merged

feat: /api/v0/dns #1172

merged 7 commits into from
Jan 16, 2018

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Jan 9, 2018

Ref.: #904, ipfs-inactive/js-ipfs-http-client#665

On Node.js we use dns.resolveTxt to get the dnslink path. On the browser, we use the public gateway.

@ghost ghost assigned hacdias Jan 9, 2018
@ghost ghost added the status/in-progress In progress label Jan 9, 2018
@hacdias
Copy link
Member Author

hacdias commented Jan 9, 2018

image

@hacdias hacdias requested a review from daviddias January 9, 2018 12:19
@hacdias hacdias changed the title [WIP] feat: /api/v0/dns feat: /api/v0/dns Jan 9, 2018
@hacdias
Copy link
Member Author

hacdias commented Jan 9, 2018

@diasdavid some tests will fail since it doesn't use the js-ipfs-api from ipfs-inactive/js-ipfs-http-client#665

@hacdias
Copy link
Member Author

hacdias commented Jan 9, 2018

And this one too is done @diasdavid 🚀

@hacdias
Copy link
Member Author

hacdias commented Jan 12, 2018

Ping @diasdavid

@ghost ghost assigned daviddias Jan 12, 2018
test/cli/dns.js Outdated
this.timeout(60 * 1000)

return ipfs('dns ipfs.io').then((res) => {
console.log(res)
Copy link
Member

Choose a reason for hiding this comment

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

@hacdias not needed, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

@hacdias
Copy link
Member Author

hacdias commented Jan 13, 2018

Fixed @diasdavid! Was using window variable which is not available on WebWorkers 😄

@hacdias
Copy link
Member Author

hacdias commented Jan 16, 2018

Is there anything missing here @diasdavid?

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.

2 participants