apache / apache/libcloud

Possible error in godaddy & hostvirtual DNS RECORD_TYPE_MAP

Abierto
#1,520 1 comentario 0 reacciones 0 asignados Ver en GitHub
stale
Lenguaje dominante
Python
Estrellas
2.1k
Forks
931
Merge medio
1 d 2 h
PR fusionados (30 d)
4

Descripción

## 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.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza con las definiciones de RECORD_TYPE_MAP en libcloud/dns/drivers/hostvirtual.py#L65 y libcloud/dns/drivers/godaddy.py#L111. Compara cómo las API de los proveedores gestionan los registros NS y SPF y, después, inspecciona las pruebas correspondientes del driver DNS, si están disponibles. Se considera terminado cuando se confirma que el mapeo es correcto o se corrige, añadiendo un comentario que explique la discrepancia si es intencionada.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
cloud, networking
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.