macfreek opened a new issue #1523:
URL: https://github.com/apache/libcloud/issues/1523


   ## Summary
   
   `libcloud.dns.types` defines a RecordType 'URL'. However, no such RR Type 
exists in DNS. Likely, this is supposed to refer to the 'URI' RR Type.
   
   ## Detailed Information
   
   The authoritative list of DNS RR types is 
https://www.iana.org/assignments/dns-parameters/dns-parameters.xhtml. There is 
no 'URL' RR type in this list. There is a 'URI' RR type in this list, as 
defined in [RFC 7553](https://tools.ietf.org/html/rfc7553)
   
   This RecordType is defined in 4 places in libcloud:
   
   [libcloud.dns.types, line 
112](https://github.com/apache/libcloud/blob/trunk/libcloud/dns/types.py#L112):
   ```
       URL = 'URL'
   ```
   
   [libcloud.dns.drivers.cloudflare, line 
171](https://github.com/apache/libcloud/blob/trunk/libcloud/dns/drivers/cloudflare.py#L171):
 [sic!]
   ```
           RecordType.URL: 'LOC'
   ```
   
   [libcloud.dns.drivers.dnsimple, line 
57](https://github.com/apache/libcloud/blob/trunk/libcloud/dns/drivers/dnsimple.py#L57)
   ```
           RecordType.URL: 'URL',
   ```
   
   [`libcloud.dns.drivers.zerigo`, line 
142](https://github.com/apache/libcloud/blob/trunk/libcloud/dns/drivers/zerigo.py#L142)
   ```
           RecordType.URL: 'URL',
   ```
   
   My recommendation is to change these occurrences into `'URI'`


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to