getresources returns nothing for types created with Generic.create since 0.7.2
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 86/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- ruby
- Domaine
- networking
Piste de recherche
Commencez par exécuter la reproduction fournie sans réseau et inspectez DNS#extract_resources, en particulier les deux branches qui sélectionnent les réponses selon la typeclass. Vérifiez que getresources renvoie l’enregistrement Generic décodé, tandis que les enregistrements nommés IN::A et IN::AAAA conservent leur comportement actuel.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Since 0.7.2, getresources and each_resource never return records for a type created with Resolv::DNS::Resource::Generic.create, for example NAPTR (type 35), which has no named class in Resolv.
Cause
fa5e689c4 ("Do not register on-demand classes for unknown DNS types and SvcParamKeys") stopped registering the classes from Generic.create. So decoding a reply gives each unknown record a new anonymous class, not the class the caller passed. 05185b432 and fcff0a19e updated Message#== and Generic#== for this with Generic.type_class_equal?. DNS#extract_resources still selects answers with when typeclass, so it drops every decoded record of such a type.
Reproduction
This needs no network: it encodes and decodes a reply and gives it to getresources.
require "resolv"
naptr = Resolv::DNS::Resource::Generic.create(35, Resolv::DNS::Resource::IN::ClassValue)
name = "example.com."
rdata = "\x00\x64\x00\x0A\x01U\x07E2U+sip\x1C!^.*$!sip:[email protected]!\x00".b
reply = Resolv::DNS::Message.new
reply.add_answer(name, 60, naptr.new(rdata))
decoded = Resolv::DNS::Message.decode(reply.encode)
dns = Resolv::DNS.new
dns.define_singleton_method(:fetch_resource) {|_name, _typeclass, &block| block.call(decoded, Resolv::DNS::Name.create(name)) }
puts "resolv #{Resolv::VERSION}: #{dns.getresources(name, naptr).size} record(s)"
resolv 0.7.0: 1 record(s)
resolv 0.7.2: 0 record(s)
resolv 0.8.0: 0 record(s)
A real query shows the same result: getresources returns [], but fetch_resource gets the NAPTR answer. Ruby 4.0.7 includes 0.7.2, so a patch update of Ruby is enough to get this behaviour.
Possible fix
Compare by type and class value in extract_resources, in both when typeclass branches:
- when typeclass
+ when ->(data) { Resource::Generic.type_class_equal?(data.class, typeclass) }
With this change, the reproduction returns 1 record on 0.8.0. Named types (IN::A, IN::AAAA) behave as before, because type_class_equal? compares identity first and returns false for classes that are not Generic.
Workaround
Register the type in the same way as Resolv's built-in types:
class NAPTR < Resolv::DNS::Resource::Generic
const_set(:TypeValue, 35)
const_set(:ClassValue, Resolv::DNS::Resource::IN::ClassValue)
Resolv::DNS::Resource::ClassHash[[self::TypeValue, self::ClassValue]] = self
end
- Langage dominant
- Ruby
- Étoiles
- 56
- Forks
- 41
- Merge moyen
- 2 h 50 min
- PR mergées (30 j)
- 8
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ruby/resolv
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
`resolv.conf` on macOS sometimes has interface suffix which breaks `resolve.rb`.Peut-être à nouveau libre @ioquatix l’a pris il y a 1253 jours, et aucune pull request n’est ouverte. Ouverte
Toutes les issues de ruby/resolv
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
mastodon/mastodon#40755 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
L: docker L: elm L: github:actions L: helm L: ruby:bundler
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
dependabot/dependabot-core#16425 ·
Les mainteneurs répondent en général sous 2 jours