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

FS#2341 - Feature Request: nslookup - support SRV queries #8770

Closed
openwrt-bot opened this issue Jun 26, 2019 · 6 comments
Closed

FS#2341 - Feature Request: nslookup - support SRV queries #8770

openwrt-bot opened this issue Jun 26, 2019 · 6 comments
Labels

Comments

@openwrt-bot
Copy link

pmelange:

Could support for SRV requests be added to the patch https://github.com/openwrt/openwrt/blob/master/package/utils/busybox/patches/230-add_nslookup_lede.patch

Currently I am working on setting up a service for freifunk which uses SRV records. Unfortunately I have to install bind-host and all it's dependencies to make the DNS query. It would be great if the built-in busybox nslookup supported this.

@jow- seems to have done a bit of work on nslookup in the past.

Any support would be appreciated.

P

@openwrt-bot
Copy link
Author

@openwrt-bot
Copy link
Author

pmelange:

Hey @jow-

Thanks for doing this so quickly, but I'm a bit confused about patch 230-add_nslookup_lede.patch. Isn't it the patched version nslookup_lede.c (with musl compatibility) which is included in openwrt? At least when I build, nslookup_lede.o is built, but nslookup.o is not.

@openwrt-bot
Copy link
Author

jow-:

That extra applet went upstream as "nslookup big" into busybox a while ago. Will drop 230-add_nslookup_lede.patch and switch to nslookup big when the SRV support landed.

@openwrt-bot
Copy link
Author

pmelange:

OK. Cool. Any chance that this can go into the 18.06 branch too?

@openwrt-bot
Copy link
Author

pmelange:

I have created PR #2287 for the 18.06 branch.

@openwrt-bot
Copy link
Author

pmelange:

The patch from Jo-Philipp is now part of Busybox 1.33.0. Also, in OpenWrt the 21.02 branch as well as master are currently both using Busybox 1.33.0.

Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant