fix(ruby): Detect content-type in form-data file upload using faraday (…

…#17851) (#17853)

* fix(ruby): Detect content-type in form-data file upload using faraday (#17851)

* Update samples

* Add needed dependency libmagic-dev into node 1 setup of Circle Parallel CI

* After merge/pull last master + regeneration of samples

CI/circle_parallel.sh

@@ -13,7 +13,7 @@ if [ "$NODE_INDEX" = "1" ]; then
   echo "Running node $NODE_INDEX ..."
   java -version
 
-  sudo apt-get -y install cpanminus
+  sudo apt-get -y install cpanminus libmagic-dev
 
   (cd samples/client/petstore/perl && /bin/bash ./test.bash)
   (cd samples/client/petstore/ruby && mvn integration-test)

modules/openapi-generator/src/main/resources/ruby-client/api_client.mustache

@@ -13,6 +13,7 @@ require 'typhoeus'
 {{#isFaraday}}
 require 'faraday'
 require 'faraday/multipart' if Gem::Version.new(Faraday::VERSION) >= Gem::Version.new('2.0')
+require 'filemagic'
 {{/isFaraday}}
 {{#isHttpx}}
 require 'httpx'

modules/openapi-generator/src/main/resources/ruby-client/api_client_faraday_partial.mustache

@@ -94,8 +94,8 @@
         form_params.each do |key, value|
           case value
           when ::File, ::Tempfile
-            # TODO hardcode to application/octet-stream, need better way to detect content type
-            data[key] = Faraday::FilePart.new(value.path, 'application/octet-stream', value.path)
+            filemagic = FileMagic.new(FileMagic::MAGIC_MIME)
+            data[key] = Faraday::FilePart.new(value.path, filemagic.file(value.path, true))
           when ::Array, nil
             # let Faraday handle Array and nil parameters
             data[key] = value

modules/openapi-generator/src/main/resources/ruby-client/gemspec.mustache

@@ -23,6 +23,7 @@ Gem::Specification.new do |s|
   {{#isFaraday}}
   s.add_runtime_dependency 'faraday', '>= 1.0.1', '< 3.0'
   s.add_runtime_dependency 'faraday-multipart'
+  s.add_runtime_dependency 'ruby-filemagic'
   {{/isFaraday}}
   {{#isTyphoeus}}
   s.add_runtime_dependency 'typhoeus', '~> 1.0', '>= 1.0.1'

samples/client/echo_api/ruby-faraday/lib/openapi_client/api_client.rb

@@ -17,6 +17,7 @@
 require 'time'
 require 'faraday'
 require 'faraday/multipart' if Gem::Version.new(Faraday::VERSION) >= Gem::Version.new('2.0')
+require 'filemagic'
 
 
 module OpenapiClient
@@ -140,8 +141,8 @@ def build_request_body(header_params, form_params, body)
         form_params.each do |key, value|
           case value
           when ::File, ::Tempfile
-            # TODO hardcode to application/octet-stream, need better way to detect content type
-            data[key] = Faraday::FilePart.new(value.path, 'application/octet-stream', value.path)
+            filemagic = FileMagic.new(FileMagic::MAGIC_MIME)
+            data[key] = Faraday::FilePart.new(value.path, filemagic.file(value.path, true))
           when ::Array, nil
             # let Faraday handle Array and nil parameters
             data[key] = value

samples/client/echo_api/ruby-faraday/openapi_client.gemspec

@@ -30,6 +30,7 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency 'faraday', '>= 1.0.1', '< 3.0'
   s.add_runtime_dependency 'faraday-multipart'
+  s.add_runtime_dependency 'ruby-filemagic'
 
   s.add_development_dependency 'rspec', '~> 3.6', '>= 3.6.0'
 

samples/client/petstore/ruby-faraday/lib/petstore/api_client.rb

@@ -17,6 +17,7 @@
 require 'time'
 require 'faraday'
 require 'faraday/multipart' if Gem::Version.new(Faraday::VERSION) >= Gem::Version.new('2.0')
+require 'filemagic'
 
 
 module Petstore
@@ -140,8 +141,8 @@ def build_request_body(header_params, form_params, body)
         form_params.each do |key, value|
           case value
           when ::File, ::Tempfile
-            # TODO hardcode to application/octet-stream, need better way to detect content type
-            data[key] = Faraday::FilePart.new(value.path, 'application/octet-stream', value.path)
+            filemagic = FileMagic.new(FileMagic::MAGIC_MIME)
+            data[key] = Faraday::FilePart.new(value.path, filemagic.file(value.path, true))
           when ::Array, nil
             # let Faraday handle Array and nil parameters
             data[key] = value

samples/client/petstore/ruby-faraday/petstore.gemspec

@@ -30,6 +30,7 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency 'faraday', '>= 1.0.1', '< 3.0'
   s.add_runtime_dependency 'faraday-multipart'
+  s.add_runtime_dependency 'ruby-filemagic'
 
   s.add_development_dependency 'rspec', '~> 3.6', '>= 3.6.0'