Skip to content
Snippets Groups Projects
Commit 74f4337d authored by Alex Hanselka's avatar Alex Hanselka
Browse files

Merge branch '11-6-stable-patch-3' into '11-6-stable'

Prepare 11.6.3 release

See merge request gitlab-org/gitlab-ce!24155
parents 4d0c547e d92d9308
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -20,7 +20,7 @@ module ApplicationSettingsHelper
def enabled_protocol
case Gitlab::CurrentSettings.enabled_git_access_protocol
when 'http'
gitlab_config.protocol
Gitlab.config.gitlab.protocol
when 'ssh'
'ssh'
end
Loading
Loading
@@ -35,7 +35,7 @@ module ApplicationSettingsHelper
end
 
def http_enabled?
all_protocols_enabled? || enabled_protocol == 'http'
all_protocols_enabled? || Gitlab::CurrentSettings.enabled_git_access_protocol == 'http'
end
 
def enabled_project_button(project, protocol)
Loading
Loading
---
title: Fix clone URL not showing if protocol is HTTPS
merge_request: 24131
author:
type: fixed
# frozen_string_literal: true
require 'spec_helper'
describe ApplicationSettingsHelper do
context 'when all protocols in use' do
before do
stub_application_setting(enabled_git_access_protocol: '')
end
it { expect(all_protocols_enabled?).to be_truthy }
it { expect(http_enabled?).to be_truthy }
it { expect(ssh_enabled?).to be_truthy }
end
context 'when SSH is only in use' do
before do
stub_application_setting(enabled_git_access_protocol: 'ssh')
end
it { expect(all_protocols_enabled?).to be_falsey }
it { expect(http_enabled?).to be_falsey }
it { expect(ssh_enabled?).to be_truthy }
end
shared_examples 'when HTTP protocol is in use' do |protocol|
before do
allow(Gitlab.config.gitlab).to receive(:protocol).and_return(protocol)
stub_application_setting(enabled_git_access_protocol: 'http')
end
it { expect(all_protocols_enabled?).to be_falsey }
it { expect(http_enabled?).to be_truthy }
it { expect(ssh_enabled?).to be_falsey }
end
it_behaves_like 'when HTTP protocol is in use', 'https'
it_behaves_like 'when HTTP protocol is in use', 'http'
end
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment