fog 1.10.1 → 1.11.0

data/lib/fog/aws/requests/storage/get_object_torrent.rb

@@ -33,10 +33,10 @@ module Fog
           request({
             :expects    => 200,
             :headers    => {},
-            :host       => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :idempotent => true,
             :method     => 'GET',
-            :path       => CGI.escape(object_name),
             :query      => {'torrent' => nil}
           })
         end

data/lib/fog/aws/requests/storage/get_object_url.rb

@@ -11,20 +11,11 @@ module Fog
           unless object_name
             raise ArgumentError.new('object_name is required')
           end
-          host, path = if bucket_name =~ Fog::AWS::COMPLIANT_BUCKET_NAMES
-            ["#{bucket_name}.#{@host}", object_name]
-          else
-            [@host, "#{bucket_name}/#{object_name}"]
-          end
-          scheme_host_path_query({
-            :scheme   => options[:scheme],
-            :headers  => {},
-            :host     => host,
-            :port     => @port,
-            :method   => 'GET',
-            :path     => path,
-            :query    => options[:query]
-          }, expires)
+          signed_url(options.merge({
+            :bucket_name => bucket_name,
+            :object_name => object_name,
+            :method => 'GET'
+          }), expires)
         end
       end
 

data/lib/fog/aws/requests/storage/get_request_payment.rb

@@ -19,7 +19,7 @@ module Fog
           request({
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :idempotent => true,
             :method   => 'GET',
             :parser   => Fog::Parsers::Storage::AWS::GetRequestPayment.new,

data/lib/fog/aws/requests/storage/head_object.rb

@@ -42,10 +42,10 @@ module Fog
           request({
             :expects    => 200,
             :headers    => headers,
-            :host       => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :idempotent => true,
             :method     => 'HEAD',
-            :path       => CGI.escape(object_name),
             :query      => query
           })
         end

data/lib/fog/aws/requests/storage/initiate_multipart_upload.rb

@@ -30,10 +30,10 @@ module Fog
           request({
             :expects    => 200,
             :headers    => options,
-            :host       => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :method     => 'POST',
             :parser     => Fog::Parsers::Storage::AWS::InitiateMultipartUpload.new,
-            :path       => CGI.escape(object_name),
             :query      => {'uploads' => nil}
           })
         end

data/lib/fog/aws/requests/storage/list_multipart_uploads.rb

@@ -7,16 +7,15 @@ module Fog
 
         # List multipart uploads for a bucket
         #
-        # @param bucket_name [String] Name of bucket to list multipart uploads for
-        # @param upload_id [String] upload id to list objects for
-        # @param options [Hash] config arguments for list.  Defaults to {}.
-        # @option options key-marker[String] limits parts to only those that appear lexicographically after this key.
-        # @option options max-uploads [Integer] limits number of uploads returned
-        # @option options upload-id-marker [String] limits uploads to only those that appear lexicographically after this upload id.
+        # @param [String] bucket_name Name of bucket to list multipart uploads for
+        # @param [Hash] options config arguments for list.  Defaults to {}.
+        # @option options [String] key-marker limits parts to only those that appear lexicographically after this key.
+        # @option options [Integer] max-uploads limits number of uploads returned
+        # @option options [String] upload-id-marker limits uploads to only those that appear lexicographically after this upload id.
         #
         # @return [Excon::Response] response:
         #   * body [Hash]:
-        #     * Bucket [string] Bucket where the multipart upload was initiated
+        #     * Bucket [String] Bucket where the multipart upload was initiated
         #     * IsTruncated [Boolean] Whether or not the listing is truncated
         #     * KeyMarker [String] first key in list, only upload ids after this lexographically will appear
         #     * MaxUploads [Integer] Maximum results to return
@@ -41,7 +40,7 @@ module Fog
           request({
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :idempotent => true,
             :method   => 'GET',
             :parser   => Fog::Parsers::Storage::AWS::ListMultipartUploads.new,

data/lib/fog/aws/requests/storage/list_parts.rb

@@ -40,11 +40,11 @@ module Fog
           request({
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :idempotent => true,
             :method   => 'GET',
             :parser   => Fog::Parsers::Storage::AWS::ListParts.new,
-            :path     => CGI.escape(object_name),
             :query    => options.merge!({'uploadId' => upload_id})
           })
         end

data/lib/fog/aws/requests/storage/put_bucket.rb

@@ -31,7 +31,7 @@ DATA
             :body       => data,
             :headers    => options,
             :idempotent => true,
-            :host       => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method     => 'PUT'
           })
         end

data/lib/fog/aws/requests/storage/put_bucket_acl.rb

@@ -46,7 +46,7 @@ module Fog
             :body     => data,
             :expects  => 200,
             :headers  => headers,
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'acl' => nil}
           })

data/lib/fog/aws/requests/storage/put_bucket_cors.rb

@@ -31,7 +31,7 @@ module Fog
             :body     => data,
             :expects  => 200,
             :headers  => headers,
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'cors' => nil}
           })

data/lib/fog/aws/requests/storage/put_bucket_lifecycle.rb

@@ -66,7 +66,7 @@ module Fog
                     :expects  => 200,
                     :headers  => {'Content-MD5' => Base64.encode64(Digest::MD5.digest(body)).chomp!,
                       'Content-Type' => 'application/xml'},
-                    :host     => "#{bucket_name}.#{@host}",
+                    :bucket_name => bucket_name,
                     :method   => 'PUT',
                     :query    => {'lifecycle' => nil}
                   })

data/lib/fog/aws/requests/storage/put_bucket_logging.rb

@@ -69,7 +69,7 @@ DATA
             :body     => data,
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'logging' => nil}
           })

data/lib/fog/aws/requests/storage/put_bucket_policy.rb

@@ -15,7 +15,7 @@ module Fog
             :body     => Fog::JSON.encode(policy),
             :expects  => 204,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'policy' => nil}
           })

data/lib/fog/aws/requests/storage/put_bucket_versioning.rb

@@ -22,7 +22,7 @@ DATA
             :body     => data,
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'versioning' => nil}
           })

data/lib/fog/aws/requests/storage/put_bucket_website.rb

@@ -35,7 +35,7 @@ DATA
             :body     => data,
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'website' => nil}
           })

data/lib/fog/aws/requests/storage/put_object.rb

@@ -33,10 +33,10 @@ module Fog
             :body       => data[:body],
             :expects    => 200,
             :headers    => headers,
-            :host       => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :idempotent => true,
             :method     => 'PUT',
-            :path       => CGI.escape(object_name)
           })
         end
 

data/lib/fog/aws/requests/storage/put_object_acl.rb

@@ -7,9 +7,9 @@ module Fog
 
         # Change access control list for an S3 object
         #
-        # @param bucket_name [String] name of bucket to modify
-        # @param object_name [String] name of object to get access control list for
-        # @param acl [Hash]:
+        # @param [String] bucket_name name of bucket to modify
+        # @param [String] object_name name of object to get access control list for
+        # @param [Hash] acl
         #   * Owner [Hash]
         #     * ID [String] id of owner
         #     * DisplayName [String] display name of owner
@@ -22,9 +22,9 @@ module Fog
         #       or
         #       * URI [String] URI of group to grant access for
         #     * Permission [String] Permission, in [FULL_CONTROL, WRITE, WRITE_ACP, READ, READ_ACP]
-        # @param acl [String] Permissions, must be in ['private', 'public-read', 'public-read-write', 'authenticated-read']
-        # @param options [Hash]
-        # @option options versionId [String] specify a particular version to retrieve
+        # @param [String] acl Permissions, must be in ['private', 'public-read', 'public-read-write', 'authenticated-read']
+        # @param [Hash] options
+        # @option options [String] versionId specify a particular version to retrieve
         #
         # @see http://docs.amazonwebservices.com/AmazonS3/latest/API/RESTObjectPUTacl.html
 
@@ -54,9 +54,9 @@ module Fog
             :body     => data,
             :expects  => 200,
             :headers  => headers,
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :method   => 'PUT',
-            :path       => CGI.escape(object_name),
             :query    => query
           })
         end

data/lib/fog/aws/requests/storage/put_object_url.rb

@@ -10,14 +10,12 @@ module Fog
           unless object_name
             raise ArgumentError.new('object_name is required')
           end
-          scheme_host_path_query({
-            :scheme   => options[:scheme],
-            :headers  => headers,
-            :host     => @host,
-            :port     => @port,
+          signed_url(options.merge({
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :method   => 'PUT',
-            :path     => "#{bucket_name}/#{object_name}"
-          }, expires)
+            :headers  => headers,
+          }), expires)
         end
       end
 

data/lib/fog/aws/requests/storage/put_request_payment.rb

@@ -21,7 +21,7 @@ DATA
             :body     => data,
             :expects  => 200,
             :headers  => {},
-            :host     => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
             :method   => 'PUT',
             :query    => {'requestPayment' => nil}
           })

data/lib/fog/aws/requests/storage/upload_part.rb

@@ -28,9 +28,9 @@ module Fog
             :expects    => 200,
             :idempotent => true,
             :headers    => headers,
-            :host       => "#{bucket_name}.#{@host}",
+            :bucket_name => bucket_name,
+            :object_name => object_name,
             :method     => 'PUT',
-            :path       => CGI.escape(object_name),
             :query      => {'uploadId' => upload_id, 'partNumber' => part_number}
           })
         end

data/lib/fog/aws/storage.rb

@@ -6,6 +6,42 @@ module Fog
     class AWS < Fog::Service
       extend Fog::AWS::CredentialFetcher::ServiceMethods
 
+      COMPLIANT_BUCKET_NAMES = /^(?:[a-z]|\d(?!\d{0,2}(?:\.\d{1,3}){3}$))(?:[a-z0-9]|\.(?![\.\-])|\-(?![\.])){1,61}[a-z0-9]$/
+
+      DEFAULT_REGION = 'us-east-1'
+
+      DEFAULT_SCHEME = 'https'
+      DEFAULT_SCHEME_PORT = {
+        'http' => 80,
+        'https' => 443
+      }
+
+      VALID_QUERY_KEYS = %w[
+        acl
+        cors
+        delete
+        lifecycle
+        location
+        logging
+        notification
+        partNumber
+        policy
+        requestPayment
+        response-cache-control
+        response-content-disposition
+        response-content-encoding
+        response-content-language
+        response-content-type
+        response-expires
+        torrent
+        uploadId
+        uploads
+        versionId
+        versioning
+        versions
+        website
+      ]
+
       requires :aws_access_key_id, :aws_secret_access_key
       recognizes :endpoint, :region, :host, :path, :port, :scheme, :persistent, :use_iam_profile, :aws_session_token, :aws_credentials_expire_at, :path_style
 
@@ -79,11 +115,11 @@ module Fog
         end
 
         def http_url(params, expires)
-          scheme_host_path_query(params.merge(:scheme => 'http', :port => 80), expires)
+          signed_url(params.merge(:scheme => 'http'), expires)
         end
 
         def https_url(params, expires)
-          scheme_host_path_query(params.merge(:scheme => 'https', :port => 443), expires)
+          signed_url(params.merge(:scheme => 'https'), expires)
         end
 
         def url(params, expires)
@@ -91,32 +127,124 @@ module Fog
           https_url(params, expires)
         end
 
+        def request_url(params)
+          params = request_params(params)
+          params_to_url(params)
+        end
+
+        def signed_url(params, expires)
+          expires = expires.to_i
+          signature = signature(params, expires)
+          params = request_params(params)
+
+          params[:query] = (params[:query] || {}).merge({
+            'AWSAccessKeyId' => @aws_access_key_id,
+            'Signature' => signature,
+            'Expires' => expires,
+          })
+          params[:query]['x-amz-security-token'] = @aws_session_token if @aws_session_token
+
+          params_to_url(params)
+        end
+
         private
 
-        def scheme_host_path_query(params, expires)
-          params[:scheme] ||= @scheme
-          if params[:port] == 80 && params[:scheme] == 'http'
-            params.delete(:port)
+        def region_to_host(region=nil)
+          case region.to_s
+          when DEFAULT_REGION, ''
+            's3.amazonaws.com'
+          else
+            "s3-#{region}.amazonaws.com"
+          end
+        end
+
+        def object_to_path(object_name=nil)
+          '/' + Fog::AWS.escape(object_name.to_s).gsub('%2F','/')
+        end
+
+        def bucket_to_path(bucket_name, path=nil)
+          "/#{Fog::AWS.escape(bucket_name.to_s)}#{path}"
+        end
+
+        # Transforms things like bucket_name, object_name, region
+        #
+        # Should yield the same result when called f*f
+        def request_params(params)
+          headers  = params[:headers] || {}
+
+          if params[:scheme]
+            scheme = params[:scheme]
+            port   = params[:port]
+          else
+            scheme = @scheme || DEFAULT_SCHEME
+            port   = @port
           end
-          if params[:port] == 443 && params[:scheme] == 'https'
-            params.delete(:port)
+          if DEFAULT_SCHEME_PORT[scheme] == port
+            port = nil
           end
-          params[:headers] ||= {}
-          params[:headers]['Date'] = expires.to_i
-          params[:path] = Fog::AWS.escape(params[:path]).gsub('%2F', '/')
-          query = []
-          params[:headers]['x-amz-security-token'] = @aws_session_token if @aws_session_token
-          if params[:query]
-            for key, value in params[:query]
-              query << "#{key}=#{Fog::AWS.escape(value)}"
+
+          if params[:region]
+            region = params[:region]
+            host   = params[:host] || region_to_host(region)
+          else
+            region = @region       || DEFAULT_REGION
+            host   = params[:host] || @host || region_to_host(region)
+          end
+
+          path     = params[:path] || object_to_path(params[:object_name])
+          path     = '/' + path if path[0..0] != '/'
+
+          if params[:bucket_name]
+            bucket_name = params[:bucket_name]
+
+            path_style = params.fetch(:path_style, @path_style)
+            if !path_style && COMPLIANT_BUCKET_NAMES !~ bucket_name
+              Fog::Logger.warning("fog: the specified s3 bucket name(#{bucket_name}) is not a valid dns name, which will negatively impact performance.  For details see: http://docs.amazonwebservices.com/AmazonS3/latest/dev/BucketRestrictions.html")
+              path_style = true
+            elsif bucket_name.include?('.')
+              Fog::Logger.warning("fog: the specified s3 bucket name(#{bucket_name}) might fail with https.")
+            end
+
+            if path_style
+              path = bucket_to_path bucket_name, path
+            else
+              host = [bucket_name, host].join('.')
             end
           end
-          query << "AWSAccessKeyId=#{@aws_access_key_id}"
-          query << "Signature=#{Fog::AWS.escape(signature(params))}"
-          query << "Expires=#{params[:headers]['Date']}"
-          query << "x-amz-security-token=#{Fog::AWS.escape(@aws_session_token)}" if @aws_session_token
-          port_part = params[:port] && ":#{params[:port]}"
-          "#{params[:scheme]}://#{params[:host]}#{port_part}/#{params[:path]}?#{query.join('&')}"
+
+          ret = params.merge({
+            :scheme       => scheme,
+            :host         => host,
+            :port         => port,
+            :path         => path,
+            :headers      => headers,
+          })
+          
+          #
+          ret.delete(:path_style)
+          ret.delete(:bucket_name)
+          ret.delete(:object_name)
+          ret.delete(:region)
+
+          ret
+        end
+
+        def params_to_url(params)
+          query = params[:query] && params[:query].map do |key, value|
+            if value
+              [key, Fog::AWS.escape(value.to_s)].join('=')
+            else
+              key
+            end
+          end.join('&')
+
+          URI::Generic.build({
+            :scheme => params[:scheme],
+            :host   => params[:host],
+            :port   => params[:port],
+            :path   => params[:path],
+            :query  => query,
+          }).to_s
         end
 
       end
@@ -210,15 +338,9 @@ module Fog
           require 'mime/types'
           @use_iam_profile = options[:use_iam_profile]
           setup_credentials(options)
-          options[:region] ||= 'us-east-1'
-          @host = options[:host] || case options[:region]
-          when 'us-east-1'
-            's3.amazonaws.com'
-          else
-            "s3-#{options[:region]}.amazonaws.com"
-          end
-          @scheme = options[:scheme] || 'https'
-          @region = options[:region]
+          @region = options[:region] || DEFAULT_REGION
+          @host   = options[:host]   || region_to_host(@region)
+          @scheme = options[:scheme] || DEFAULT_SCHEME
         end
 
         def data
@@ -229,7 +351,7 @@ module Fog
           self.class.data[@region].delete(@aws_access_key_id)
         end
 
-        def signature(params)
+        def signature(params, expires)
           "foo"
         end
 
@@ -270,6 +392,7 @@ module Fog
           @use_iam_profile = options[:use_iam_profile]
           setup_credentials(options)
           @connection_options     = options[:connection_options] || {}
+          @persistent = options.fetch(:persistent, false)
 
           if @endpoint = options[:endpoint]
             endpoint = URI.parse(@endpoint)
@@ -279,23 +402,17 @@ module Fog
             else
               endpoint.path
             end
-            @port = endpoint.port
             @scheme = endpoint.scheme
+            @port = endpoint.port
           else
-            options[:region] ||= 'us-east-1'
-            @region = options[:region]
-            @host = options[:host] || case options[:region]
-            when 'us-east-1'
-              's3.amazonaws.com'
-            else
-              "s3-#{options[:region]}.amazonaws.com"
-            end
+            @region     = options[:region]      || DEFAULT_REGION
+            @host       = options[:host]        || region_to_host(@region)
             @path       = options[:path]        || '/'
-            @persistent = options.fetch(:persistent, false)
-            @port       = options[:port]        || 443
-            @scheme     = options[:scheme]      || 'https'
+            @scheme     = options[:scheme]      || DEFAULT_SCHEME
+            @port       = options[:port]        || DEFAULT_SCHEME_PORT[@scheme]
             @path_style = options[:path_style]  || false
           end
+
           @connection = Fog::Connection.new("#{@scheme}://#{@host}:#{@port}#{@path}", @persistent, @connection_options)
         end
 
@@ -303,17 +420,19 @@ module Fog
           @connection.reset
         end
 
-        def signature(params)
+        def signature(params, expires)
+          headers = params[:headers] || {}
+
           string_to_sign =
 <<-DATA
 #{params[:method].to_s.upcase}
-#{params[:headers]['Content-MD5']}
-#{params[:headers]['Content-Type']}
-#{params[:headers]['Date']}
+#{headers['Content-MD5']}
+#{headers['Content-Type']}
+#{expires}
 DATA
 
           amz_headers, canonical_amz_headers = {}, ''
-          for key, value in params[:headers]
+          for key, value in headers
             if key[0..5] == 'x-amz-'
               amz_headers[key] = value
             end
@@ -324,55 +443,33 @@ DATA
           end
           string_to_sign << canonical_amz_headers
 
-          subdomain = params[:host].split(".#{@host}").first
-          valid_dns = !!(subdomain =~ /^(?:[a-z]|\d(?!\d{0,2}(?:\.\d{1,3}){3}$))(?:[a-z0-9]|\.(?![\.\-])|\-(?![\.])){1,61}[a-z0-9]$/)
-          if !valid_dns || @path_style
-            Fog::Logger.warning("fog: the specified s3 bucket name(#{subdomain}) is not a valid dns name, which will negatively impact performance.  For details see: http://docs.amazonwebservices.com/AmazonS3/latest/dev/BucketRestrictions.html") unless valid_dns
-            params[:host] = params[:host].split("#{subdomain}.")[-1]
-            if params[:path]
-              params[:path] = "#{subdomain}/#{params[:path]}" unless subdomain == @host
-            else
-              params[:path] = subdomain
+
+          query_string = ''
+          if params[:query]
+            query_args = []
+            for key in params[:query].keys.sort
+              if VALID_QUERY_KEYS.include?(key)
+                value = params[:query][key]
+                if value
+                  query_args << "#{key}=#{Fog::AWS.escape(value.to_s)}"
+                else
+                  query_args << key
+                end
+              end
+            end
+            if query_args.any?
+              query_string = '?' + query_args.join('&')
             end
-            subdomain = nil
           end
 
-          canonical_resource  = @path.dup
-          unless subdomain.nil? || subdomain == @host
-            canonical_resource << "#{Fog::AWS.escape(subdomain).downcase}/"
-          end
-          canonical_resource << params[:path].to_s
-          canonical_resource << '?'
-          for key in (params[:query] || {}).keys.sort
-            if %w{
-              acl
-              cors
-              delete
-              lifecycle
-              location
-              logging
-              notification
-              partNumber
-              policy
-              requestPayment
-              response-cache-control
-              response-content-disposition
-              response-content-encoding
-              response-content-language
-              response-content-type
-              response-expires
-              torrent
-              uploadId
-              uploads
-              versionId
-              versioning
-              versions
-              website
-            }.include?(key)
-              canonical_resource << "#{key}#{"=#{params[:query][key]}" unless params[:query][key].nil?}&"
-            end
+          canonical_path = (params[:path] || object_to_path(params[:object_name])).to_s
+          canonical_path = '/' + canonical_path if canonical_path[0..0] != '/'
+          if params[:bucket_name]
+            canonical_resource = "/#{params[:bucket_name]}#{canonical_path}"
+          else
+            canonical_resource = canonical_path
           end
-          canonical_resource.chop!
+          canonical_resource << query_string
           string_to_sign << canonical_resource
 
           signed_string = @hmac.sign(string_to_sign)
@@ -393,16 +490,23 @@ DATA
         def request(params, &block)
           refresh_credentials_if_expired
 
-          params[:headers]['Date'] = Fog::Time.now.to_date_header
+          expires = Fog::Time.now.to_date_header
+          signature = signature(params, expires)
+          
+          params = request_params(params)
+          params.delete(:port) unless params[:port]
+
+          params[:headers]['Date'] = expires
           params[:headers]['x-amz-security-token'] = @aws_session_token if @aws_session_token
-          params[:headers]['Authorization'] = "AWS #{@aws_access_key_id}:#{signature(params)}"
+          params[:headers]['Authorization'] = "AWS #{@aws_access_key_id}:#{signature}"
           # FIXME: ToHashParser should make this not needed
           original_params = params.dup
 
           begin
             response = @connection.request(params, &block)
           rescue Excon::Errors::TemporaryRedirect => error
-            uri = URI.parse(error.response.is_a?(Hash) ? error.response[:headers]['Location'] : error.response.headers['Location'])
+            headers = (error.response.is_a?(Hash) ? error.response[:headers] : error.response.headers)
+            uri = URI.parse(headers['Location'])
             Fog::Logger.warning("fog: followed redirect to #{uri.host}, connecting to the matching region will be more performant")
             response = Fog::Connection.new("#{@scheme}://#{uri.host}:#{@port}", false, @connection_options).request(original_params, &block)
           end