wash_out 0.9.0.beta.1 → 0.9.0.beta.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4791d7a762e3d5a75e9f8d3233ddd7d0b10a240f
-  data.tar.gz: d0d43203c81135e8040258cf971bd5b849c95b2e
+  metadata.gz: b640e8dbbc4f8e47d5b4bf02e02a8eddb87f4022
+  data.tar.gz: d262ad3bd148080fc799e202ceacafde106413ec
 SHA512:
-  metadata.gz: 925b2b09044af493d24e94e63a089cd81e56797c04c52ed8555605e0c306f9e99acec82427d4c010092af825508addb61e7b11b7bc3d28100db0d5353075bcc9
-  data.tar.gz: 300d800215c4b1442e9eb85b1c07d67126b04b3d8cca50120072b5543e32d51f14db91d33fb7258aa8580c29942cc8518924dec2c63b5ebfd00e89479b509e09
+  metadata.gz: 6b50419b1b9249d150c5adc7a85ea40e0ca05cd88efd5596e3ec0a1f94606ade0cf88fa8997daca86c4bfb993e11db2052f0c6421047d19cdd4d1526e04cad4a
+  data.tar.gz: b8f74d77676cb1e6ae2084444dee8d8034bff01337c3a573727d9fbccd8675f677be2e19308d1faf5eeb80696344a9fea1da75e0ee573bee6e8d6749e2be16c2

data/README.md

@@ -22,7 +22,7 @@ In your Gemfile, add this line:
 
     gem 'wash_out'
 
-If you are upgrading from version < 0.8.5, replace `include WashOut::Soap` with `soap_service` at the controller level.
+Please read [release details](https://github.com/inossidabile/wash_out/releases) if you are upgrading. We break backward compatibility between large ticks but you can expect it to be specified at release notes.
 
 ## Usage
 
@@ -147,10 +147,9 @@ To override the values on a specific controller just add an override as part of
 Available properties are:
 
 * **parser**: XML parser to use – `:rexml` or `:nokogiri`. The first one is default but the latter is much faster. Be sure to add `gem nokogiri` if you want to use it.
-* **style**: sets WSDL style. Supported values are: 'document' and 'rpc'.
+* **wsdl_style**: sets WSDL style. Supported values are: 'document' and 'rpc'.
 * **catch_xml_errors**: intercept Rails parsing exceptions to return correct XML response for corrupt XML input. Default is `false`.
 * **namespace**: SOAP namespace to use. Default is `urn:WashOut`.
-* **snakecase**: *(DEPRECATED SINCE 0.4.0)* Determines if WashOut should modify parameters keys to snakecase. Default is `false`.
 * **snakecase_input**: Determines if WashOut should modify parameters keys to snakecase. Default is `false`.
 * **camelize_wsdl**: Determines if WashOut should camelize types within WSDL and responses. Supports `true` for CamelCase and `:lower` for camelCase. Default is `false`.
 

data/lib/wash_out/dispatcher.rb

@@ -1,5 +1,3 @@
-require 'nori'
-
 module WashOut
   # The WashOut::Dispatcher module should be included in a controller acting
   # as a SOAP endpoint. It includes actions for generating WSDL and handling
@@ -10,28 +8,10 @@ module WashOut
     class SOAPError < Exception; end
     class ProgrammerError < Exception; end
 
-    # This filter parses the SOAP request and puts it into +params+ array.
-    def _parse_soap_parameters
-
-      nori_parser = Nori.new(
-        :parser => soap_config.parser,
-        :strip_namespaces => true,
-        :advanced_typecasting => true,
-        :convert_tags_to => ( soap_config.snakecase_input ? lambda { |tag| tag.snakecase.to_sym } : lambda { |tag| tag.to_sym } ))
-
-      @_params = nori_parser.parse(request.body.read)
-      references = WashOut::Dispatcher.deep_select(@_params){|k,v| v.is_a?(Hash) && v.has_key?(:@id)}
-
-      unless references.blank?
-        replaces = {}; references.each{|r| replaces['#'+r[:@id]] = r}
-        @_params = WashOut::Dispatcher.deep_replace_href(@_params, replaces)
-      end
-    end
-
     def _authenticate_wsse
 
       begin
-        xml_security   = @_params.values_at(:envelope, :Envelope).compact.first
+        xml_security   = env['wash_out.soap_data'].values_at(:envelope, :Envelope).compact.first
         xml_security   = xml_security.values_at(:header, :Header).compact.first
         xml_security   = xml_security.values_at(:security, :Security).compact.first
         username_token = xml_security.values_at(:username_token, :UsernameToken).compact.first
@@ -49,7 +29,7 @@ module WashOut
       soap_action = request.env['wash_out.soap_action']
       action_spec = self.class.soap_actions[soap_action]
 
-      xml_data = @_params.values_at(:envelope, :Envelope).compact.first
+      xml_data = env['wash_out.soap_data'].values_at(:envelope, :Envelope).compact.first
       xml_data = xml_data.values_at(:body, :Body).compact.first
       xml_data = xml_data.values_at(soap_action.underscore.to_sym,
                                     soap_action.to_sym).compact.first || {}
@@ -176,8 +156,6 @@ module WashOut
     def self.included(controller)
       controller.send :rescue_from, SOAPError, :with => :_render_soap_exception
       controller.send :helper, :wash_out
-      controller.send :before_filter, :_parse_soap_parameters, :except => [
-        :_generate_wsdl, :_invalid_action ]
       controller.send :before_filter, :_authenticate_wsse,     :except => [
         :_generate_wsdl, :_invalid_action ]
       controller.send :before_filter, :_map_soap_parameters,   :except => [

data/lib/wash_out/router.rb

@@ -1,3 +1,5 @@
+require 'nori'
+
 module WashOut
   # This class is a Rack middleware used to route SOAP requests to a proper
   # action of a given SOAP controller.
@@ -6,24 +8,64 @@ module WashOut
       @controller_name = "#{controller_name.to_s}_controller".camelize
     end
 
-    def call(env)
-      controller = @controller_name.constantize
+    def controller
+      @controller
+    end
+
+    def parse_soap_action(env)
+      return env['wash_out.soap_action'] if env['wash_out.soap_action']
+
+      soap_action = env['HTTP_SOAPACTION']
+
+      if soap_action.blank?
+        soap_action = parse_soap_parameters(env)
+            .values_at(:envelope, :Envelope).compact.first
+            .values_at(:body, :Body).compact.first
+            .keys.first.to_s
+      end
+
+      # RUBY18 1.8 does not have force_encoding.
+      soap_action.force_encoding('UTF-8') if soap_action.respond_to? :force_encoding
+
+      if controller.soap_config.namespace
+        namespace = Regexp.escape controller.soap_config.namespace.to_s
+        soap_action.gsub!(/^"?(#{namespace}(\/|#)?)?([^"]*)"?$/, '\3')
+      else
+        soap_action = soap_action[1...-1] if soap_action.starts_with?('"')
+      end
+
+      env['wash_out.soap_action'] = soap_action
+    end
 
-      if soap_action = env['HTTP_SOAPACTION']
-        # RUBY18 1.8 does not have force_encoding.
-        soap_action.force_encoding('UTF-8') if soap_action.respond_to? :force_encoding
+    def parse_soap_parameters(env)
+      return env['wash_out.soap_data'] if env['wash_out.soap_data']
 
-        if controller.soap_config.namespace
-          namespace = Regexp.escape controller.soap_config.namespace.to_s
-          soap_action.gsub!(/^\"(#{namespace}(\/|#)?)?(.*)\"$/, '\3')
-        else
-          soap_action = soap_action[1...-1]
-        end
+      nori_parser = Nori.new(
+        :parser => controller.soap_config.parser,
+        :strip_namespaces => true,
+        :advanced_typecasting => true,
+        :convert_tags_to => ( controller.soap_config.snakecase_input ? lambda { |tag| tag.snakecase.to_sym } : lambda { |tag| tag.to_sym } ))
 
-        env['wash_out.soap_action'] = soap_action
+      env['wash_out.soap_data'] = if env['rack.input'].respond_to? :string then env['rack.input'].string else env['rack.input'].read end
+      env['wash_out.soap_data'] = nori_parser.parse(env['wash_out.soap_data'])
+      references = WashOut::Dispatcher.deep_select(env['wash_out.soap_data']){|k,v| v.is_a?(Hash) && v.has_key?(:@id)}
+
+      unless references.blank?
+        replaces = {}; references.each{|r| replaces['#'+r[:@id]] = r}
+        env['wash_out.soap_data'] = WashOut::Dispatcher.deep_replace_href(env['wash_out.soap_data'], replaces)
       end
 
+      env['wash_out.soap_data']
+    end
+
+    def call(env)
+      @controller = @controller_name.constantize
+
+      soap_action     = parse_soap_action(env)
+      soap_parameters = parse_soap_parameters(env)
+
       action_spec = controller.soap_actions[soap_action]
+
       if action_spec
         action = action_spec[:to]
       else

data/lib/wash_out/version.rb

@@ -1,3 +1,3 @@
 module WashOut
-  VERSION = "0.9.0.beta.1"
+  VERSION = "0.9.0.beta.2"
 end

data/spec/lib/wash_out/dispatcher_spec.rb

@@ -7,12 +7,6 @@ describe WashOut::Dispatcher do
   class Dispatcher < ApplicationController
     soap_service
 
-    def self.mock(text="")
-      dispatcher = self.new
-      dispatcher.request = OpenStruct.new(:body => OpenStruct.new(:read => text))
-      dispatcher
-    end
-
     def params
       @_params
     end
@@ -28,13 +22,13 @@ describe WashOut::Dispatcher do
     WashOut::Dispatcher.deep_replace_href({:bar => {:foo => {:@href => 1}}}, {1 => 2}).should == {:bar => {:foo => 2}}
   end
 
-  it "parses typical request" do
+  xit "parses typical request" do
     dispatcher = Dispatcher.mock("<foo>1</foo>")
     dispatcher._parse_soap_parameters
     dispatcher.params.should == {:foo => "1"}
   end
 
-  it "parses href request" do
+  xit "parses href request" do
     dispatcher = Dispatcher.mock <<-XML
       <root>
         <request>

data/spec/lib/wash_out_spec.rb

@@ -90,6 +90,37 @@ describe WashOut do
   describe "Dispatcher" do
 
     context "simple actions" do
+      it "accepts requests with no HTTP header" do
+        mock_controller do
+          soap_action "answer", :args => nil, :return => :int
+          def answer
+            render :soap => "42"
+          end
+        end
+
+        request = <<-XML
+          <?xml version="1.0" encoding="UTF-8"?>
+          <env:Envelope xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:tns="false" xmlns:env="http://schemas.xmlsoap.org/soap/envelope/">
+          <env:Body>
+            <tns:answer>
+              <value>42</value>
+            </tns:answer>
+          </env:Body>
+          </env:Envelope>
+        XML
+
+        HTTPI.post("http://app/api/action", request).body.should == <<-XML
+<?xml version="1.0" encoding="UTF-8"?>
+<soap:Envelope xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:tns="false">
+  <soap:Body>
+    <tns:answerResponse>
+      <Value xsi:type="xsd:int">42</Value>
+    </tns:answerResponse>
+  </soap:Body>
+</soap:Envelope>
+        XML
+      end
+
       it "accept no parameters" do
         mock_controller do
           soap_action "answer", :args => nil, :return => :int

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wash_out
 version: !ruby/object:Gem::Version
-  version: 0.9.0.beta.1
+  version: 0.9.0.beta.2
 platform: ruby
 authors:
 - Boris Staal
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-08 00:00:00.000000000 Z
+date: 2013-09-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nori
@@ -113,7 +113,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.6
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: Dead simple Rails 3 SOAP server library