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

go-namesys v0.1.1 #8057

Merged
merged 1 commit into from
Apr 12, 2021
Merged

go-namesys v0.1.1 #8057

merged 1 commit into from
Apr 12, 2021

Conversation

gammazero
Copy link
Contributor

@gammazero gammazero commented Apr 7, 2021

The new version has an improved resolve error and fixes #8000

Changelog: https://github.com/ipfs/go-namesys/releases/tag/v0.1.1

@gammazero
Copy link
Contributor Author

If it is preferable to retag go-namesys, this PR can wait for that and be updated after.

@gammazero gammazero requested a review from lidel April 7, 2021 05:52
@gammazero gammazero mentioned this pull request Apr 7, 2021
@lidel lidel added this to the go-ipfs 0.9 milestone Apr 7, 2021
@lidel lidel force-pushed the fix/improve-resolve-error branch from 6ce6ed3 to 79de5ff Compare April 7, 2021 20:40
@lidel lidel changed the title Update the dependency on go-namesys go-namesys v0.1.1 Apr 7, 2021
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

Tagged go-namesys v0.1.1 and rebased this PR to use that.

This PR includes both ipfs/go-namesys#9 and ipfs/go-namesys#8 – tested both again locally, works as expected 👍

$ ipfs resolve -r /ipns/ipfs.eth
/ipfs/bafybeiddamod5xk4uipjz52xodu4z62ul5xk6lzo3qaiz32yc3t6vd7xqa

$ ipfs resolve -r /ipns/google.com
Error: could not resolve name: "google.com" is missing a DNSLink record (https://docs.ipfs.io/concepts/dnslink/)

@Stebalien Stebalien merged commit efe5abd into master Apr 12, 2021
@Stebalien Stebalien deleted the fix/improve-resolve-error branch April 12, 2021 15:49
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.

Improve DNSLink error
3 participants