Possible error in godaddy & hostvirtual DNS RECORD_TYPE_MAP
- Langage dominant
- Python
- Étoiles
- 2.1k
- Forks
- 931
- Merge moyen
- 1 j 2 h
- PR mergées (30 j)
- 4
Description
## Summary
[libcloud/dns/drivers/hostvirtual.py](libcloud/dns/drivers/hostvirtual.py#L65) and [libcloud/dns/drivers/godaddy.py](libcloud/dns/drivers/godaddy.py#L111) both contain the following definition:
```
RECORD_TYPE_MAP = {
RecordType.A: 'A',
RecordType.AAAA: 'AAAA',
RecordType.CNAME: 'CNAME',
RecordType.MX: 'MX',
RecordType.NS: 'SPF',
RecordType.SRV: 'SRV',
RecordType.TXT: 'TXT',
}
```
The line `RecordType.NS: 'SPF',` is very unusual. I suspect this is a bug. This is only two of three places in the libcloud/driver library where there is a mismatch between record type and name. (The other is in cloudflare.py, which defines `RecordType.URL: 'LOC'`).
If it is really not a bug, I suggest to add a comment explaining this mismatch.
My apologies for not adding a full bug report with reproducible code. I encountered this during a coarse check which record types are commonly supported.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par les définitions de RECORD_TYPE_MAP dans libcloud/dns/drivers/hostvirtual.py#L65 et libcloud/dns/drivers/godaddy.py#L111. Comparez la gestion des enregistrements NS et SPF par les API des fournisseurs, puis examinez les tests correspondants du driver DNS s’ils sont disponibles. Le travail est considéré comme terminé lorsque vous avez confirmé que le mapping est correct ou que vous l’avez corrigé, avec un commentaire expliquant la divergence si elle est intentionnelle.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- cloud, networking
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100