resolver.5: document six previously undocumented options
ClosedPublic
Actions

Authored by chris.longros_gmail.com on Sun, Mar 15, 10:40 AM.

Details

Reviewers

des

Commits

rG462a1f6197fa: resolver.5: document six previously undocumented options

Summary

Document the edns0, inet6, insecure1, insecure2, no-check-names,
and rotate options which are parsed by res_init(3) but were not
described in the resolver(5) man page.

Signed-off-by: Christos Longros <chris.longros@gmail.com>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chris.longros_gmail.com created this revision.Sun, Mar 15, 10:40 AM

Herald added subscribers: ziaee, imp. · View Herald TranscriptSun, Mar 15, 10:40 AM

chris.longros_gmail.com requested review of this revision.Sun, Mar 15, 10:40 AM

Harbormaster completed remote builds in B71411: Diff 173711.Sun, Mar 15, 10:40 AM

chris.longros_gmail.com added a reviewer: des.Sun, Mar 15, 10:44 AM

des accepted this revision.Sun, Mar 15, 12:08 PM

des added inline comments.

share/man/man5/resolver.5
216	I wonder if we should make this the default, like GNU libc does.

This revision is now accepted and ready to land.Sun, Mar 15, 12:08 PM

chris.longros_gmail.com added a comment.Sun, Mar 15, 12:18 PM

This comment was removed by chris.longros_gmail.com.

I wonder if we should make this the default, like GNU libc does.

Taking a look at glibc's resolv.h it appears that the default options do not include RES_ROTATE.

#define RES_DEFAULT	(RES_RECURSE|RES_DEFNAMES|RES_DNSRCH)

In D55864#1278123, @chris.longros_gmail.com wrote:
Taking a look at glibc's resolv.h it appears that the default options do not include RES_ROTATE.
#define RES_DEFAULT	(RES_RECURSE|RES_DEFNAMES|RES_DNSRCH)

I don't think they have RES_ROTATE at all, their resolver is hardcoded to always rotate between configured nameservers.

Closed by commit rG462a1f6197fa: resolver.5: document six previously undocumented options (authored by chris.longros_gmail.com, committed by des). · Explain WhySun, Mar 15, 3:19 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rG462a1f6197fa: resolver.5: document six previously undocumented options.