glare 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 27e49ade7bb6778f14de878da4ad2bbc90fdf192542682086c22888805d957fe
-  data.tar.gz: d2f684300c0acb3652dccbf0956b28a49a82b016f8cbb0864c48dbc33a2fcba4
+  metadata.gz: 3bc9315cdcc4aa4a6c2b3b293763b09829a6f3c187f2c4f060facca091005a1b
+  data.tar.gz: d6e79343737a20fa16545c082b52766a6cdee2196a75a53e605e72a77210c35d
 SHA512:
-  metadata.gz: b0b1b129746cdedf5bd3ac073345396a746e162103802231f3363bcecc39da13ab249abaccae66037bc5607f30570b5538e87760020b0b7f739654a111d75d32
-  data.tar.gz: be5bd9e5ebf93bc27ce1e9966ec95f9fd766eae8c9620ec12f0e16f1ed1935342cccd3f49ccc3a11368e78dd49323fc692b654756bc0b69fac2c12f9d56f45d3
+  metadata.gz: e389dad3306135b05f2209cfbb73f0c3f816aee3f1a09d7b08be08aac60e11fc8e4f1602ff746f3c9ff2e5e88a3cd7b041fda683445236eaa1c5879ff848798f
+  data.tar.gz: e430a12761d7ec021098f6d2eac4ee421b9c994ffb3a2c86fc8adeddd10a6edac930224771560396ba237e8ae0d20c525a347348c9ae7e854b942e1cefdfe723

data/lib/glare.rb

@@ -25,6 +25,11 @@ module Glare
       Domain.new(client).deregister(fqdn, type)
     end
 
+    def proxied?(fqdn, type)
+      client = build_client
+      Domain.new(client).proxied?(fqdn, type)
+    end
+
     private
 
     CF_EMAIL = 'CF_EMAIL'.freeze
@@ -35,8 +40,8 @@ module Glare
     end
 
     def default_credentials
-      email = ENV[CF_EMAIL]
-      auth_key = ENV[CF_AUTH_KEY]
+      email = ENV.fetch(CF_EMAIL)
+      auth_key = ENV.fetch(CF_AUTH_KEY)
       Credentials.new(email, auth_key)
     end
 

data/lib/glare/cf_dns_records.rb

@@ -41,6 +41,10 @@ module Glare
       @records.map(&:content)
     end
 
+    def all_proxied?
+      @records.all? { |r| r.proxied == true }
+    end
+
     def each
       @records.each { |record| yield(record) }
     end

data/lib/glare/cf_dns_records/updater.rb

@@ -76,9 +76,10 @@ module Glare
         operations = []
 
         @current_records.delete_if do |record|
-          if new_record = @new_contents.shift
+          if (new_record = @new_contents.shift)
             final_record = record.dup
             final_record.content = new_record.content
+            final_record.proxied = new_record.proxied
             operations << Operation.new(final_record, :update)
             true
           else

data/lib/glare/domain.rb

@@ -7,7 +7,7 @@ module Glare
       @client = client
     end
 
-    def register(fqdn, destinations, type, proxied = false)
+    def register(fqdn, destinations, type, proxied=false)
       dns_records = Array(destinations).map do |destination|
         DnsRecord.new(type: type, name: fqdn, content: destination, proxied: proxied)
       end
@@ -27,5 +27,11 @@ module Glare
       dns_records = zone.records(type)
       Record.deregister(@client, zone, dns_records)
     end
+
+    def proxied?(fqdn, type)
+      zone = Zone.new(@client, fqdn)
+      records = zone.records(type)
+      records.all_proxied?
+    end
   end
 end

data/lib/glare/domain/record.rb

@@ -2,7 +2,7 @@ module Glare
   class Domain
     class Record
       class << self
-        def register(client, zone, dns_records, proxied = false)
+        def register(client, zone, dns_records)
           @client = client
           existing_records = zone.records(dns_records.first.type)
           zone_id = zone.id

data/lib/glare/version.rb

@@ -1,3 +1,3 @@
 module Glare
-  VERSION = '0.6.0'.freeze
+  VERSION = '0.7.0'.freeze
 end

data/spec/proxied_spec.rb

@@ -0,0 +1,27 @@
+require 'glare'
+
+RSpec.describe 'retrieves proxied values', integration: true do
+  context 'when a domain contains more than one destination' do
+    let(:domain) { 'a.flywire.com.cn' }
+    let(:type) { 'A' }
+    before do
+      register_domain(domain, destination)
+    end
+
+    context 'two new records' do
+      let(:destination) { ['1.2.3.4', '5.6.7.8'] }
+
+      it 'resolves to right destination' do
+        expect(proxied?(domain)).to eq(true)
+      end
+    end
+  end
+
+  def register_domain(domain, destination)
+    Glare.register(domain, destination, type, true)
+  end
+
+  def proxied?(domain)
+    Glare.proxied?(domain, type)
+  end
+end

data/spec/units/glare_spec.rb

@@ -2,8 +2,8 @@ require 'glare'
 
 RSpec.describe Glare do
   before do
-    allow(ENV).to receive(:[]).with('CF_EMAIL').and_return('an_email')
-    allow(ENV).to receive(:[]).with('CF_AUTH_KEY').and_return('an_auth_key')
+    allow(ENV).to receive(:fetch).with('CF_EMAIL').and_return('an_email')
+    allow(ENV).to receive(:fetch).with('CF_AUTH_KEY').and_return('an_auth_key')
 
     allow(Glare::Client).to receive(:new).and_return(client)
   end
@@ -102,19 +102,19 @@ RSpec.describe Glare do
         name: 'not-exist.example.com', type: 'CNAME'
       ).and_return(empty_result)
 
-      Glare.register('not-exist.example.com', ['a_destination', 'another_destination'].shuffle, 'CNAME')
+      Glare.register('not-exist.example.com', ['a_destination', 'another_destination'].shuffle, 'CNAME', true)
 
       expect(client).not_to have_received(:put).
         with('/zones/9de4eb694c380d79845d35cd939cc7a7/dns_records', any_args)
 
       expect(client).to have_received(:post).with(
         '/zones/9de4eb694c380d79845d35cd939cc7a7/dns_records',
-        type: 'CNAME', name: 'not-exist.example.com', content: 'a_destination', proxied: false
+        type: 'CNAME', name: 'not-exist.example.com', content: 'a_destination', proxied: true
       )
 
       expect(client).to have_received(:post).with(
         '/zones/9de4eb694c380d79845d35cd939cc7a7/dns_records',
-        type: 'CNAME', name: 'not-exist.example.com', content: 'another_destination', proxied: false
+        type: 'CNAME', name: 'not-exist.example.com', content: 'another_destination', proxied: true
       )
     end
 
@@ -165,25 +165,25 @@ RSpec.describe Glare do
           end
         end
 
-        context 'all records contents are different' do
+        context 'all records proxied attributes are different' do
           it 'sends registration data to update endpoint' do
-            Glare.register('wadus.example.com', ['a_destination.com', 'yet_another_destination.com'].shuffle, 'CNAME')
+            destinations = ['destination.com', 'another_destination.com'].shuffle
+            Glare.register('wadus.example.com', destinations, 'CNAME', true)
 
             expect(client).not_to have_received(:post).
               with('/zones/9de4eb694c380d79845d35cd939cc7a7/dns_records', any_args)
 
             expect(client).to have_received(:put).with(
               any_args,
-              type: 'CNAME', name: 'wadus.example.com', content: 'a_destination.com', proxied: false
+              type: 'CNAME', name: 'wadus.example.com', content: 'destination.com', proxied: true
             )
 
             expect(client).to have_received(:put).with(
               any_args,
-              type: 'CNAME', name: 'wadus.example.com', content: 'yet_another_destination.com', proxied: false
+              type: 'CNAME', name: 'wadus.example.com', content: 'another_destination.com', proxied: true
             )
           end
         end
-
       end
 
       it 'updates different records and deletes extra ones' do
@@ -194,14 +194,15 @@ RSpec.describe Glare do
 
         expect(client).to have_received(:put).with(
           any_args,
-          { type: 'CNAME', name: 'wadus.example.com', content: 'a_destination.com', proxied: false }
+          type: 'CNAME', name: 'wadus.example.com', content: 'a_destination.com', proxied: false
         )
 
         expect(client).to have_received(:delete).once
       end
 
       it 'updates different records and creates new ones' do
-        Glare.register('wadus.example.com', ['destination.com', 'another_destination.com', 'a_third_destination.com'].shuffle, 'CNAME')
+        destinations = ['destination.com', 'another_destination.com', 'a_third_destination.com'].shuffle
+        Glare.register('wadus.example.com', destinations, 'CNAME')
 
         expect(client).not_to have_received(:put).with(
           '/zones/9de4eb694c380d79845d35cd939cc7a7/dns_records/a1f984afe5544840505494298f54c33e',

data/spec/units/operations_spec.rb

@@ -71,20 +71,23 @@ RSpec.describe Glare::CfDnsRecords::Updater do
     current_records = Glare::CfDnsRecords.new([current_record2, current_record, current_record3])
 
     new_record = dns_record(content: '1.2.3.8')
-    update_record = dns_record(content: '1.2.3.4')
+    update_record = dns_record(content: '1.2.3.4', proxied: true)
     new_records = [new_record, update_record].shuffle
 
     operations = Glare::CfDnsRecords::Updater.new(current_records, new_records).calculate
 
-    current_record2.content = '1.2.3.8'
-    update_operation = Glare::CfDnsRecords::Updater::Operation.new(current_record2, :update)
+    updated_record = current_record.dup.tap { |r| r.proxied = true }
+    updated_record2 = current_record2.dup.tap { |r| r.content = '1.2.3.8' }
+
+    update_operation = Glare::CfDnsRecords::Updater::Operation.new(updated_record2, :update)
+    update_operation2 = Glare::CfDnsRecords::Updater::Operation.new(updated_record, :update)
     delete_operation = Glare::CfDnsRecords::Updater::Operation.new(current_record3, :delete)
 
-    expect(operations.updates).to eq([update_operation])
-    expect(operations.deletions).to eq([delete_operation])
+    expect(operations.updates).to match_array([update_operation, update_operation2])
+    expect(operations.deletions).to match_array([delete_operation])
   end
 
-  it 'can detects new records to delete and update' do
+  it 'can detect new records to delete and update' do
     current_record = existing_record(content: '1.2.3.4')
     current_record2 = existing_record(content: '1.2.3.6')
     current_record3 = existing_record(content: '1.2.3.5')
@@ -104,7 +107,7 @@ RSpec.describe Glare::CfDnsRecords::Updater do
     Glare::CfDnsRecord.new(id: id, name: name, type: type, content: content)
   end
 
-  def dns_record(name: 'name', type: 'A', content:)
-    Glare::DnsRecord.new(name: name, type: type, content: content)
+  def dns_record(name: 'name', type: 'A', content:, proxied: false)
+    Glare::DnsRecord.new(name: name, type: type, content: content, proxied: proxied)
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: glare
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Jose Luis Salas
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-27 00:00:00.000000000 Z
+date: 2019-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -127,6 +127,7 @@ files:
 - spec/fixtures/list_zone.json
 - spec/fixtures/wadus_records.json
 - spec/fixtures/wadus_records_reverse_order.json
+- spec/proxied_spec.rb
 - spec/resolve_domain_spec.rb
 - spec/spec_helper.rb
 - spec/units/api_response_spec.rb
@@ -162,6 +163,7 @@ test_files:
 - spec/units/glare_spec.rb
 - spec/resolve_domain_spec.rb
 - spec/spec_helper.rb
+- spec/proxied_spec.rb
 - spec/delete_domain_spec.rb
 - spec/fixtures/wadus_records_reverse_order.json
 - spec/fixtures/wadus_records.json