-
-
Notifications
You must be signed in to change notification settings - Fork 298
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
rDNS might not work on latest macos #166
Comments
Hey, thanks for reporting this. That's quite odd. If you run bandwhich with the -n flag like this |
That works! |
Hey @richstokes, so if you use the same terminal window from which you run bandwhich (when it doesn't work, without the -n flag) in order to do a |
Yep, DNS works fine for other apps;
|
Hum... just for debugging, would you be willing to try setting your dns server to 1.1.1.1 and seeing if it works out (with both bandwhich and nslookup)? |
Sure. Switched DNS to 1.1.1.1, cleared cache, retried, same error. |
Thanks! I'd like to give you a debug branch that'll give us a more verbose error in this case with a quick hack. Would you be comfortable running such a branch from this repo? I can provide some instructions if you're unfamiliar with rust. |
Can do when I have a few moments spare - not used Rust so would need info on how to compile/run |
Same error here on Arch Linux, |
@richstokes @DianaNites any chance you're running The problem gets fixed on master (hickory-dns/resolv-conf@83c0f25), so it should be possible to fix this issue by updating the I stumbled upon this issue after my latest OS update (which included systemd-246) which uses @imsnif I only confirmed that |
This fixes the parsing of `/etc/resolv.conf` with `options trust-ad` which is e.g. used in `systemd-resolved-v246`[1]. Refs imsnif#166 [1] systemd/systemd@a742f98
ChangeLogs: * https://github.com/imsnif/bandwhich/releases/tag/0.18.1 * https://github.com/imsnif/bandwhich/releases/tag/0.18.0 Also applied a fix[1] for rDNS support which broke with `systemd-v246` because the `resolv-conf` crate didn't support the `options trust_ad`-declaration in systemd's `/etc/resolv.conf`[2]. [1] imsnif/bandwhich#184 [2] imsnif/bandwhich#166 (comment)
ChangeLogs: * https://github.com/imsnif/bandwhich/releases/tag/0.18.1 * https://github.com/imsnif/bandwhich/releases/tag/0.18.0 Also applied a fix[1] for rDNS support which broke with `systemd-v246` because the `resolv-conf` crate didn't support the `options trust_ad`-declaration in systemd's `/etc/resolv.conf`[2]. [1] imsnif/bandwhich#184 [2] imsnif/bandwhich#166 (comment) (cherry picked from commit 8149ab1)
Thanks very much for finding this @Ma27, and for the quick fix. My apologies for not finding the time for debugging this myself. |
Hmm I'm actually realizing now that the reporter has the issue on macos, so I don't if they have the same issue. However my patch fixes the issue for me. |
This fixes the parsing of `/etc/resolv.conf` with `options trust-ad` which is e.g. used in `systemd-resolved-v246`[1]. Refs imsnif#166 [1] systemd/systemd@a742f98
At the very least we'll get clearer errors after your patch on this. |
…conf` crate (#184) * Show the error cause if initializing a DNS resolver fails * Manually bump `resolv-conf` to `master` at rev `83c0f25` This fixes the parsing of `/etc/resolv.conf` with `options trust-ad` which is e.g. used in `systemd-resolved-v246`[1]. Refs #166 [1] systemd/systemd@a742f98
FWIW I was seeing this on an ubuntu install (rolling rhino, thieir kinda rolling release) and installing from git with the patch for resolv.conf actually did fix it for me |
@Ma27 I'm on Arch Linux, which indeed uses systemd-networkd 246. The latest bandwhich version, 0.19.0, does not fix it for me, though the error is now more detailed.
|
Hey @DianaNites (and others experiencing this issue), I added a feature that would allow users to explicitly specify a dns server to use for the rDNS lookups as a workaround. Something like I'm going to release a new version with it in the next few days (I'm waiting on another fix I'd like to see in that version too). Thanks for your patience. :) |
New version released. Starting from version |
Closing this issue as completed. If you disagree please comment/reopen. |
Installed via homebrew
The text was updated successfully, but these errors were encountered: