gandi_v5 0.1.0 → 0.6.0

data/lib/gandi_v5/live_dns/domain.rb

@@ -9,6 +9,7 @@ class GandiV5
     #   @return [String]
     class Domain
       include GandiV5::Data
+      include GandiV5::LiveDNS::HasZoneRecords
 
       members :fqdn
 
@@ -20,179 +21,49 @@ class GandiV5
 
       # Refetch the information for this domain from Gandi.
       # @return [GandiV5::LiveDNS::Domain]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def refresh
-        data = GandiV5.get url
+        _response, data = GandiV5.get url
         from_gandi data
       end
 
-      # @overload fetch_records()
-      #   Fetch all records for this domain.
-      # @overload fetch_records(name)
-      #   Fetch records for a name.
-      #   @param name [String] the name to fetch records for.
-      # @overload fetch_records(name, type)
-      #   Fetch records of a type for a name.
-      #   @param name [String] the name to fetch records for.
-      #   @param type [String] the record type to fetch.
-      # @return [Array<GandiV5::LiveDNS::RecordSet>]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def fetch_records(name = nil, type = nil)
-        GandiV5::LiveDNS.require_valid_record_type type if type
-
-        url_ = "#{url}/records"
-        url_ += "/#{CGI.escape name}" if name
-        url_ += "/#{CGI.escape type}" if type
-
-        data = GandiV5.get url_
-        data = [data] unless data.is_a?(Array)
-        data.map { |item| GandiV5::LiveDNS::RecordSet.from_gandi item }
-      end
-
-      # @overload fetch_zone_lines()
-      #   Fetch all records for this domain.
-      # @overload fetch_zone_lines(name)
-      #   Fetch records for a name.
-      #   @param name [String] the name to fetch records for.
-      # @overload fetch_zone_lines(name, type)
-      #   Fetch records of a type for a name.
-      #   @param name [String] the name to fetch records for.
-      #   @param type [String] the record type to fetch.
-      # @return [String]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def fetch_zone_lines(name = nil, type = nil)
-        GandiV5::LiveDNS.require_valid_record_type type if type
-
-        url_ = "#{url}/records"
-        url_ += "/#{CGI.escape name}" if name
-        url_ += "/#{CGI.escape type}" if type
-
-        GandiV5.get url_, accept: 'text/plain'
-      end
-
-      # Add record to this domain.
-      # @param name [String]
-      # @param type [String]
-      # @param ttl [Integer]
-      # @param values [Array<String>]
-      # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def add_record(name, type, ttl, *values)
-        GandiV5::LiveDNS.require_valid_record_type type
-        fail ArgumentError, 'ttl must be positive and non-zero' unless ttl.positive?
-        fail ArgumentError, 'there must be at least one value' if values.none?
-
-        body = {
-          rrset_name: name,
-          rrset_type: type,
-          rrset_ttl: ttl,
-          rrset_values: values
-        }.to_json
-        data = GandiV5.post "#{url}/records", body
-        data['message']
-      end
-
-      # @overload delete_records()
-      #   Delete all records for this domain.
-      # @overload delete_records(name)
-      #   Delete records for a name.
-      #   @param name [String] the name to delete records for.
-      # @overload delete_records(name, type)
-      #   Delete records of a type for a name.
-      #   @param name [String] the name to delete records for.
-      #   @param type [String] the record type to delete.
-      # @return [nil]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def delete_records(name = nil, type = nil)
-        GandiV5::LiveDNS.require_valid_record_type(type) if type
-
-        url_ = "#{url}/records"
-        url_ += "/#{CGI.escape name}" if name
-        url_ += "/#{CGI.escape type}" if type
-        GandiV5.delete url_
-      end
-
-      # Replace all records for this domain.
-      # @param records
-      #   [Array<Hash<:name, :type => String, :ttl => Integer, :values => Array<String>>>]
-      #   the records to add.
-      # @param text [String] zone file lines to replace the records with.
-      # @return [String] The confirmation message from Gandi.
-      # @raise [ArgumentError] if neither/both of records & test are passed.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def replace_records(records: nil, text: nil)
-        unless [records, text].count(&:nil?).eql?(1)
-          fail ArgumentError, 'you must pass ONE of records: or text:'
-        end
-
-        if records
-          body = {
-            items: records.map { |r| r.transform_keys { |k| "rrset_#{k}" } }
-          }.to_json
-          data = GandiV5.put "#{url}/records", body
-        elsif text
-          data = GandiV5.put "#{url}/records", text, 'content-type': 'text/plain'
-        end
-        data['message']
-      end
-
-      # Replace records for a name in this domain.
-      # @param name [String]
-      # @param records
-      #   [Array<Hash<type: String, ttl: Integer, values: Array<String>>>]
-      #   the records to add.
-      # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def replace_records_for(name, *records)
-        body = {
-          items: records.map { |r| r.transform_keys { |k| "rrset_#{k}" } }
-        }.to_json
-        data = GandiV5.put "#{url}/records/#{name}", body
-        data['message']
-      end
-
-      GandiV5::LiveDNS::RECORD_TYPES.each do |type|
-        # Replace records of a given type for a name in this domain.
-        # TODO: @param name [Type] description.
-        # TODO: @param ttl [Type] description.
-        # TODO: documentation for *values
-        # @return [String] The confirmation message from Gandi.
-        # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-        define_method "replace_#{type.downcase}_records_for" do |name, ttl, *values|
-          body = {
-            rrset_ttl: ttl,
-            rrset_values: values
-          }.to_json
-          data = GandiV5.put "#{url}/records/#{name}/#{type}", body
-          data['message']
-        end
-      end
-
       # Change the zone used by this domain.
       # @param uuid [String, #uuid, #to_s] the UUID of the zone this domain should now use.
       # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def change_zone(uuid)
         uuid = uuid.uuid if uuid.respond_to?(:uuid)
-        data = GandiV5.patch url, { zone_uuid: uuid }.to_json
+        _response, data = GandiV5.patch url, { zone_uuid: uuid }.to_json
         self.zone_uuid = uuid
         data['message']
       end
 
+      # @see GandiV5::LiveDNS::Zone.fetch
+      def fetch_zone
+        GandiV5::LiveDNS::Zone.fetch zone_uuid
+      end
+
+      # The domain's zone (fetching from Gandi if required).
+      # @return [GandiV5::LiveDNS::Zone]
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def zone
+        @zone ||= fetch_zone
+      end
+
       # List the domains.
       # @return [Array<GandiV5::LiveDNS::Domain>]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def self.list
-        data = GandiV5.get url
+        _response, data = GandiV5.get url
         data.map { |item| from_gandi item }
       end
 
       # Get a domain.
       # @param fqdn [String, #to_s] the fully qualified domain name to fetch.
       # @return [GandiV5::LiveDNS::Domain]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def self.fetch(fqdn)
-        data = GandiV5.get url(fqdn)
+        _response, data = GandiV5.get url(fqdn)
         from_gandi data
       end
 

data/lib/gandi_v5/live_dns/has_zone_records.rb

@@ -0,0 +1,153 @@
+# frozen_string_literal: true
+
+class GandiV5
+  class LiveDNS
+    # Methods for handling record related requests in both
+    # GandiV5::LiveDNS::Domain and GandiV5::LiveDNS::Zone.
+    module HasZoneRecords
+      # @overload fetch_records()
+      #   Fetch all records for this domain.
+      # @overload fetch_records(name)
+      #   Fetch records for a name.
+      #   @param name [String] the name to fetch records for.
+      # @overload fetch_records(name, type)
+      #   Fetch records of a type for a name.
+      #   @param name [String] the name to fetch records for.
+      #   @param type [String] the record type to fetch.
+      # @return [Array<GandiV5::LiveDNS::RecordSet>]
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def fetch_records(name = nil, type = nil)
+        GandiV5::LiveDNS.require_valid_record_type type if type
+
+        url_ = "#{url}/records"
+        url_ += "/#{CGI.escape name}" if name
+        url_ += "/#{CGI.escape type}" if type
+
+        _response, data = GandiV5.get url_
+        data = [data] unless data.is_a?(Array)
+        data.map { |item| GandiV5::LiveDNS::RecordSet.from_gandi item }
+      end
+
+      # @overload fetch_zone_lines()
+      #   Fetch all records for this domain.
+      # @overload fetch_zone_lines(name)
+      #   Fetch records for a name.
+      #   @param name [String] the name to fetch records for.
+      # @overload fetch_zone_lines(name, type)
+      #   Fetch records of a type for a name.
+      #   @param name [String] the name to fetch records for.
+      #   @param type [String] the record type to fetch.
+      # @return [String]
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def fetch_zone_lines(name = nil, type = nil)
+        GandiV5::LiveDNS.require_valid_record_type type if type
+
+        url_ = "#{url}/records"
+        url_ += "/#{CGI.escape name}" if name
+        url_ += "/#{CGI.escape type}" if type
+
+        GandiV5.get(url_, accept: 'text/plain').last
+      end
+
+      # Add record to this domain.
+      # @param name [String]
+      # @param type [String]
+      # @param ttl [Integer]
+      # @param values [Array<String>]
+      # @return [String] The confirmation message from Gandi.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def add_record(name, type, ttl, *values)
+        GandiV5::LiveDNS.require_valid_record_type type
+        fail ArgumentError, 'ttl must be positive and non-zero' unless ttl.positive?
+        fail ArgumentError, 'there must be at least one value' if values.none?
+
+        body = {
+          rrset_name: name,
+          rrset_type: type,
+          rrset_ttl: ttl,
+          rrset_values: values
+        }.to_json
+        _response, data = GandiV5.post "#{url}/records", body
+        data['message']
+      end
+
+      # @overload delete_records()
+      #   Delete all records for this domain.
+      # @overload delete_records(name)
+      #   Delete records for a name.
+      #   @param name [String] the name to delete records for.
+      # @overload delete_records(name, type)
+      #   Delete records of a type for a name.
+      #   @param name [String] the name to delete records for.
+      #   @param type [String] the record type to delete.
+      # @return [nil]
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def delete_records(name = nil, type = nil)
+        GandiV5::LiveDNS.require_valid_record_type(type) if type
+
+        url_ = "#{url}/records"
+        url_ += "/#{CGI.escape name}" if name
+        url_ += "/#{CGI.escape type}" if type
+        GandiV5.delete(url_).last
+      end
+
+      # Replace all records for this domain.
+      # @param records
+      #   [Array<Hash<:name, :type => String, :ttl => Integer, :values => Array<String>>>]
+      #   the records to add.
+      # @param text [String] zone file lines to replace the records with.
+      # @return [String] The confirmation message from Gandi.
+      # @raise [ArgumentError] if neither/both of records & test are passed.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def replace_records(records: nil, text: nil)
+        unless [records, text].count(&:nil?).eql?(1)
+          fail ArgumentError, 'you must pass ONE of records: or text:'
+        end
+
+        if records
+          body = {
+            items: records.map { |r| r.transform_keys { |k| "rrset_#{k}" } }
+          }.to_json
+          _response, data = GandiV5.put "#{url}/records", body
+        elsif text
+          _response, data = GandiV5.put "#{url}/records", text, 'content-type': 'text/plain'
+        end
+        data['message']
+      end
+
+      # @override replace_records_for(name, records)
+      #   Replace records for a name in this domain.
+      #   @param name [String]
+      #   @param records
+      #     [Array<Hash<type: String, ttl: Integer, values: Array<String>>>]
+      #     the records to add.
+      # @override replace_records_for(name, values, type: nil, ttl: nil)
+      #   Replace records for a name in this domain.
+      #   @param name [String]
+      #   @param type [String] the record type.
+      #   @param ttl [Integer] the TTL to set for the record.
+      #   @param values [Array<String>] the values to set for the record.
+      #   @raise [ArgumentError] if ttl is present and type is absent.
+      # @return [String] The confirmation message from Gandi.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
+      def replace_records_for(name, records, type: nil, ttl: nil)
+        fail ArgumentError, 'missing keyword: type' if ttl && type.nil?
+
+        if type
+          GandiV5::LiveDNS.require_valid_record_type type
+          body = { rrset_values: records, rrset_ttl: ttl }
+          # body[:rrset_ttl] = ttl if ttl
+          _response, data = GandiV5.put "#{url}/records/#{name}/#{type}", body.to_json
+
+        else
+          body = {
+            items: records.map { |r| r.transform_keys { |k| "rrset_#{k}" } }
+          }
+          _response, data = GandiV5.put "#{url}/records/#{name}", body.to_json
+        end
+
+        data['message']
+      end
+    end
+  end
+end

data/lib/gandi_v5/live_dns/record_set.rb

@@ -15,7 +15,7 @@ class GandiV5
       include GandiV5::Data
 
       member :type, gandi_key: 'rrset_type'
-      member :ttl, gandi_key: 'rrset_ttl'
+      member :ttl, gandi_key: 'rrset_ttl', converter: GandiV5::Data::Converter::Integer
       member :name, gandi_key: 'rrset_name'
       member :values, gandi_key: 'rrset_values'
 

data/lib/gandi_v5/live_dns/zone.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative 'zone/snapshot'
-
 class GandiV5
   class LiveDNS
     # A zone within the LiveDNS system.
@@ -28,6 +26,7 @@ class GandiV5
     #   @return [String] primary name server, as reported in SOA record.
     class Zone
       include GandiV5::Data
+      include GandiV5::LiveDNS::HasZoneRecords
 
       members :uuid, :name
       member :sharing_uuid, gandi_key: 'sharing_id'
@@ -53,150 +52,11 @@ class GandiV5
         ')'
       end
 
-      # @overload fetch_records()
-      #   Fetch all records for this zone.
-      # @overload fetch_records(name)
-      #   Fetch records for a name.
-      #   @param name [String] the name to fetch records for.
-      # @overload fetch_records(name, type)
-      #   Fetch records of a type for a name.
-      #   @param name [String] the name to fetch records for.
-      #   @param type [String] the record type to fetch.
-      # @return [Array<GandiV5::LiveDNS::RecordSet>]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def fetch_records(name = nil, type = nil)
-        GandiV5::LiveDNS.require_valid_record_type type if type
-
-        url_ = "#{url}/records"
-        url_ += "/#{CGI.escape name}" if name
-        url_ += "/#{CGI.escape type}" if type
-
-        data = GandiV5.get url_
-        data = [data] unless data.is_a?(Array)
-        data.map { |item| GandiV5::LiveDNS::RecordSet.from_gandi item }
-      end
-
-      # @overload fetch_zone_lines()
-      #   Fetch all records for this zone.
-      # @overload fetch_zone_lines(name)
-      #   Fetch records for a name.
-      #   @param name [String] the name to fetch records for.
-      # @overload fetch_zone_lines(name, type)
-      #   Fetch records of a type for a name.
-      #   @param name [String] the name to fetch records for.
-      #   @param type [String] the record type to fetch.
-      # @return [String]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def fetch_zone_lines(name = nil, type = nil)
-        GandiV5::LiveDNS.require_valid_record_type type if type
-
-        url_ = "#{url}/records"
-        url_ += "/#{CGI.escape name}" if name
-        url_ += "/#{CGI.escape type}" if type
-
-        GandiV5.get url_, accept: 'text/plain'
-      end
-
-      # Add record to this zone.
-      # @param name [String]
-      # @param type [String]
-      # @param ttl [Integer]
-      # @param values [Array<String>]
-      # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def add_record(name, type, ttl, *values)
-        GandiV5::LiveDNS.require_valid_record_type type
-        fail ArgumentError, 'ttl must be positive and non-zero' unless ttl.positive?
-        fail ArgumentError, 'there must be at least one value' if values.none?
-
-        body = {
-          rrset_name: name,
-          rrset_type: type,
-          rrset_ttl: ttl,
-          rrset_values: values
-        }.to_json
-        data = GandiV5.post "#{url}/records", body
-        data['message']
-      end
-
-      # @overload delete_records()
-      #   Delete all records for this zone.
-      # @overload delete_records(name)
-      #   Delete records for a name.
-      #   @param name [String] the name to delete records for.
-      # @overload delete_records(name, type)
-      #   Delete records of a type for a name.
-      #   @param name [String] the name to delete records for.
-      #   @param type [String] the record type to delete.
-      # @return [nil]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def delete_records(name = nil, type = nil)
-        GandiV5::LiveDNS.require_valid_record_type type if type
-
-        url_ = "#{url}/records"
-        url_ += "/#{CGI.escape name}" if name
-        url_ += "/#{CGI.escape type}" if type
-        GandiV5.delete url_
-      end
-
-      # Replace all records for this zone.
-      # @param records
-      #   [Array<Hash<:name, :type => String, :ttl => Integer, :values => Array<String>>>]
-      #   the records to add.
-      # @param text [String] zone file lines to replace the records with.
-      # @return [String] The confirmation message from Gandi.
-      # @raise [ArgumentError] if neither/both of records & test are passed.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def replace_records(records: nil, text: nil)
-        unless [records, text].count(&:nil?).eql?(1)
-          fail ArgumentError, 'you must pass ONE of records: or text:'
-        end
-
-        if records
-          body = {
-            items: records.map { |r| r.transform_keys { |k| "rrset_#{k}" } }
-          }.to_json
-          data = GandiV5.put "#{url}/records", body
-        elsif text
-          data = GandiV5.put "#{url}/records", text, 'content-type': 'text/plain'
-        end
-        data['message']
-      end
-
-      # Replace records for a name in this zone.
-      # @param name [String]
-      # @param records
-      #   [Array<Hash<type: String, ttl: Integer, values: Array<String>>>]
-      #   the records to add.
-      # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      def replace_records_for(name, *records)
-        body = {
-          items: records.map { |r| r.transform_keys { |k| "rrset_#{k}" } }
-        }.to_json
-        data = GandiV5.put "#{url}/records/#{name}", body
-        data['message']
-      end
-
-      GandiV5::LiveDNS::RECORD_TYPES.each do |type|
-        # Replace records of a given type for a name in this zone.
-        # @return [String] The confirmation message from Gandi.
-        # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-        define_method "replace_#{type.downcase}_records_for" do |name, ttl, *values|
-          body = {
-            rrset_ttl: ttl,
-            rrset_values: values
-          }.to_json
-          data = GandiV5.put "#{url}/records/#{name}/#{type}", body
-          data['message']
-        end
-      end
-
       # List the domains that use this zone.
       # @return [Array<String>] The FQDNs.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def list_domains
-        data = GandiV5.get "#{url}/domains"
+        _response, data = GandiV5.get "#{url}/domains"
         data.map { |item| item['fqdn'] }
       end
 
@@ -204,50 +64,49 @@ class GandiV5
       # @param fqdn [String, #fqdn, #to_s] the fully qualified domain name
       #   that should start using this zone.
       # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def attach_domain(fqdn)
         fqdn = fqdn.fqdn if fqdn.respond_to?(:fqdn)
-        data = GandiV5.patch "#{BASE}domains/#{CGI.escape fqdn}", { zone_uuid: uuid }.to_json
+        _resp, data = GandiV5.patch "#{BASE}domains/#{CGI.escape fqdn}", { zone_uuid: uuid }.to_json
         data['message']
       end
 
-      # Get snapshot UUIDs for this zone from Gandi.lib/gandi_v5/domain/auto_renew.rb
-      # @return [Hash{String => Time}] Mapping UUID to time made.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # Get snapshot UUIDs for this zone from Gandi.
+      # @return [Hash{String => Time}] Mapping snapshot UUID to time made.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def snapshots
-        data = GandiV5.get "#{url}/snapshots"
-        Hash[data.map { |snapshot| [snapshot['uuid'], Time.parse(snapshot['date_created'])] }]
+        GandiV5::LiveDNS::Zone::Snapshot.list uuid
       end
 
       # Get snapshot from Gandi.
       # @param uuid [String] the UUID of the snapshot to fetch.
       # @return [GandiV5::LiveDNS::Zone::Snapshot]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def snapshot(uuid)
         GandiV5::LiveDNS::Zone::Snapshot.fetch self.uuid, uuid
       end
 
       # Take a snapshot of this zone.
       # @return [GandiV5::LiveDNS::Zone::Snapshot]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def take_snapshot
-        data = GandiV5.post "#{url}/snapshots"
+        _response, data = GandiV5.post "#{url}/snapshots"
         snapshot data['uuid']
       end
 
       # Update this zone.
       # @param name [String, #to_s] new name for the zone.
       # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def update(name:)
-        data = GandiV5.patch url, { name: name }.to_json
+        _response, data = GandiV5.patch url, { name: name }.to_json
         self.name = name
         data['message']
       end
 
       # Delete this zone from Gandi.
       # @return [nil]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def delete
         GandiV5.delete url
       end
@@ -255,34 +114,34 @@ class GandiV5
       # Create a new zone.
       # @param name [String] the name for the created zone.
       # @param sharing_id [String] the UUID of the account to ceate the zone under.
-      # @return [String] The confirmation message from Gandi.
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
-      # TODO: Fetch created zone
+      # @return [GandiV5::LiveDNS::Zone] The created zone.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def self.create(name, sharing_id: nil)
         params = sharing_id ? { sharing_id: sharing_id } : {}
 
-        data = GandiV5.post(
+        response, _data = GandiV5.post(
           url,
           { name: name }.to_json,
           params: params
         )
-        data['message']
+
+        fetch response.headers[:location].split('/').last
       end
 
       # List the zones.
       # @return [Array<GandiV5::LiveDNS::Zone>]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def self.list
-        data = GandiV5.get url
+        _response, data = GandiV5.get url
         data.map { |item| from_gandi item }
       end
 
       # Get a zone from Gandi.
       # @param uuid [String, #to_s] the UUID of the zone to fetch.
       # @return [GandiV5::LiveDNS::Zone]
-      # @raise [GandiV5::Error::GandiError::GandiError] if Gandi returns an error.
+      # @raise [GandiV5::Error::GandiError] if Gandi returns an error.
       def self.fetch(uuid)
-        data = GandiV5.get url(uuid)
+        _response, data = GandiV5.get url(uuid)
         from_gandi data
       end