modern 0.4.2 → 0.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2758b166b5113cad1819574884d04dafe166f70033e87cfcf54fc0615a33b7f6
-  data.tar.gz: df1d24226b92fbfb1ec37a06e802390857a5a86d0cfc2ab56ac20dc669d49dc6
+  metadata.gz: f483d74d2aa71c07344b8ab689d473b52255fe52e93c45116f8f898887989f70
+  data.tar.gz: 81a87980bb858c2839c779f98364f699c4126a908cd4a6b732812dd749956262
 SHA512:
-  metadata.gz: b3fdfd253b7f161a2f850a4a71dcd4effa39d0561b0ebbd1d2203a2fa7801c250d333fc4f9e69d7072f2dfa41d9937a112b97052f08bd030ec64a122dd4a0458
-  data.tar.gz: 23291ac933a23779d4d5b762020f1fc67a42e5585b05593801f760aee8b2ed6cd7f850b3a21bd6d26f8ed0080dfbdbf7db7fef0ea94ed197c169cf9497543fc9
+  metadata.gz: b168abd8e525ab5f8ab193debbc99b94b61b42eb7dab708825d89be5cb0a804c6b15697036db70619c360cf779c1a95847a956e843adad4ca67bf4c71507b958
+  data.tar.gz: 0a0407e5e6c50609c8ecf7ae2d791c33a76fe1a0eebeb28ad0515e196a32ab235db4cb70f701f845d9f1b22c478a7443b24f497e3eb2dbbeb446d295c6cf50e2

data/lib/modern/app/request_handling.rb

@@ -32,7 +32,7 @@ module Modern
 
         params = parse_parameters(request, route)
         body = parse_request_body(request, route) unless route.request_body.nil?
-        raise Errors::BadRequestError \
+        raise Errors::BadRequestError, "Request body is required." \
           if body.nil? && route.request_body&.required
 
         begin

data/lib/modern/app/request_handling/input_handling.rb

@@ -20,8 +20,13 @@ module Modern
         end
 
         def parse_request_body(request, route)
-          input_converter = determine_input_converter(request, route)
-          raise Modern::Errors::UnsupportedMediaTypeError if input_converter.nil?
+          # RFC 2616; you MAY sniff content but SHOULD return application/octet-stream
+          # if the input Content-Type remains unknown. However, it seems like Rack may
+          # default to application/x-www-form-urlencoded; Rack::Test definitely does.
+          content_type = (request.content_type || "application/octet-stream").downcase.strip
+
+          input_converter = determine_input_converter_for_type(content_type, route)
+          raise Modern::Errors::UnsupportedMediaTypeError, content_type if input_converter.nil?
 
           raw = input_converter.converter.call(request.body)
 
@@ -52,11 +57,7 @@ module Modern
           end
         end
 
-        def determine_input_converter(request, route)
-          # RFC 2616; you MAY sniff content but SHOULD return application/octet-stream
-          # if the input Content-Type remains unknown. However, it seems like Rack may
-          # default to application/x-www-form-urlencoded; Rack::Test definitely does.
-          content_type = (request.content_type || "application/octet-stream").downcase.strip
+        def determine_input_converter_for_type(content_type, route)
           route.input_converters_by_type[content_type]
         end
       end

data/lib/modern/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Modern
-  VERSION = "0.4.2"
+  VERSION = "0.4.3"
   OPENAPI_VERSION = "3.0.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: modern
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.4.3
 platform: ruby
 authors:
 - Ed Ropple
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-03-01 00:00:00.000000000 Z
+date: 2018-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler