gmo 0.5.8 → 0.5.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 01b3f2fbd63de33e391333dd6e1d1a2b47ee785ea330aebda9aa461e2adcf6c8
-  data.tar.gz: 675aa139a33705769efa1764c8176778a492da09232e8ff915b92e376d46e252
+  metadata.gz: af1dd3f7ad4b9e6ac9d6ca3031f4667297e6f031aba145d50665d01fc32025e5
+  data.tar.gz: 616883c20b27bb5d94a0c4b11abfb3396247abe6da1e592d30282cba9dccb0ac
 SHA512:
-  metadata.gz: '0944fb5d4702aa5a1b2e3c035125063ba484f6420c037ad775895d6af5ea99fbc55be979750e37688b78a25eb4a2ffbb9abd4dc72e3e010811ac8a1ce5d4658c'
-  data.tar.gz: ca482bcc26e1eda87d96c77c24165c9dbf5f4cdc63f451ae3c6ce94d7e0760d41913c4372eafc20b7b7a5c91439255e816a3d17d5ee65ba7eb9231a68228c88b
+  metadata.gz: d570dc713d1281eebb404d022f2fdb07eca419f770bdd685d217b2ac7798474e5f03c292b606c59d3beb30f24996152e6b812e05849a636b694edf4658c1bfb4
+  data.tar.gz: 121ec172132e99240b194da859e997140758d0a822e49bd55ad36ea114efb38442b462ac8126cc99a99826627bc72d582819d35e4d202f7877031e75f1582522

data/CHANGELOG.md

@@ -1,4 +1,7 @@
 # CHANGELOG
+## 0.5.9
+- net-http 0.7.0+でcontent typeを明示的に設定する必要がある問題を修正 [#79](https://github.com/t-k/gmo-payment-ruby/pull/79) (Thanks [@johnnyshields](https://github.com/johnnyshields))
+
 ## 0.5.8
 - クレジットカード決済の3Dセキュア2.0に対応 [#76](https://github.com/t-k/gmo-payment-ruby/pull/76)
   - 3DS2.0認証実行 (`tds2_auth`)

data/lib/gmo/http_services.rb

@@ -44,7 +44,8 @@ module GMO
                 headers = { "Content-Type" => "application/json" }
                 h.post(path, args.to_json, headers)
               else
-                h.post(path, encode_params(args))
+                headers = { "Content-Type" => "application/x-www-form-urlencoded" }
+                h.post(path, encode_params(args), headers)
               end
             else
               h.get("#{path}?#{encode_params(args)}")

data/lib/gmo/version.rb

@@ -1,3 +1,3 @@
 module GMO
-  VERSION = "0.5.8"
+  VERSION = "0.5.9"
 end

data/spec/gmo/net_http_service_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+class FakeNetHTTPService
+  include GMO::NetHTTPService
+end
+
+describe "GMO::NetHTTPService" do
+
+  describe "#make_request" do
+    let(:mock_http) { double("Net::HTTP") }
+    let(:mock_http_session) { double("Net::HTTP session") }
+    let(:mock_response) { double("Net::HTTPResponse", code: "200", body: "AccessID=test123") }
+
+    before do
+      allow(Net::HTTP).to receive(:new).and_return(mock_http)
+      allow(mock_http).to receive(:use_ssl=)
+      allow(mock_http).to receive(:start).and_yield(mock_http_session)
+    end
+
+    describe "Content-Type header for form-encoded POST requests" do
+      it "sets Content-Type to application/x-www-form-urlencoded" do
+        expect(mock_http_session).to receive(:post) do |path, body, headers|
+          expect(path).to eq("/payment/EntryTran.idPass")
+          expect(headers["Content-Type"]).to eq("application/x-www-form-urlencoded")
+          mock_response
+        end
+
+        FakeNetHTTPService.make_request(
+          "/payment/EntryTran.idPass",
+          { "ShopID" => "shop123", "OrderID" => "order456" },
+          "post",
+          { :host => "example.com" }
+        )
+      end
+
+      it "properly encodes form parameters" do
+        expect(mock_http_session).to receive(:post) do |path, body, headers|
+          expect(path).to eq("/payment/EntryTran.idPass")
+          expect(body).to include("ShopID=")
+          expect(body).to include("OrderID=")
+          expect(headers["Content-Type"]).to eq("application/x-www-form-urlencoded")
+          mock_response
+        end
+
+        FakeNetHTTPService.make_request(
+          "/payment/EntryTran.idPass",
+          { "ShopID" => "shop123", "OrderID" => "order456" },
+          "post",
+          { :host => "example.com" }
+        )
+      end
+    end
+
+    describe "Content-Type header for JSON POST requests" do
+      it "sets Content-Type to application/json for .json paths" do
+        expect(mock_http_session).to receive(:post) do |path, body, headers|
+          expect(path).to eq("/payment/EntryTran.json")
+          expect(headers["Content-Type"]).to eq("application/json")
+          mock_response
+        end
+
+        FakeNetHTTPService.make_request(
+          "/payment/EntryTran.json",
+          { "ShopID" => "shop123" },
+          "post",
+          { :host => "example.com" }
+        )
+      end
+    end
+
+    describe "GET requests" do
+      it "appends parameters to query string without Content-Type header" do
+        expect(mock_http_session).to receive(:get) do |path_with_query|
+          expect(path_with_query).to include("/payment/SearchTrade.idPass?")
+          expect(path_with_query).to include("ShopID=")
+          mock_response
+        end
+
+        FakeNetHTTPService.make_request(
+          "/payment/SearchTrade.idPass",
+          { "ShopID" => "shop123" },
+          "get",
+          { :host => "example.com" }
+        )
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gmo
 version: !ruby/object:Gem::Version
-  version: 0.5.8
+  version: 0.5.9
 platform: ruby
 authors:
 - Tatsuo Kaniwa
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-07-29 00:00:00.000000000 Z
+date: 2026-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -236,6 +236,7 @@ files:
 - spec/gmo/errors_spec.rb
 - spec/gmo/http_service_spec.rb
 - spec/gmo/json_spec.rb
+- spec/gmo/net_http_service_spec.rb
 - spec/gmo/remittance_api_spec.rb
 - spec/gmo/shop_and_site_api_spec.rb
 - spec/gmo/shop_api_spec.rb