avro_turf 1.11.0 → 1.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 31248ad33908f238baea49fc106bc3e6292f45a1fe7fce0c0ae091dc10a9ea17
-  data.tar.gz: 25af934023a269b28acf004a1ec13c052380f5a664696ce881ef27efb209a7e5
+  metadata.gz: 72327ac5cc542eaaf881cd7ab9fc6ad3189f488abdcd6119a39b7e51c9545e6e
+  data.tar.gz: e5af812736913725238ac20778151c706776cc619625e60ec6448460289827c8
 SHA512:
-  metadata.gz: 6e0796ce28027a654fa189f561f8179567fbebf50ba6c225be9b8623999e7ef424b58b05d1282a151474aaf48057b0000ce7999f6f8aed1c5f56f53f2d1f8cab
-  data.tar.gz: 70fb1dc693589e71691ab8bf18cef28ebe533921efe401c6c0f7c3f1ace8ac880e5e3bd1161bd3235148f7f469c837d5224f90ea96a3867aa0fe24a16de91200
+  metadata.gz: 404704ea23a71ccbfd34354904c3a45ef52abf6c6d75c67570355e67a83b33a3f1b694baf6134e5d539c062aebd7509ee65da5ca8a9684f973d749860288f66b
+  data.tar.gz: bd5ffed0531358a52c355d73994ab6daec49f8da69b96f914d0f75dfcf8f049a84c02d8ac6f3b25cabe416640cfb82130c97ebfa6253c2321cefdafca44b1a32

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## Unreleased
 
+## v1.13.0
+
+- Set `idempotent: true` for the request except one that registers a new schema (#199)
+- Use `connect_timeout` for `Excon`'s `dns_timeouts` that set the timeout for the connection to the Domain Name Server (#201)
+
+## v1.12.0
+
+- Add `connect_timeout` parameter to `AvroTurf::Messaging` to set the timeout for the connection to the schema registry (#197)
+
 ## v1.11.0
 
 - Add `decode_all` and `decode_all_from_stream` methods to return all entries in a data file (#194)

data/avro_turf.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "avro", ">= 1.8.0", "< 1.12"
-  spec.add_dependency "excon", "~> 0.71"
+  spec.add_dependency "excon", "~> 0.103"
 
   spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "rake", "~> 13.0"

data/lib/avro_turf/confluent_schema_registry.rb

@@ -15,7 +15,8 @@ class AvroTurf::ConfluentSchemaRegistry
     client_key_pass: nil,
     client_cert_data: nil,
     client_key_data: nil,
-    path_prefix: nil
+    path_prefix: nil,
+    connect_timeout: nil
   )
     @path_prefix = path_prefix
     @logger = logger
@@ -33,13 +34,15 @@ class AvroTurf::ConfluentSchemaRegistry
       client_key: client_key,
       client_key_pass: client_key_pass,
       client_cert_data: client_cert_data,
-      client_key_data: client_key_data
+      client_key_data: client_key_data,
+      connect_timeout: connect_timeout,
+      dns_timeouts: connect_timeout
     )
   end
 
   def fetch(id)
     @logger.info "Fetching schema with id #{id}"
-    data = get("/schemas/ids/#{id}")
+    data = get("/schemas/ids/#{id}", idempotent: true)
     data.fetch("schema")
   end
 
@@ -55,29 +58,30 @@ class AvroTurf::ConfluentSchemaRegistry
 
   # List all subjects
   def subjects
-    get('/subjects')
+    get('/subjects', idempotent: true)
   end
 
   # List all versions for a subject
   def subject_versions(subject)
-    get("/subjects/#{subject}/versions")
+    get("/subjects/#{subject}/versions", idempotent: true)
   end
 
   # Get a specific version for a subject
   def subject_version(subject, version = 'latest')
-    get("/subjects/#{subject}/versions/#{version}")
+    get("/subjects/#{subject}/versions/#{version}", idempotent: true)
   end
 
   # Get the subject and version for a schema id
   def schema_subject_versions(schema_id)
-    get("/schemas/ids/#{schema_id}/versions")
+    get("/schemas/ids/#{schema_id}/versions", idempotent: true)
   end
 
   # Check if a schema exists. Returns nil if not found.
   def check(subject, schema)
     data = post("/subjects/#{subject}",
                 expects: [200, 404],
-                body: { schema: schema.to_s }.to_json)
+                body: { schema: schema.to_s }.to_json,
+                idempotent: true)
     data unless data.has_key?("error_code")
   end
 
@@ -89,28 +93,28 @@ class AvroTurf::ConfluentSchemaRegistry
   # http://docs.confluent.io/3.1.2/schema-registry/docs/api.html#compatibility
   def compatible?(subject, schema, version = 'latest')
     data = post("/compatibility/subjects/#{subject}/versions/#{version}",
-                expects: [200, 404], body: { schema: schema.to_s }.to_json)
+                expects: [200, 404], body: { schema: schema.to_s }.to_json, idempotent: true)
     data.fetch('is_compatible', false) unless data.has_key?('error_code')
   end
 
   # Get global config
   def global_config
-    get("/config")
+    get("/config", idempotent: true)
   end
 
   # Update global config
   def update_global_config(config)
-    put("/config", body: config.to_json)
+    put("/config", body: config.to_json, idempotent: true)
   end
 
   # Get config for subject
   def subject_config(subject)
-    get("/config/#{subject}")
+    get("/config/#{subject}", idempotent: true)
   end
 
   # Update config for subject
   def update_subject_config(subject, config)
-    put("/config/#{subject}", body: config.to_json)
+    put("/config/#{subject}", body: config.to_json, idempotent: true)
   end
 
   private

data/lib/avro_turf/messaging.rb

@@ -55,6 +55,7 @@ class AvroTurf
     # client_key_pass      - Password to go with client_key (optional).
     # client_cert_data     - In-memory client certificate (optional).
     # client_key_data      - In-memory client private key to go with client_cert_data (optional).
+    # connect_timeout      - Timeout to use in the connection with the domain name server and the schema registry (optional).
     def initialize(
       registry: nil,
       registry_url: nil,
@@ -71,7 +72,8 @@ class AvroTurf
       client_key: nil,
       client_key_pass: nil,
       client_cert_data: nil,
-      client_key_data: nil
+      client_key_data: nil,
+      connect_timeout: nil
     )
       @logger = logger || Logger.new($stderr)
       @namespace = namespace
@@ -89,7 +91,8 @@ class AvroTurf
           client_key_pass: client_key_pass,
           client_cert_data: client_cert_data,
           client_key_data: client_key_data,
-          path_prefix: registry_path_prefix
+          path_prefix: registry_path_prefix,
+          connect_timeout: connect_timeout
         )
       )
       @schemas_by_id = {}

data/lib/avro_turf/version.rb

@@ -1,3 +1,3 @@
 class AvroTurf
-  VERSION = "1.11.0"
+  VERSION = "1.13.0"
 end

data/spec/confluent_schema_registry_spec.rb

@@ -8,6 +8,7 @@ describe AvroTurf::ConfluentSchemaRegistry do
   let(:client_cert) { "test client cert" }
   let(:client_key) { "test client key" }
   let(:client_key_pass) { "test client key password" }
+  let(:connect_timeout) { 10 }
 
   it_behaves_like "a confluent schema registry client" do
     let(:registry) {
@@ -30,4 +31,15 @@ describe AvroTurf::ConfluentSchemaRegistry do
       )
     }
   end
+
+  it_behaves_like "a confluent schema registry client" do
+    let(:registry) {
+      described_class.new(
+        registry_url,
+        user: user,
+        password: password,
+        connect_timeout: connect_timeout
+      )
+    }
+  end
 end

data/spec/messaging_spec.rb

@@ -425,4 +425,27 @@ describe AvroTurf::Messaging do
     it_behaves_like 'encoding and decoding with the schema from registry'
     it_behaves_like 'encoding and decoding with the schema_id from registry'
   end
-end
+
+  context 'with a connect timeout' do
+    let(:avro) {
+      AvroTurf::Messaging.new(
+        registry_url: registry_url,
+        schemas_path: "spec/schemas",
+        logger: logger,
+        client_cert: client_cert,
+        client_key: client_key,
+        client_key_pass: client_key_pass,
+        connect_timeout: 10
+      )
+    }
+
+    it_behaves_like "encoding and decoding with the schema from schema store"
+    it_behaves_like 'encoding and decoding with the schema from registry'
+    it_behaves_like 'encoding and decoding with the schema_id from registry'
+
+    it 'passes the connect timeout setting to Excon' do
+      expect(Excon).to receive(:new).with(anything, hash_including(connect_timeout: 10, dns_timeouts: 10)).and_call_original
+      avro
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: avro_turf
 version: !ruby/object:Gem::Version
-  version: 1.11.0
+  version: 1.13.0
 platform: ruby
 authors:
 - Daniel Schierbeck
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-29 00:00:00.000000000 Z
+date: 2023-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: avro
@@ -36,14 +36,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.71'
+        version: '0.103'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.71'
+        version: '0.103'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement