bzync-nextsql 0.0.1 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 22fcc7ed7950fdad5c655543af1d813989ac8a76e291e2e1ff3a699268af052c
-  data.tar.gz: e90a274dced110f5883f471c6daa4caa60d8757164962b11fdb778df843b9796
+  metadata.gz: 7f8ea5d62b319951e5324da7495acd2b082dc3c05945b9c7cd8810f81ab204ed
+  data.tar.gz: 773874cbee89f62cea0754249800a087f8de553e79a90c7f0d5d6e85026f0a39
 SHA512:
-  metadata.gz: 8adc88df55005abbf9b1d643dc6d026d8786d4e954813327a0ddc84adb8cf27a1c4a22ed64b552ff2e3ddf9490cafdb3694ddf494392cbb5f74671a11a8d5df0
-  data.tar.gz: 43e99ed4daf17b81e83eeba3ef2cb1c715679061534159bf712a5e6e433d6764949c2d477a8908437f7ee0b5bfad4bfb7ca9656cd90938b8a06a405fedb21e94
+  metadata.gz: 4c6cd6f4f39cd9aceedaaf0a373e8deedbe7a217ea9fdd0e04231bfc28409eeceb8cd84294b776ec1a48d8621fdf797e5007bc4203ff9bec950d95799333c128
+  data.tar.gz: 302b1dcfc03a25f1ac6ef126d99c9a1cfcb0c2c04dbf44414f8e4558e20b9029c8c0c5f2153f4f0de4326ff8ccb421944970ddb25412822d6cfc01887296050b

data/lib/nextsql/client.rb

@@ -6,7 +6,6 @@
 
 require "socket"
 require "openssl"
-require "ipaddr"
 
 require_relative "protocol"
 require_relative "errors"
@@ -41,6 +40,7 @@ module NextSQL
   # from multiple threads/fibers — open one Connection per worker, or use
   # +Cluster+ which pools one connection per node.
   class Connection
+    LOOPBACK_RE = /\A127\.\d{1,3}\.\d{1,3}\.\d{1,3}\z/.freeze
     CONNECT_TIMEOUT = 10.0
 
     class << self
@@ -73,12 +73,9 @@ module NextSQL
         host, = split_host_port(addr, allow_bare: true)
         host = host.strip.downcase
         return true if host == "localhost"
+        return true if %w[::1 0:0:0:0:0:0:0:1].include?(host)
 
-        begin
-          IPAddr.new(host).loopback?
-        rescue IPAddr::Error
-          false
-        end
+        LOOPBACK_RE.match?(host)
       end
 
       def validate_config!(cfg)
@@ -372,17 +369,12 @@ module NextSQL
 
     def handshake
       cfg = @cfg
-      write_frame(Protocol::TYPE_HELLO,
-                  Protocol.encode_hello(Protocol::VERSION, Protocol::FLAG_PUBLIC_ERROR_CODES, "\x00" * 8,
-                                        cfg.database, cfg.user, cfg.realm))
+      write_frame(Protocol::TYPE_HELLO, Protocol.encode_hello(Protocol::VERSION, 0, "\x00" * 8, cfg.database, cfg.user, cfg.realm))
       typ, payload = read_frame
       raise unexpected(typ, payload) if typ != Protocol::TYPE_HELLO_OK
 
-      _version, auth_method, secret, flags = Protocol.decode_hello_ok(payload)
+      _version, auth_method, secret = Protocol.decode_hello_ok(payload)
       @secret = secret
-      # Diagnostic only: decode_error reads the field whenever it is present,
-      # so nothing depends on this having been echoed.
-      @public_error_codes = (flags & Protocol::FLAG_PUBLIC_ERROR_CODES) != 0
       write_frame(Protocol::TYPE_AUTH, Protocol.u16str(cfg.password))
       typ, payload = read_frame
       raise unexpected(typ, payload) if typ != Protocol::TYPE_AUTH_OK

data/lib/nextsql/errors.rb

@@ -10,16 +10,8 @@ module NextSQL
   class Error < StandardError
     attr_reader :error_code
 
-    # +public_code+ is the stable ERR_* name from docs/error-codes.md. It is
-    # set only on an error received over a connection that negotiated the
-    # public taxonomy, and is "" otherwise -- for a local error, or an older
-    # server. +error_code+ always carries the legacy class, so retry logic
-    # keeps reading that and nothing should require +public_code+.
-    attr_reader :public_code
-
-    def initialize(error_code, message = "", public_code = "")
+    def initialize(error_code, message = "")
       @error_code = error_code
-      @public_code = public_code
       super(message.empty? ? error_code : message)
     end
   end

data/lib/nextsql/protocol.rb

@@ -53,12 +53,6 @@ module NextSQL
     AUTH_PASSWORD = 1
     AUTH_PASSWORD_KEY = 2
     FLAG_CANCEL = 1
-    # FLAG_PUBLIC_ERROR_CODES asks the server for the stable ERR_* error
-    # taxonomy (docs/error-codes.md). A server that does not implement it
-    # ignores the bit and keeps the NSQL v1 error shape, which decode_error
-    # still reads, so setting it is safe against any server version. The server
-    # echoes it in the hello-ok flags when it was accepted.
-    FLAG_PUBLIC_ERROR_CODES = 2
     FLAG_NULL = 0x01
 
     KIND_UUID = 1
@@ -319,16 +313,9 @@ module NextSQL
     end
 
     def decode_hello_ok(b)
-      raise ProtocolError, "bad hello-ok length" unless [11, 13].include?(b.bytesize)
-
-      flags = 0
-      if b.bytesize == 13
-        flags = u16(b, 11)
-        # The server omits the field when it accepted no capability, so a
-        # present-but-zero field is a second encoding of the v1 hello-ok.
-        raise ProtocolError, "empty hello-ok flags" if flags.zero?
-      end
-      [u16(b, 0), b.getbyte(2), b.byteslice(3, 8), flags]
+      raise ProtocolError, "bad hello-ok length" unless b.bytesize == 11
+
+      [u16(b, 0), b.getbyte(2), b.byteslice(3, 8)]
     end
 
     def encode_query(sql, params)
@@ -959,16 +946,8 @@ module NextSQL
 
     def decode_error(b)
       code, off = read_u16_string(b, 0, MAX_NAME)
-      msg, off = read_u16_string(b, off, MAX_NAME)
-      # Optional trailing field, present only from a server that accepted
-      # FLAG_PUBLIC_ERROR_CODES. Its absence is normal -- an older server
-      # ignores the request bit -- so this must never be required.
-      public_code = ""
-      if off < b.bytesize
-        public_code, = read_u16_string(b, off, MAX_NAME)
-        raise ProtocolError, "empty public error code" if public_code.empty?
-      end
-      Error.new(code, msg, public_code)
+      msg, = read_u16_string(b, off, MAX_NAME)
+      Error.new(code, msg)
     end
 
     def encode_set_read_consistency(mode, max_staleness_ms)

data/lib/nextsql/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module NextSQL
-  VERSION = "0.0.1"
+  VERSION = "0.1.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bzync-nextsql
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
 platform: ruby
 authors:
 - Rayan Reynaldo