From 09b8270653daed48c30dc33b65513cc2a4a7588f Mon Sep 17 00:00:00 2001
From: Martin Lovell <martin.lovell@yale.edu>
Date: Fri, 11 Sep 2020 16:17:19 -0400
Subject: [PATCH] update client to work with https endpoint and make https the
 default

---
 lib/summon/service.rb        | 2 +-
 lib/summon/transport/http.rb | 3 ++-
 spec/summon/service_spec.rb  | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/summon/service.rb b/lib/summon/service.rb
index 0a23531..ff5c444 100644
--- a/lib/summon/service.rb
+++ b/lib/summon/service.rb
@@ -5,7 +5,7 @@ class Service
     attr_reader :transport, :url, :access_id, :client_key
     
     def initialize(options = {})
-      @url        = options[:url] || "http://api.summon.serialssolutions.com/2.0.0"
+      @url        = options[:url] || "https://api.summon.serialssolutions.com/2.0.0"
       @access_id  = options[:access_id]
       @secret_key = options[:secret_key]
       @client_key = options[:client_key]
diff --git a/lib/summon/transport/http.rb b/lib/summon/transport/http.rb
index 9ff5f4d..052681a 100644
--- a/lib/summon/transport/http.rb
+++ b/lib/summon/transport/http.rb
@@ -5,7 +5,7 @@ module Summon::Transport
   class Http
     include Qstring
 
-    DEFAULTS = {:url => "http://api.summon.serialssolutions.com"}
+    DEFAULTS = {:url => "https://api.summon.serialssolutions.com"}
 
     def initialize(options = {:url => nil, :access_id => nil, :secret_key => nil, :client_key => nil, :session_id => nil, :log => nil})
       @options    = DEFAULTS.merge options
@@ -47,6 +47,7 @@ def urlget(url, params = nil, session_id = nil)
         }
         result = nil
           http = Net::HTTP.new(uri.host, uri.port)
+          http.use_ssl = uri.scheme == "https"
           http.start do
             get = Net::HTTP::Get.new("#{uri.path}#{'?' + uri.query if uri.query && uri.query != ''}")
             get.merge! headers
diff --git a/spec/summon/service_spec.rb b/spec/summon/service_spec.rb
index ddb29d7..f3411ad 100644
--- a/spec/summon/service_spec.rb
+++ b/spec/summon/service_spec.rb
@@ -12,7 +12,7 @@
   end
   
   it "has a default url which is the public production summon url" do
-    Summon::Service.new.url.should == "http://api.summon.serialssolutions.com/2.0.0"
+    Summon::Service.new.url.should == "https://api.summon.serialssolutions.com/2.0.0"
   end
 
   it "allows cloning a service with overridden settings" do