13996: Explicitly get string representation of URLs

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz@veritasgenetics.com>

diff --git a/services/api/app/controllers/arvados/v1/schema_controller.rb b/services/api/app/controllers/arvados/v1/schema_controller.rb
index 5f042877de2afa5e3a8fa62cd0004cb75e0aa22e..222b106a970feed7bbfcfffd801b61f9b9197d9a 100644 (file)
--- a/services/api/app/controllers/arvados/v1/schema_controller.rb
+++ b/services/api/app/controllers/arvados/v1/schema_controller.rb
@@ -63,10 +63,10 @@ class Arvados::V1::SchemaController < ApplicationController
         crunchLogUpdateSize: Rails.configuration.Containers["Logging"]["LogUpdateSize"],
         remoteHosts: Rails.configuration.RemoteClusters.map {|k,v| v["Host"]},
         remoteHostsViaDNS: Rails.configuration.RemoteClusters["*"]["Proxy"],
-        websocketUrl: Rails.configuration.Services["Websocket"]["ExternalURL"],
-        workbenchUrl: Rails.configuration.Services["Workbench1"]["ExternalURL"],
-        keepWebServiceUrl: Rails.configuration.Services["WebDAV"]["ExternalURL"],
-        gitUrl: Rails.configuration.Services["GitHTTP"]["ExternalURL"],
+        websocketUrl: Rails.configuration.Services["Websocket"]["ExternalURL"].to_s,
+        workbenchUrl: Rails.configuration.Services["Workbench1"]["ExternalURL"].to_s,
+        keepWebServiceUrl: Rails.configuration.Services["WebDAV"]["ExternalURL"].to_s,
+        gitUrl: Rails.configuration.Services["GitHTTP"]["ExternalURL"].to_s,
         parameters: {
           alt: {
             type: "string",

diff --git a/services/api/test/functional/arvados/v1/schema_controller_test.rb b/services/api/test/functional/arvados/v1/schema_controller_test.rb
index 5c7b54067b53aaca2c6282c20ab4b7c19045b300..80cd6de16ace407c55b01b945c9d42a3b091be62 100644 (file)
--- a/services/api/test/functional/arvados/v1/schema_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/schema_controller_test.rb
@@ -33,8 +33,8 @@ class Arvados::V1::SchemaControllerTest < ActionController::TestCase
     assert_match(/^[0-9a-f]+(-modified)?$/, discovery_doc['source_version'])
     assert_match(/^[0-9a-f]+(-modified)?$/, discovery_doc['sourceVersion'])
     assert_match(/^unknown$/, discovery_doc['packageVersion'])
-    assert_equal discovery_doc['websocketUrl'], Rails.configuration.Services["Websocket"]["ExternalURL"]
-    assert_equal discovery_doc['workbenchUrl'], Rails.configuration.Services["Workbench1"]["ExternalURL"]
+    assert_equal discovery_doc['websocketUrl'], Rails.configuration.Services["Websocket"]["ExternalURL"].to_s
+    assert_equal discovery_doc['workbenchUrl'], Rails.configuration.Services["Workbench1"]["ExternalURL"].to_s
     assert_equal('zzzzz', discovery_doc['uuidPrefix'])
   end