pfSense

Here is the patch I am using with the System Patches package to work around this issue in 2.1:

--- /etc/inc/dyndns.class.orig    2015-06-05 08:14:09.000000000 -0500
+++ /etc/inc/dyndns.class    2015-06-05 08:18:00.000000000 -0500
@@ -538,44 +538,9 @@

                     /* Set Amazon AWS Credentials for this record */
                     $r53 = new Route53($AccessKeyId, $SecretAccessKey);
+                    $changes = $r53->prepareChange('UPSERT', $hostname, 'A', $NewTTL, $NewIP);
+                    $result = $r53->changeResourceRecordSets("/hostedzone/$ZoneID", $changes);

-                    /* Function to find old values of records in Route 53 */
-                    if(!function_exists('Searchrecords')) {
-                        function SearchRecords($records, $name) {
-                            $result = array();
-                            foreach($records as $record) {
-                                if(strtolower($record['Name']) == strtolower($name)) {
-                                    $result [] = $record;
-                                }
-                            }
-                            return ($result) ? $result : false;
-                        }
-                    }
-
-                    $records = $r53->listResourceRecordSets("/hostedzone/$ZoneID");
-
-                    /* Get IP for your hostname in Route 53 */
-                    if(false !== ($a_result = SearchRecords($records['ResourceRecordSets'], "$hostname"))) {
-                        $OldTTL=$a_result[0][TTL];
-                        $OldIP=$a_result[0][ResourceRecords][0];
-                    } else {
-                        $OldIP="";
-                    }
-
-                    /* Check if we need to update DNS Record */
-                    if ($OldIP !== $NewIP) {
-                        if(!empty($OldIP)) {
-                            /* Your Hostname already exists, deleting and creating it again */
-                            $changes = array();
-                            $changes[] = $r53->prepareChange(DELETE, $hostname, A, $OldTTL, $OldIP);
-                            $changes[] = $r53->prepareChange(CREATE, $hostname, A, $NewTTL, $NewIP);
-                            $result = $r53->changeResourceRecordSets("/hostedzone/$ZoneID", $changes);
-                        } else {
-                            /* Your Hostname does not exist yet, creating it */
-                            $changes = $r53->prepareChange(CREATE, $hostname, A, $NewTTL, $NewIP);
-                            $result = $r53->changeResourceRecordSets("/hostedzone/$ZoneID", $changes);
-                        }
-                    }
                     $this->_checkStatus(0, $result);
                     break;
                 case 'custom':

Jim,

Did you (or anyone) every submit a commit to correct this? I know it becomes problematic when the TTL changes, but I would like to work on submitting this to have it included with the master branch. Also looking to add some geo features with that change. Thoughts?

M

The use of the UPCERT action in 6751 should address this bug.