From 725d314f4e6a217cc0faf3fbb876f53a68d448a3 Mon Sep 17 00:00:00 2001
From: ashishprajapati <ashishprajapati0404@gmail.com>
Date: Wed, 16 Aug 2017 18:54:35 +0530
Subject: [PATCH] Features# Geo support added for CNAME Records. ISSUE - #62

---
 octodns/provider/route53.py | 39 +++++++++++++++++++++++++++++--------
 octodns/record.py           |  6 +++---
 2 files changed, 34 insertions(+), 11 deletions(-)

diff --git a/octodns/provider/route53.py b/octodns/provider/route53.py
index 490d630..6068141 100644
--- a/octodns/provider/route53.py
+++ b/octodns/provider/route53.py
@@ -92,7 +92,7 @@ class _Route53Record(object):
     def _values_for_value(self, record):
         return [record.value]
 
-    _values_for_CNAME = _values_for_value
+    _values_for_CNAME = _values_for_values
     _values_for_PTR = _values_for_value
 
     def _values_for_MX(self, record):
@@ -317,8 +317,19 @@ class Route53Provider(BaseProvider):
             'ttl': int(rrset['TTL'])
         }
 
+    def _data_for_geo_cname(self, rrset):
+        ret = {
+            'type': rrset['Type'],
+            'values': [v['Value'] for v in rrset['ResourceRecords']],
+            'ttl': int(rrset['TTL'])
+        }
+        geo = self._parse_geo(rrset)
+        if geo:
+            ret['geo'] = geo
+        return ret
+
     _data_for_PTR = _data_for_single
-    _data_for_CNAME = _data_for_single
+    _data_for_CNAME = _data_for_geo_cname
 
     _fix_semicolons = re.compile(r'(?<!\\);')
 
@@ -513,9 +524,12 @@ class Route53Provider(BaseProvider):
                 # not a version & type match, ignore
                 continue
             config = health_check['HealthCheckConfig']
-            if host == config['FullyQualifiedDomainName'] and \
-               first_value == config['IPAddress']:
-                # this is the health check we're looking for
+            try:
+                if host == config['FullyQualifiedDomainName'] and \
+                   first_value == config['IPAddress']:
+                    # this is the health check we're looking for
+                    return id
+            except:
                 return id
 
         if not create:
@@ -524,16 +538,25 @@ class Route53Provider(BaseProvider):
 
         # no existing matches, we need to create a new health check
         config = {
-            'EnableSNI': True,
             'FailureThreshold': 6,
-            'FullyQualifiedDomainName': host,
-            'IPAddress': first_value,
             'MeasureLatency': True,
             'Port': 443,
             'RequestInterval': 10,
             'ResourcePath': '/_dns',
             'Type': 'HTTPS',
         }
+
+        if not record._type == "CNAME":
+            config.update({
+                'FullyQualifiedDomainName': host,
+                'EnableSNI': True,
+                'IPAddress':  first_value,
+            })
+        else:
+            config.update({
+                'FullyQualifiedDomainName': first_value
+            })
+
         ref = '{}:{}:{}'.format(self.HEALTH_CHECK_VERSION, record._type,
                                 uuid4().hex[:16])
         resp = self._conn.create_health_check(CallerReference=ref,
diff --git a/octodns/record.py b/octodns/record.py
index 6ee9dff..8c61582 100644
--- a/octodns/record.py
+++ b/octodns/record.py
@@ -398,7 +398,7 @@ class AliasRecord(_ValueMixin, Record):
         return value
 
 
-class CnameRecord(_ValueMixin, Record):
+class CnameRecord(_GeoMixin, Record):
     _type = 'CNAME'
 
     @classmethod
@@ -416,8 +416,8 @@ class CnameRecord(_ValueMixin, Record):
             reasons.append('missing trailing .')
         return reasons
 
-    def _process_value(self, value):
-        return value
+    def _process_values(self, values):
+        return values
 
 
 class MxValue(object):