validates_zipcode 0.5.3 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eaccf2cf4e72190040829da37f3da72cd319020b9e97c1dd3a064cb1b4d6b381
-  data.tar.gz: dac1daf1928badb6279f0007ae20ab969e854adacbd8e0a25d7e4ded91483125
+  metadata.gz: e698716ffc57c2d3fcdbfcda2dea3817842ed64b10c7739709a7cdad8914789a
+  data.tar.gz: a50c8edeeadf3aa499d92260214fd65cc6e2bb1ed34f866133eb5480ecb89d5c
 SHA512:
-  metadata.gz: 52810b93259d83ba29088bc3b3eb0596bb89f15d8d92f7b8947bf08333f7191fa7e37946dbed18d4045bb4d4fd0d574f73e1c94846bc2c95f39295a42e996616
-  data.tar.gz: 3129ba6f20cbe9850aa9448b98271bee9e0d60496fbd47ed9f4df55db896377aa20998832864d37388bc0c4d8c18b2ee19955ff4d4e0de89d4628061842c5861
+  metadata.gz: b50f55f7f3265cf5de4f56292cc4167cd860afa19e28752b830238bb9e622fdf6948f7da0c60dc5d004524a7595029e8cd45da272deb8978c98bf7be080737a6
+  data.tar.gz: 7750e6f341ca45e65cfa0c5fde71e2b5c451fe99a7fa910cfd6c6ab33695864baef875c041587fe5d451b32e8700cc8147bf54b2a75ef796e489e6c2abac8706

data/CHANGELOG.md

@@ -4,6 +4,14 @@ All notable changes to this project will be documented in this file.
 
 ## Unreleased
 
+## 0.6.0
+
+**Heads up:** This release changes the behaviour of `ValidatesZipcode.valid?` when an unknown or malformed country code is passed. Previously, passing a country code not recognised by the gem (e.g. `'UKXXXXX'`) would return `true` — no regex to match against meant no failure. It will now return `false`.
+
+If you were relying on the old behaviour (passing arbitrary country codes and expecting `true`), this is a breaking change for those cases. For everyone else, this is the expected fix — an unrecognised country is not a valid one.
+
+- Return `false` when the country code is not a known ISO 3166-1 alpha-2 code, instead of silently returning `true` — fixes #67
+
 ## 0.5.3
 
 - See commits up to 4e5a3132061d57d220d1616beb15d15a47d301df

data/lib/validates_zipcode/cldr_regex_collection.rb

@@ -80,7 +80,7 @@ module ValidatesZipcode
       NG: /\A\d{6}\z/,
       OM: /\A(PC )?\d{3}\z/i,
       PK: /\A\d{5}\z/,
-      PY: /\A\d{4}\z/,
+      PY: /\A(\d{4}|\d{6})\z/,
       PH: /\A\d{4}\z/,
       PL: /\A\d{2}[ -]?\d{3}\z/,
       PR: /\A00[679]\d{2}([ \-]\d{4})?\z/,
@@ -154,7 +154,7 @@ module ValidatesZipcode
       BI: /\A([A-Z\d\s]){3,}\z/i,
       BS: /\A([A-Z\d\s]){3,}\z/i,
       BZ: /\A([A-Z\d\s]){3,}\z/i,
-      BR: /\A\d{5}(-?\d{3})?\z/,
+      BR: /\A\d{5}(-?\d{3})\z/,
       BJ: /\A([A-Z\d\s]){3,}\z/i,
       BT: /\A\d{5}\z/,
       BQ: /\A([A-Z\d\s]){3,}\z/i,

data/lib/validates_zipcode/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module ValidatesZipcode
-  VERSION = '0.5.3'
+  VERSION = '0.6.0'
 end

data/lib/validates_zipcode/zipcode.rb

@@ -12,6 +12,7 @@ module ValidatesZipcode
 
     def valid?
       return true if @excluded_country_codes.include?(@country_alpha2)
+      return false unless valid_country_alpha2?
       return true unless regexp
 
       zipcode = @format ? formatted_zip : @zipcode
@@ -31,6 +32,13 @@ module ValidatesZipcode
       @regexp ||= regexp_for_country_alpha2(@country_alpha2)
     end
 
+    # ISO 3166-1 alpha-2 codes are exactly 2 uppercase ASCII letters.
+    # Anything longer or shorter (e.g. "UKXXXXX", "G", "12") is not a
+    # valid country code and should fail validation immediately.
+    def valid_country_alpha2?
+      @country_alpha2.to_s.match?(/\A[A-Za-z]{2}\z/)
+    end
+
     def formatted_zip
       @formatted_zip ||= Formatter.new(zipcode: @zipcode, country_alpha2: @country_alpha2).format
     end

data/spec/spec_helper.rb

@@ -554,9 +554,9 @@ TEST_DATA = {
   },
   KH: {
     valid: %w[
-      07113
-      02566
-      66261
+      071131
+      025662
+      662613
     ],
     invalid: [
       nil,
@@ -1117,6 +1117,7 @@ TEST_DATA = {
       2566
       6261
       7113
+      111421
     ],
     invalid: [
       nil,
@@ -2167,7 +2168,8 @@ TEST_DATA = {
       '',
       'invalid_zip',
       '1234',
-      '123456789'
+      '123456789',
+      '12345'
     ]
   },
   BJ: {

data/spec/validates_zipcode_spec.rb

@@ -82,6 +82,14 @@ describe ValidatesZipcode, '.valid?' do
       expect(ValidatesZipcode.valid?('12345', 'ZZ')).to eq(true)
     end
 
+    it "is false with a malformed country code that is not a valid ISO 3166-1 alpha-2 code" do
+      expect(ValidatesZipcode.valid?('Sw1A 2aA', 'UKXXXXX')).to eq(false)
+      expect(ValidatesZipcode.valid?('12345', 'USA')).to eq(false)    # 3 letters
+      expect(ValidatesZipcode.valid?('12345', 'U')).to eq(false)      # 1 letter
+      expect(ValidatesZipcode.valid?('12345', '12')).to eq(false)     # digits
+      expect(ValidatesZipcode.valid?('12345', '')).to eq(false)       # empty
+    end
+
     it "is true with an excluded country code -  we don't want those to fail and nil is hairy" do
       expect(ValidatesZipcode.valid?('XXXX!!!XXXX', 'PA', excluded_country_codes: %w[PA])).to eq(true)
     end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: validates_zipcode
 version: !ruby/object:Gem::Version
-  version: 0.5.3
+  version: 0.6.0
 platform: ruby
 authors:
 - David Gil
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-13 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -155,7 +154,6 @@ homepage: http://github.com/dgilperez/validates_zipcode
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -170,8 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.17
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Localizable zipcode validation for Rails.
 test_files: