Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

make DNS resolver pluggable#10

Merged
Stebalien merged 12 commits intomasterfrom
feat/custom-resolver
Apr 23, 2021
Merged

make DNS resolver pluggable#10
Stebalien merged 12 commits intomasterfrom
feat/custom-resolver

Conversation

@vyzo
Copy link
Copy Markdown
Contributor

@vyzo vyzo commented Apr 12, 2021

Part of ipfs/kubo#6532

@vyzo vyzo requested review from aschmahmann and lidel April 12, 2021 09:12
@vyzo vyzo force-pushed the feat/custom-resolver branch from ec9e1df to 2b85ea5 Compare April 12, 2021 09:27
@lidel lidel mentioned this pull request Apr 12, 2021
9 tasks
@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Apr 15, 2021

So what do we want to do here? Make the constructor take options and also remove the hard-coded shim for .eth resolution?

@vyzo vyzo changed the title make DNS resolver pluggable, use new madns.BasicResolver interface make DNS resolver pluggable Apr 15, 2021
@vyzo vyzo force-pushed the feat/custom-resolver branch from 0e1deee to 7a6125f Compare April 15, 2021 21:41
@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Apr 15, 2021

Per discussion, I have updated the constructor to take options.

@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Apr 15, 2021

also removed the special casing of .eth domains, as we now have resolvers capable of resolving them.

@vyzo vyzo requested review from Stebalien and aschmahmann April 15, 2021 21:54
Copy link
Copy Markdown
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.

Removal of special-casing of .eth looks good.

@vyzo vyzo force-pushed the feat/custom-resolver branch from 2e3a925 to 415b353 Compare April 23, 2021 17:09
@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Apr 23, 2021

rebased on master and resolved the conflict; will go ahead and merge.

@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Apr 23, 2021

seems travis is having some issues...

@Stebalien Stebalien merged commit 2454122 into master Apr 23, 2021
@Stebalien Stebalien deleted the feat/custom-resolver branch April 23, 2021 17:30
guseggert pushed a commit to ipfs/boxo that referenced this pull request Dec 6, 2022
Jorropo pushed a commit to ipfs/go-libipfs-rapide that referenced this pull request Mar 23, 2023
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.

4 participants