Commit 69eaf332 authored by Kamil Trzcinski's avatar Kamil Trzcinski

Fix store credentials

parent 6b8d88d2
......@@ -20,6 +20,14 @@ class ObjectStoreUploader < CarrierWave::Uploader::Base
def object_store_enabled?
object_store_options&.enabled
end
def object_store_credentials
@object_store_credentials ||= object_store_options&.connection&.to_hash&.deep_symbolize_keys
end
def object_store_directory
object_store_options&.remote_directory
end
end
attr_reader :subject, :field
......@@ -98,11 +106,11 @@ class ObjectStoreUploader < CarrierWave::Uploader::Base
end
def fog_directory
self.class.object_store_options.remote_directory
self.class.object_store_directory
end
def fog_credentials
self.class.object_store_options.connection
self.class.object_store_credentials
end
def fog_public
......
......@@ -7,7 +7,7 @@ module StubConfiguration
return unless enabled
::Fog::Storage.new(Gitlab.config.artifacts.object_store.connection).tap do |connection|
::Fog::Storage.new(ArtifactUploader.object_store_credentials).tap do |connection|
begin
connection.directories.create(key: 'artifacts')
rescue Excon::Error::Conflict
......
......@@ -239,7 +239,7 @@ describe ObjectStoreUploader do
end
describe '#fog_credentials' do
let(:connection) { 'connection' }
let(:connection) { Settingslogic.new("provider" => "AWS") }
before do
uploader_class.storage_options double(
......@@ -248,7 +248,7 @@ describe ObjectStoreUploader do
subject { uploader.fog_credentials }
it { is_expected.to eq(connection) }
it { is_expected.to eq(provider: 'AWS') }
end
describe '#fog_public' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment