stun-client 1.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9ea649544689d1e375c9df68c7d89f27453ff2aee4c22ed21ebc305c9dfd2763
-  data.tar.gz: ea80323fdf18d49c55a1c3ace5c6ba82b8729c6b116ff615f4e0faf8fe498ef3
+  metadata.gz: b3a3dc75283f39519208d5be9521167433cb2a74542193cd47f70e8ce0e012dc
+  data.tar.gz: d30d85146266df9e4a6c05deb5aff18754fdfa06ff7197bf763cf77638877303
 SHA512:
-  metadata.gz: f7df8d98b38a4bf969bd604959eb4e3424fb24214e1e4dc9483e19bf7aeee8c254ecb6c924914f42f9e98528e118898a3ab6c7c18ffc090709d876d1c741fdb6
-  data.tar.gz: 268abe533cf78836373dfc7a95f7af84cda388fc1d0eb6ede0295bbe570d26e1c3957c36d4e981845ee460889637d904c14f9003e770a376d4d05816203668d6
+  metadata.gz: 1a357fb2721d222c4661175d5bf9c2da82b18172d249393ae99ee4223cd9f8850981b4574ef605144ba382e0c958babe8d0fcc026c9a7fb89c1bc60dbeaf292c
+  data.tar.gz: 77c64711ef67b299f3c1b4dc4c54914d38d23b066dddc7aa2ec840345d31bdabc0e6f45b7fd93a53958d939e446c48dee0cfcf3002c17317f7c1740631776eb3

data/lib/stun-client/module.rb

@@ -4,5 +4,5 @@
 # The StunClient library
 module StunClient
   # Contains the latest version of STUN Client.
-  VERSION = 1.0
+  VERSION = "1.0.1"
 end

data/lib/stun-client/rfc3489/discovery/nat.rb

@@ -28,7 +28,7 @@ module StunClient
     #                          are `:IPv4` or `:IPv6`.
     def initialize host = nil, port = nil, protocol = :IPv4
       raise ArgumentError, 'Host must be a string' if host && (! host.is_a? String)
-      raise ArgumentError, 'Port must be a integer' if port && (! host.is_a? Integer)
+      raise ArgumentError, 'Port must be a integer' if port && (! port.is_a? Integer)
 
       case protocol
       when :IPv6
@@ -114,16 +114,11 @@ module StunClient
           if test2
             flags << :full_cone_nat
           else
-            # Run test 3 before test 1 variation 2, otherwise false NATs might be detected.
-            # This occurs when the changed port is the same as the port the server responds
-            # with in the change port request. Since you have already punched a hole in the
-            # NAT with the previous request, the response from the STUN server gets through.
-            test3 = do_query @host, @port, change_port: true
-
-            test1v2 = do_query test1v1[:changed_address], test1v1[:changed_port].to_i
+            test1v2 = do_query test1v1[:changed_host], test1v1[:changed_port].to_i
             if test1v2
               if test1v1[:host] == test1v2[:host] &&
                  test1v1[:port] == test1v2[:port]
+                test3 = do_query @host, @port, change_port: true
                 if test3
                   flags << :restricted_nat
                 else

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stun-client
 version: !ruby/object:Gem::Version
-  version: '1.0'
+  version: 1.0.1
 platform: ruby
 authors:
 - Marek Küthe