cae-multipart_parser 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b40160cadbf8cc8a3f697358faf9fbc86189f69a
-  data.tar.gz: 9c53814388b57e07c4fe9067f1bcac1c642d5498
+  metadata.gz: 58b85fbc0eb96b1b5d3d124d74b726df472fb1ed
+  data.tar.gz: 34cf8eabe80ace9e5df827b27b1355cd377d0e2d
 SHA512:
-  metadata.gz: 01ed9067615cf6c444f92558b6d6232ff3b431afbd09264fbc5348f1a83183cb7e8c20d3b095f0e825fad8c1c788c12b19719cf554fa57232227693e3181c47b
-  data.tar.gz: 9cd5bf6885f0a60098497089797c9dc0345279a2458d69b8f11e198662f70d084af9a8bd9ca522a73e35e63881db17f38ee462564710050fd535fac6e8905d88
+  metadata.gz: 5ec5356efc56b49734c768631fda770bf8e51dd3aa6edd60cc9fa4d24bd604ba077fdac53268e86e398e1215e84415603fc54f750932f0ecf722989bf06824bd
+  data.tar.gz: 39a4915b557153bd5f39630471d2a03b29b58e496cf7ca7fdbadbabe8f7d9b001baa8e48dd018b99389baaade121731e575fc715e709e29936bd60cbc8fbf2af

data/lib/cae/multipart_parser/parser.rb

@@ -12,6 +12,8 @@ module Cae
 
       ContentLengthUnsetError = Class.new(StandardError)
 
+      ParseError = Class.new(StandardError)
+
       attr_accessor :read_buffer_size
 
       def initialize(opts = {})
@@ -39,13 +41,9 @@ module Cae
           i = 0
           data_start = 0
 
-          #p "parsing #{length} chars: #{buffer}"
-
           while i < length
             c = buffer[i]
 
-            #p "state=#{@state}: i=#{i}. index=#{@index}, d_start=#{data_start} chars=#{buffer[i, 50]}"
-
             case @state
             when :start
               if @index == @boundary_length - 2
@@ -150,7 +148,7 @@ module Cae
           end # while
 
           if i != length
-            raise "unexpected char at #{i} (#{buffer[i].inspect})"
+            raise ParseError, "unexpected char at char #{parsed+i} (#{c.inspect})"
           end
 
           parsed += length

data/lib/cae/multipart_parser/part.rb

@@ -18,8 +18,9 @@ module Cae
         str.split(/\r\n/).each do |h|
           key, value = h.split ':'
 
-          # normalize content-length -> Content-Length
-          key = key.split('-').map(&:capitalize).join('-')
+          # normalize content-length -> CONTENT_LENGTH
+          key.upcase!
+          key.tr! '-', '_'
 
           @headers[key] = value.lstrip
         end
@@ -27,7 +28,7 @@ module Cae
       end
 
       def content_length
-        @headers['Content-Length'].to_i
+        @headers['CONTENT_LENGTH'].to_i
       end
 
       def on(event, &callback)

data/lib/cae/multipart_parser/version.rb

@@ -1,5 +1,5 @@
 module Cae
   module MultipartParser
-    VERSION = "1.0.0"
+    VERSION = "1.1.0"
   end
 end

data/spec/part_spec.rb

@@ -9,37 +9,37 @@ describe Cae::MultipartParser::Part do
     it "parses a simple input" do
       input = "Content-Type: text/html"
       part.parse_header_str(input).must_equal({
-        'Content-Type' => 'text/html'
+        'CONTENT_TYPE' => 'text/html'
       })
     end
 
     it "ignores trailing CRLF" do
       input = "Content-Type: text/html\r\n"
       part.parse_header_str(input).must_equal({
-        'Content-Type' => 'text/html'
+        'CONTENT_TYPE' => 'text/html'
       })
     end
 
     it "parses multiple lines" do
       input = "Content-Type: text/html\r\nContent-Length: 100"
       part.parse_header_str(input).must_equal({
-        'Content-Type' => 'text/html',
-        'Content-Length' => '100'
+        'CONTENT_TYPE' => 'text/html',
+        'CONTENT_LENGTH' => '100'
       })
     end
 
     it "parses multiline headers" do
       input = "Content-Type: multipart/form-data;\r\n\tboundary=foo"
       part.parse_header_str(input).must_equal({
-        'Content-Type' => 'multipart/form-data; boundary=foo'
+        'CONTENT_TYPE' => 'multipart/form-data; boundary=foo'
       })
     end
 
     it "parses multiple multiline headers" do
       input = "Content-Type: multipart/form-data;\r\n\tboundary=foo\r\nContent-Type2: multipart/form-data;\r\n\tboundary=bar"
       part.parse_header_str(input).must_equal({
-        'Content-Type' => 'multipart/form-data; boundary=foo',
-        'Content-Type2' => 'multipart/form-data; boundary=bar'
+        'CONTENT_TYPE' => 'multipart/form-data; boundary=foo',
+        'CONTENT_TYPE2' => 'multipart/form-data; boundary=bar'
       })
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cae-multipart_parser
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Chris Elsworth
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-03 00:00:00.000000000 Z
+date: 2015-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multipart-post