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

Mention that deno-dns is not implemented #69

Open
oxcabe opened this issue Sep 15, 2024 · 0 comments
Open

Mention that deno-dns is not implemented #69

oxcabe opened this issue Sep 15, 2024 · 0 comments

Comments

@oxcabe
Copy link

oxcabe commented Sep 15, 2024

The README.md file states that Deno is supported:

* Works in all JavaScript runtimes: Browsers, Node.js, CloudFlare Workers, Deno, Bun, etc

But it seems that is not implemented yet:

dns-records/src/index.ts

Lines 93 to 95 in 7e58d04

} else if (resolver === 'deno-dns') {
throw new Error('Deno DNS not yet implemented')
}

I understand that it could be possible to make this library work in Deno through its Node compatibility layer. That said, the README.md file should, still, mention that the pure Deno implementation (using Deno.resolveDns) is a work in progress.

Finally, I will probably implement a Deno-only util module for retrieving DNS records and perform subdomain enumeration in a few days. I'm more than happy to ping back so you can use the code here.

@oxcabe oxcabe changed the title Mention that for Deno is not implemented Mention that deno-dnsis not implemented Sep 15, 2024
@oxcabe oxcabe changed the title Mention that deno-dnsis not implemented Mention that deno-dns is not implemented Sep 15, 2024
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

No branches or pull requests

1 participant