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

make DNS resolver pluggable #10

Merged
merged 12 commits into from
Apr 23, 2021
Merged

make DNS resolver pluggable #10

merged 12 commits into from
Apr 23, 2021

Conversation

vyzo
Copy link
Contributor

@vyzo vyzo commented Apr 12, 2021

Part of ipfs/kubo#6532

@vyzo vyzo requested review from lidel and aschmahmann 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
namesys.go Outdated Show resolved Hide resolved
@vyzo
Copy link
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
Contributor Author

vyzo commented Apr 15, 2021

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

@vyzo
Copy link
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
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.

namesys.go Outdated Show resolved Hide resolved
@vyzo vyzo force-pushed the feat/custom-resolver branch from 2e3a925 to 415b353 Compare April 23, 2021 17:09
@vyzo
Copy link
Contributor Author

vyzo commented Apr 23, 2021

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

@vyzo
Copy link
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