Skip to content

Commit c6221e0

Browse files
authored
Merge pull request #174 from seandilda/no_rd_in_recursor
Ensure Recursor sends requrests without the rd flag set
2 parents d5c51b9 + ad7fde0 commit c6221e0

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

lib/dnsruby/recursor.rb

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ def Recursor.set_hints(hints, resolver)
227227
# Nice idea.
228228

229229
# if (!@@hints || @@hints.length == 0)
230-
resolver.recurse=(1)
230+
resolver.recurse = true
231231
packet=resolver.query_no_validation_or_recursion(".", "NS", "IN")
232232
hints = Hash.new
233233
if (packet)
@@ -303,7 +303,7 @@ def Recursor.set_hints(hints, resolver)
303303
end
304304

305305
# Disable recursion flag.
306-
resolver.recurse=(0)
306+
resolver.recurse = false
307307
# end
308308

309309
# return $self->nameservers( map { @{ $_ } } values %{ $self->{'hints'} } );
@@ -406,7 +406,7 @@ def query(name, type=Types.A, klass=Classes.IN, no_validation = false)
406406
@@mutex.synchronize {
407407
self.hints=(Hash.new) unless @@hints
408408
}
409-
@resolver.recurse=(0)
409+
@resolver.recurse = false
410410
# Make sure the authority cache is clean.
411411
# It is only used to store A and AAAA records of
412412
# the suposedly authoritative name servers.
@@ -622,6 +622,7 @@ def _dorecursion(name, type, klass, known_zone, known_authorities, depth, no_val
622622
end
623623
resolver = Resolver.new({:nameserver=>nameservers})
624624
resolver.dnssec = @dnssec
625+
resolver.recurse = false
625626
servers = []
626627
resolver.single_resolvers.each {|s|
627628
servers.push(s.server)

0 commit comments

Comments
 (0)