Skip to content

Commit 1343550

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent 1f5225c commit 1343550

File tree

49 files changed

+1160
-77
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

49 files changed

+1160
-77
lines changed

.rubocop.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -538,6 +538,7 @@ Gitlab/RSpec/AvoidSetup:
538538
- 'ee/spec/features/registrations/saas/**/*'
539539
- 'ee/spec/features/trials/saas/**/*'
540540
- 'ee/spec/features/gitlab_subscriptions/trials/duo_pro/**/*'
541+
- 'ee/spec/features/gitlab_subscriptions/trials/duo_enterprise/**/*'
541542

542543
RSpec/DuplicateSpecLocation:
543544
Enabled: true

.rubocop_todo/layout/line_end_string_concatenation_indentation.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,6 @@ Layout/LineEndStringConcatenationIndentation:
134134
- 'ee/app/components/namespaces/storage/user_pre_enforcement_alert_component.rb'
135135
- 'ee/app/controllers/concerns/insights_actions.rb'
136136
- 'ee/app/controllers/ee/ldap/omniauth_callbacks_controller.rb'
137-
- 'ee/app/controllers/gitlab_subscriptions/trials/duo_pro_controller.rb'
138137
- 'ee/app/finders/geo/framework_registry_finder.rb'
139138
- 'ee/app/graphql/ee/mutations/ci/project_ci_cd_settings_update.rb'
140139
- 'ee/app/graphql/ee/mutations/issues/create.rb'

.rubocop_todo/rails/strong_params.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,6 @@ Rails/StrongParams:
335335
- 'ee/app/controllers/smartcard_controller.rb'
336336
- 'ee/app/controllers/subscriptions/groups_controller.rb'
337337
- 'ee/app/controllers/subscriptions/hand_raise_leads_controller.rb'
338-
- 'ee/app/controllers/gitlab_subscriptions/trials/duo_pro_controller.rb'
339338
- 'ee/app/controllers/subscriptions/trials_controller.rb'
340339
- 'ee/app/controllers/subscriptions_controller.rb'
341340
- 'ee/app/controllers/users/base_identity_verification_controller.rb'

Gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -626,7 +626,7 @@ gem 'ssh_data', '~> 1.3' # rubocop:todo Gemfile/MissingFeatureCategory
626626
gem 'spamcheck', '~> 1.3.0' # rubocop:todo Gemfile/MissingFeatureCategory
627627

628628
# Gitaly GRPC protocol definitions
629-
gem 'gitaly', '~> 17.2.0', feature_category: :gitaly
629+
gem 'gitaly', '~> 17.4.0.pre.rc1', feature_category: :gitaly
630630

631631
# KAS GRPC protocol definitions
632632
gem 'gitlab-kas-grpc', '~> 17.4.0.pre.rc1', feature_category: :deployment_management

Gemfile.checksum

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@
207207
{"name":"gettext","version":"3.4.9","platform":"ruby","checksum":"292864fe6a15c224cee4125a4a72fab426fdbb280e4cff3cfe44935f549b009a"},
208208
{"name":"gettext_i18n_rails","version":"1.12.0","platform":"ruby","checksum":"6ac4817731a9e2ce47e1e83381ac34f9142263bc2911aaaafb2526d2f1afc1be"},
209209
{"name":"git","version":"1.18.0","platform":"ruby","checksum":"c9b80462e4565cd3d7a9ba8440c41d2c52244b17b0dad0bfddb46de70630c465"},
210-
{"name":"gitaly","version":"17.2.0","platform":"ruby","checksum":"48eee8883c43bb2f8fedbb43e4543439cfe37c33becebaec9ea1d425f9cce865"},
210+
{"name":"gitaly","version":"17.4.0.pre.rc1","platform":"ruby","checksum":"72c69dfa77871be78dd2e017be3131b9515b396d779df5785f77df291219f7b3"},
211211
{"name":"gitlab","version":"4.19.0","platform":"ruby","checksum":"3f645e3e195dbc24f0834fbf83e8ccfb2056d8e9712b01a640aad418a6949679"},
212212
{"name":"gitlab-chronic","version":"0.10.5","platform":"ruby","checksum":"f80f18dc699b708870a80685243331290bc10cfeedb6b99c92219722f729c875"},
213213
{"name":"gitlab-dangerfiles","version":"4.8.0","platform":"ruby","checksum":"b327d079552ec974a63bf34d749a0308425af6ebf51d01064f1a6ff216a523db"},

Gemfile.lock

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -701,7 +701,7 @@ GEM
701701
git (1.18.0)
702702
addressable (~> 2.8)
703703
rchardet (~> 1.8)
704-
gitaly (17.2.0)
704+
gitaly (17.4.0.pre.rc1)
705705
grpc (~> 1.0)
706706
gitlab (4.19.0)
707707
httparty (~> 0.20)
@@ -2053,7 +2053,7 @@ DEPENDENCIES
20532053
gdk-toogle (~> 0.9, >= 0.9.5)
20542054
gettext (~> 3.4, >= 3.4.9)
20552055
gettext_i18n_rails (~> 1.12.0)
2056-
gitaly (~> 17.2.0)
2056+
gitaly (~> 17.4.0.pre.rc1)
20572057
gitlab-backup-cli!
20582058
gitlab-chronic (~> 0.10.5)
20592059
gitlab-dangerfiles (~> 4.8.0)

app/assets/javascripts/editor/schema/ci.json

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,10 @@
259259
"markdownDescription": "Reports will be uploaded as artifacts, and often displayed in the Gitlab UI, such as in merge requests. [Learn More](https://docs.gitlab.com/ee/ci/yaml/#artifactsreports).",
260260
"additionalProperties": false,
261261
"properties": {
262+
"annotations": {
263+
"type": "string",
264+
"description": "Path to JSON file with annotations report."
265+
},
262266
"junit": {
263267
"description": "Path for file(s) that should be parsed as JUnit XML result",
264268
"oneOf": [

app/controllers/users_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ def calendar_activities
186186

187187
def exists
188188
if Gitlab::CurrentSettings.signup_enabled? || current_user
189-
render json: { exists: !!Namespace.without_project_namespaces.find_by_path_or_name(params[:username]) }
189+
render json: { exists: Namespace.username_reserved?(params[:username]) }
190190
else
191191
render json: { error: _('You must be authenticated to access this path.') }, status: :unauthorized
192192
end

app/finders/concerns/namespaces/groups_filter.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,10 @@ def by_min_access_level(groups)
3737

3838
groups.by_min_access_level(current_user, params[:min_access_level])
3939
end
40+
41+
def filter_groups(groups)
42+
by_search(groups)
43+
.then { |filtered_groups| by_min_access_level(filtered_groups) }
44+
end
4045
end
4146
end

app/finders/namespaces/groups/invited_groups_finder.rb

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
# group
88
# current_user
99
# params:
10-
# relations: string
10+
# relation: string - groups by relation (direct or inherited)
1111
# search: string
1212
# min_access_level: integer
1313
#
@@ -18,8 +18,6 @@ class InvitedGroupsFinder
1818
include Namespaces::GroupsFilter
1919
include Gitlab::Allowable
2020

21-
attr_reader :group, :current_user, :params
22-
2321
def initialize(group, current_user = nil, params = {})
2422
@group = group
2523
@current_user = current_user
@@ -31,19 +29,16 @@ def execute
3129

3230
group_links = group_group_links(group, include_relations)
3331
groups = Group.id_in(group_links.select(:shared_with_group_id)).public_or_visible_to_user(current_user)
34-
groups = filter_invited_groups(groups)
32+
groups = filter_groups(groups)
3533
sort(groups).with_route
3634
end
3735

3836
private
3937

40-
def filter_invited_groups(groups)
41-
by_search(groups)
42-
.then { |filtered_groups| by_min_access_level(filtered_groups) }
43-
end
38+
attr_reader :group, :current_user, :params
4439

4540
def include_relations
46-
[params[:relation].try(:to_sym)]
41+
Array(params[:relation]).map(&:to_sym)
4742
end
4843
end
4944
end

0 commit comments

Comments
 (0)