PC-124: Fix setting current URL for ActiveStorage

Author: vb-cmd

app/controllers/application_controller.rb

@@ -1,15 +1,16 @@
 class ApplicationController < ActionController::Base
   allow_browser versions: :modern
+
   before_action :authenticate_user!, unless: :active_admin_controller?
-  before_action :set_current_url
+  before_action :set_active_storage_url_options
 
   private
 
   def active_admin_controller?
     is_a?(ActiveAdmin::BaseController)
   end
 
-  def set_current_url
-    ActiveStorage::Current.url_options = { host: request.host_with_port }
+  def set_active_storage_url_options
+    ActiveStorage::Current.url_options = { protocol: request.protocol, host: request.host, port: request.port }
   end
 end

spec/helpers/api/v1/settings_helper_spec.rb

@@ -1,15 +1,5 @@
 require 'rails_helper'
 
-# Specs in this file have access to a helper object that includes
-# the Api::V1::SettingsHelper. For example:
-#
-# describe Api::V1::SettingsHelper do
-#   describe "string concat" do
-#     it "concats two strings with spaces" do
-#       expect(helper.concat_strings("this","that")).to eq("this that")
-#     end
-#   end
-# end
 RSpec.describe Api::V1::SettingsHelper, type: :helper do
   let(:setting) { Setting.current }