Commit a47b32c7 authored by Ethan Urie's avatar Ethan Urie Committed by Matthias Käppler

Remove production env check

parent 9f96fd6a
...@@ -99,7 +99,7 @@ module Gitlab ...@@ -99,7 +99,7 @@ module Gitlab
end end
def client_creds(url) def client_creds(url)
if URI(url).scheme == 'tls' || Rails.env.production? if URI(url).scheme == 'tls'
GRPC::Core::ChannelCredentials.new(::Gitlab::X509::Certificate.ca_certs_bundle) GRPC::Core::ChannelCredentials.new(::Gitlab::X509::Certificate.ca_certs_bundle)
else else
:this_channel_is_insecure :this_channel_is_insecure
......
...@@ -56,36 +56,6 @@ RSpec.describe Gitlab::Spamcheck::Client do ...@@ -56,36 +56,6 @@ RSpec.describe Gitlab::Spamcheck::Client do
end end
end end
describe "Rails environment" do
let(:stub) { double(:spamcheck_stub, check_for_spam_issue: response) }
context "production" do
before do
allow(Rails.env).to receive(:production?).and_return(true)
end
it 'uses secure connection' do
expect(Spamcheck::SpamcheckService::Stub).to receive(:new).with(endpoint.sub(%r{^grpc://}, ''),
instance_of(GRPC::Core::ChannelCredentials),
anything).and_return(stub)
subject
end
end
context "not production" do
before do
allow(Rails.env).to receive(:production?).and_return(false)
end
it 'uses insecure connection' do
expect(Spamcheck::SpamcheckService::Stub).to receive(:new).with(endpoint.sub(%r{^grpc://}, ''),
:this_channel_is_insecure,
anything).and_return(stub)
subject
end
end
end
describe '#issue_spam?' do describe '#issue_spam?' do
before do before do
allow_next_instance_of(::Spamcheck::SpamcheckService::Stub) do |instance| allow_next_instance_of(::Spamcheck::SpamcheckService::Stub) do |instance|
......
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