vcard 0.2.13 → 0.2.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8cd694341dcc9a8deb453629cdcadb62beb83d2c
-  data.tar.gz: 8f02acb17425244c540188442c18db6a35936334
+  metadata.gz: dab62931019aa50589848685e0ece97fce22a816
+  data.tar.gz: da5844c2ec7d8fff390055b6ae247845d0c4111f
 SHA512:
-  metadata.gz: dc5f7ad6dd56250ceaf3f76315cc51b8ce5eca0664b63f1c28ef9b6e5a534cc84b7b5eb5b043c08ff9981b12ab8b5f42167af614159e9d6c297ab4b5f1920566
-  data.tar.gz: b7452473ea636bf33f0f01cf6a21856ae112e7bbccc2c5591b5ed72f33eff58307fd83aaec7a379d80d49a832868797f4cb2af1c5bbe3724d5c9e7e1228c4745
+  metadata.gz: cfd6decb35dcd2a61aea730f5ded93c7555c1a5cb1796a98fcf897172f8f3b0e4116f4021efb4924d803132223ed875833dc96c55504058b7bf30cb46703a22c
+  data.tar.gz: 986f18eb24dea4885a53916969fe7161dc8dd05f0b291614250a1106ebc09453c742f45210a4fcc2b2fb22cd80425ef41f139878a679b4db6e9f45da9cca3773

data/README.md

@@ -16,6 +16,30 @@ Or install it yourself as:
 
     $ gem install vcard
 
+## Configuration
+
+You can configure how to deal with invalid lines. The gem supports three behaviours:
+
+1. `raise_on_invalid_line = true`
+
+ Vcard::InvalidEncodingError will be raised if any invalid line is found.
+
+2. `raise_on_invalid_line = false, ignore_invalid_vcards = true`
+
+ If the vcard source has an invalid line, this vcard object will be ignored.
+ If you have only one vcard object in your source string, an empty array will be returned from `Vcard.decode`.
+
+3. `raise_on_invalid_line = false, ignore_invalid_vcards = false`
+
+ If the vcard is marked as invalid, invalid fields will be ignored, but the vcard will be present in the results of `Vcard#decode`.
+
+```
+Vcard.configure do |config|
+  config.raise_on_invalid_line = false # default true
+  config.ignore_invalid_vcards = false # default true
+end
+```
+
 ## Upgrade Notes
 
 We are no longer testing against Ruby 1.8.7.

data/lib/vcard.rb

@@ -7,6 +7,7 @@ require "date"
 require "open-uri"
 require "stringio"
 
+require "vcard/configuration"
 require "vcard/attachment"
 require "vcard/bnf"
 require "vcard/dirinfo"
@@ -285,4 +286,13 @@ module Vcard
     end
     return outer, inner
   end
+
+  def configuration
+    @configuration ||= Configuration.new
+  end
+
+  def configure
+    yield configuration
+  end
+
 end

data/lib/vcard/configuration.rb

@@ -0,0 +1,16 @@
+module Vcard
+  class Configuration
+
+    attr_accessor :raise_on_invalid_line
+    alias_method :raise_on_invalid_line?, :raise_on_invalid_line
+
+    attr_accessor :ignore_invalid_vcards
+    alias_method :ignore_invalid_vcards?, :ignore_invalid_vcards
+
+    def initialize
+      @raise_on_invalid = true
+      @ignore_invalid_vcard = true
+    end
+
+  end
+end

data/lib/vcard/dirinfo.rb

@@ -31,16 +31,27 @@ module Vcard
     # Initialize a DirectoryInfo object from +fields+. If +profile+ is
     # specified, check the BEGIN/END fields.
     def initialize(fields, profile = nil) #:nodoc:
-      if fields.detect { |f| ! f.kind_of? DirectoryInfo::Field }
-        raise ArgumentError, "fields must be an array of DirectoryInfo::Field objects"
+      @valid = true
+      @fields = []
+
+      fields.each do |f|
+        raise ArgumentError, "fields must be an array of DirectoryInfo::Field objects" unless f.kind_of? DirectoryInfo::Field
+        if f.valid?
+          @fields << f
+        else
+          @valid = false
+        end
       end
 
       @string = nil # this is used as a flag to indicate that recoding will be necessary
-      @fields = fields
 
       check_begin_end(profile) if profile
     end
 
+    def valid?
+      @valid
+    end
+
     # Decode +card+ into a DirectoryInfo object.
     #
     # +card+ may either be a something that is convertible to a string using

data/lib/vcard/field.rb

@@ -101,10 +101,12 @@ module Vcard
         line
       end
 
+
       # Decode a field.
       def Field.decode0(atline) # :nodoc:
         if !(atline =~ Bnf::LINE)
-          raise ::Vcard::InvalidEncodingError, atline
+          raise(::Vcard::InvalidEncodingError, atline) if ::Vcard.configuration.raise_on_invalid_line?
+          return false
         end
 
         atgroup = $1.upcase
@@ -165,19 +167,27 @@ module Vcard
           end
         end
 
-        [ atgroup, atname, atparams, atvalue ]
+        [ true, atgroup, atname, atparams, atvalue ]
       end
 
       def initialize(line) # :nodoc:
         @line = line.to_str
-        @group, @name, @params, @value = Field.decode0(@line)
+        @valid, @group, @name, @params, @value = Field.decode0(@line)
 
-        @params.each do |pname,pvalues|
-          pvalues.freeze
+        if valid?
+          @params.each do |pname,pvalues|
+            pvalues.freeze
+          end
+        else
+          @group = @name = ''
         end
         self
       end
 
+      def valid?
+        @valid
+      end
+
       # Create a field by decoding +line+, a String which must already be
       # unfolded. Decoded fields are frozen, but see #copy().
       def Field.decode(line)
@@ -323,7 +333,7 @@ module Vcard
 
       # Is the #name of this Field +name+? Names are case insensitive.
       def name?(name)
-        @name.casecmp(name) == 0
+        @name.to_s.casecmp(name) == 0
       end
 
       # Is the #group of this field +group+? Group names are case insensitive.
@@ -590,7 +600,7 @@ module Vcard
       # new fields, not old fields.
       def mutate(g, n, p, v) #:nodoc:
         line = Field.encode0(g, n, p, v)
-        @group, @name, @params, @value = Field.decode0(line)
+        @valid, @group, @name, @params, @value = Field.decode0(line)
         @line = line
         self
       rescue ::Vcard::InvalidEncodingError => e

data/lib/vcard/vcard.rb

@@ -659,7 +659,8 @@ module Vcard
       vcards = []
 
       for e in entities
-        vcards.push(new(e.flatten, "VCARD"))
+        vcard  = new(e.flatten, "VCARD")
+        vcards.push(vcard) if vcard.valid? || !::Vcard.configuration.ignore_invalid_vcards?
       end
 
       vcards

data/lib/vcard/version.rb

@@ -1,3 +1,3 @@
 module Vcard
-  VERSION = "0.2.13"
+  VERSION = "0.2.14"
 end

data/test/field_test.rb

@@ -21,31 +21,31 @@ class FieldTest < Test::Unit::TestCase
   def test_field4
     line = "t;e=a,b: 4 "
     part = Field.decode0(line)
-    assert_equal("4", part[ 3 ])
+    assert_equal("4", part[ 4 ])
   end
 
   def test_field3
     line = "t;e=a,b:4"
     part = Field.decode0(line)
-    assert_equal("4", part[ 3 ])
-    assert_equal( {"E" => [ "a","b" ] }, part[ 2 ])
+    assert_equal("4", part[ 4 ])
+    assert_equal( {"E" => [ "a","b" ] }, part[ 3 ])
   end
 
   def test_field2
     line = "tel;type=work,voice,msg:+1 313 747-4454"
     part = Field.decode0(line)
-    assert_equal("+1 313 747-4454", part[ 3 ])
-    assert_equal( {"TYPE" => [ "work","voice","msg" ] }, part[ 2 ])
+    assert_equal("+1 313 747-4454", part[ 4 ])
+    assert_equal( {"TYPE" => [ "work","voice","msg" ] }, part[ 3 ])
   end
 
   def test_field1
     line = 'ORGANIZER;CN="xxxx, xxxx [SC100:370:EXCH]":MAILTO:xxxx@americasm01.nt.com'
     parts = Field.decode0(line)
 
-    assert_equal(nil, parts[0])
-    assert_equal("ORGANIZER", parts[1])
-    assert_equal({ "CN" => [ "xxxx, xxxx [SC100:370:EXCH]" ] }, parts[2])
-    assert_equal("MAILTO:xxxx@americasm01.nt.com", parts[3])
+    assert_equal(nil, parts[1])
+    assert_equal("ORGANIZER", parts[2])
+    assert_equal({ "CN" => [ "xxxx, xxxx [SC100:370:EXCH]" ] }, parts[3])
+    assert_equal("MAILTO:xxxx@americasm01.nt.com", parts[4])
   end
 
 =begin this can not be done :-(
@@ -67,20 +67,21 @@ class FieldTest < Test::Unit::TestCase
 
   def test_field0
     assert_equal("name:", line = Field.encode0(nil, "name"))
-    assert_equal([ nil, "NAME", {}, ""], Field.decode0(line))
+    assert_equal([ true, nil, "NAME", {}, ""], Field.decode0(line))
 
     assert_equal("name:value", line = Field.encode0(nil, "name", {}, "value"))
-    assert_equal([ nil, "NAME", {}, "value"], Field.decode0(line))
+    assert_equal([ true, nil, "NAME", {}, "value"], Field.decode0(line))
 
     assert_equal("name;encoding=B:dmFsdWU=", line = Field.encode0(nil, "name", { "encoding"=>:b64 }, "value"))
-    assert_equal([ nil, "NAME", { "ENCODING"=>["B"]}, ["value"].pack("m").chomp ], Field.decode0(line))
+    assert_equal([ true, nil, "NAME", { "ENCODING"=>["B"]}, ["value"].pack("m").chomp ], Field.decode0(line))
 
     line = Field.encode0("group", "name", {}, "value")
     assert_equal "group.name:value", line
-    assert_equal [ "GROUP", "NAME", {}, "value"], Field.decode0(line)
+    assert_equal [ true, "GROUP", "NAME", {}, "value"], Field.decode0(line)
   end
 
-  def test_invalid_fields
+  def test_invalid_fields_wih_raise_error
+    Vcard::configuration.raise_on_invalid_line = true
     [
       "g.:",
       ":v",
@@ -89,6 +90,16 @@ class FieldTest < Test::Unit::TestCase
     end
   end
 
+  def test_invalid_fields_wihout_raise_error
+    Vcard.configuration.raise_on_invalid_line = false
+    [
+        "g.:",
+        ":v",
+    ].each do |line|
+      assert_nothing_raised { Field.decode0(line) }
+    end
+  end
+
   def test_date_encode
     assert_equal("DTSTART:20040101\n", Field.create("DTSTART",  Date.new(2004, 1, 1) ).to_s)
     assert_equal("DTSTART:20040101\n", Field.create("DTSTART", [Date.new(2004, 1, 1)]).to_s)
@@ -97,6 +108,7 @@ class FieldTest < Test::Unit::TestCase
   def test_field_modify
     f = Field.create("name")
 
+
     assert_equal("", f.value)
     f.value = ""
     assert_equal("", f.value)

data/test/vcard_test.rb

@@ -169,12 +169,50 @@ EOF
   end
 
   def test_bad
-    # FIXME: this should THROW, it's badly encoded!
+    Vcard::configuration.raise_on_invalid_line = true
     assert_raises(::Vcard::InvalidEncodingError) do
       Vcard::Vcard.decode("BEGIN:VCARD\nVERSION:3.0\nKEYencoding=b:this could be \nmy certificate\n\nEND:VCARD\n")
     end
   end
 
+  def test_not_raise_error_if_configured_to_ignore
+    Vcard::configuration.raise_on_invalid_line = false
+    Vcard::configuration.ignore_invalid_vcards = false
+    assert_nothing_raised do
+      Vcard::Vcard.decode("BEGIN:VCARD\nVERSION:3.0\nKEYencoding=b:this could be \nmy certificate\n\nEND:VCARD\n")
+    end
+  end
+
+  def test_ignore_vcards_with_invalid_fields
+    Vcard::configuration.raise_on_invalid_line = false
+    Vcard::configuration.ignore_invalid_vcards = true
+    src = <<'EOF'
+BEGIN:VCARD
+VERSION:3.0
+KEYencoding=b:this could be
+my certificate
+EMAIL:valid@field.value
+END:VCARD
+BEGIN:VCARD
+VERSION:3.0
+EMAIL:valid@field.value
+END:VCARD
+EOF
+
+    cards = Vcard::Vcard.decode(src)
+    assert_equal 1, cards.size
+  end
+
+  def test_ignore_only_invalid_fields
+    Vcard::configuration.raise_on_invalid_line = false
+    Vcard::configuration.ignore_invalid_vcards = false
+    email = 'test@example.com'
+    cards = Vcard::Vcard.decode("BEGIN:VCARD\nVERSION:3.0\nKEYencoding=b:this could be \nmy certificate\nEMAIL:#{email}\n\nEND:VCARD\n")
+    assert_equal email, cards.first.email
+    # [BEGIN, VERSION, EMAIL, END].size == 4
+    assert_equal 4, cards.first.fields.size
+  end
+
   def test_create
     card = Vcard::Vcard.create
     key = Vcard::DirectoryInfo.decode("key;type=x509;encoding=B:dGhpcyBjb3VsZCBiZSAKbXkgY2VydGlmaWNhdGUK\n")['key']

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vcard
 version: !ruby/object:Gem::Version
-  version: 0.2.13
+  version: 0.2.14
 platform: ruby
 authors:
 - Kuba Kuźma
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-23 00:00:00.000000000 Z
+date: 2016-11-22 00:00:00.000000000 Z
 dependencies: []
 description: Vcard extracted from Vpim
 email:
@@ -28,6 +28,7 @@ files:
 - lib/vcard.rb
 - lib/vcard/attachment.rb
 - lib/vcard/bnf.rb
+- lib/vcard/configuration.rb
 - lib/vcard/dirinfo.rb
 - lib/vcard/enumerator.rb
 - lib/vcard/errors.rb