diff --git a/app/views/shared/_import_form.html.haml b/app/views/shared/_import_form.html.haml index 75fec53b29c..d0f9374e832 100644 --- a/app/views/shared/_import_form.html.haml +++ b/app/views/shared/_import_form.html.haml @@ -1,7 +1,7 @@ - ci_cd_only = local_assigns.fetch(:ci_cd_only, false) - import_url = Gitlab::UrlSanitizer.new(f.object.import_url) -.form-group.import-url-data +.import-url-data .form-group = f.label :import_url, class: 'label-bold' do %span @@ -9,24 +9,18 @@ = f.text_field :import_url, value: import_url.sanitized_url, autocomplete: 'off', class: 'form-control', placeholder: 'https://gitlab.company.com/group/project.git', required: true - .form-group#import_url_auth_method - = label :import_url_auth_method, class: 'label-bold' do - %span - = _('Authentication method') - = select_tag :import_url_auth_method, options_for_select([[_('None'), 'none'], [_('Username and Password'), 'username-and-password']]), class: 'form-control' - - .div#import_url_auth_group{ style: 'display: none' } - .form-group + .row + .form-group.col-md-6 = f.label :import_url_user, class: 'label-bold' do %span = _('Username (optional)') = f.text_field :import_url_user, value: import_url.user, class: 'form-control', required: false, autocomplete: 'new-password' - .form-group + .form-group.col-md-6 = f.label :import_url_password, class: 'label-bold' do %span - = _('Git repository password') - = f.password_field :import_url_password, class: 'form-control', required: false, autocomplete: 'new-password', placeholder: 'Basic Auth Password' + = _('Password (optional)') + = f.password_field :import_url_password, class: 'form-control', required: false, autocomplete: 'new-password' .info-well.prepend-top-20 .well-segment @@ -34,7 +28,7 @@ %li = _('The repository must be accessible over http://, https:// or git://.').html_safe %li - = _('If your HTTP repository is not publicly accessible, add authentication information to the URL: https://username:password@gitlab.company.com/group/project.git.').html_safe + = _('If your HTTP repository is not publicly accessible, add your credentials.') %li = import_will_timeout_message(ci_cd_only) %li diff --git a/app/workers/concerns/project_import_options.rb b/app/workers/concerns/project_import_options.rb index 53b0459c48c..2baf768bfd1 100644 --- a/app/workers/concerns/project_import_options.rb +++ b/app/workers/concerns/project_import_options.rb @@ -3,7 +3,7 @@ module ProjectImportOptions extend ActiveSupport::Concern - IMPORT_RETRY_COUNT = 0 + IMPORT_RETRY_COUNT = 5 included do sidekiq_options retry: IMPORT_RETRY_COUNT, status_expiration: StuckImportJobsWorker::IMPORT_JOBS_EXPIRATION diff --git a/locale/gitlab.pot b/locale/gitlab.pot index 99e7bb5c9a0..6ba55560b4c 100644 --- a/locale/gitlab.pot +++ b/locale/gitlab.pot @@ -4584,9 +4584,6 @@ msgstr "" msgid "Git repository URL" msgstr "" -msgid "Git repository password" -msgstr "" - msgid "Git revision" msgstr "" @@ -5036,7 +5033,7 @@ msgstr "" msgid "If this was a mistake you can leave the %{source_type}." msgstr "" -msgid "If your HTTP repository is not publicly accessible, add authentication information to the URL: https://username:password@gitlab.company.com/group/project.git." +msgid "If your HTTP repository is not publicly accessible, add your credentials." msgstr "" msgid "ImageDiffViewer|2-up" @@ -6772,6 +6769,9 @@ msgstr "" msgid "Password" msgstr "" +msgid "Password (optional)" +msgstr "" + msgid "Password authentication is unavailable." msgstr "" @@ -10795,9 +10795,6 @@ msgstr "" msgid "Username (optional)" msgstr "" -msgid "Username and Password" -msgstr "" - msgid "Username is already taken." msgstr "" diff --git a/spec/javascripts/projects/project_new_spec.js b/spec/javascripts/projects/project_new_spec.js index b61e0ac872f..106a3ba94e4 100644 --- a/spec/javascripts/projects/project_new_spec.js +++ b/spec/javascripts/projects/project_new_spec.js @@ -10,7 +10,17 @@ describe('New Project', () => { setFixtures(`
- +
+ +
+
+
+ +
+
+ +
+
@@ -119,7 +129,7 @@ describe('New Project', () => { }); it('changes project path for HTTPS URL in $projectImportUrl', () => { - $projectImportUrl.val('https://username:password@gitlab.company.com/group/project.git'); + $projectImportUrl.val('https://gitlab.company.com/group/project.git'); projectNew.deriveProjectPathFromUrl($projectImportUrl);