Add latest changes from gitlab-org/gitlab@master

app/graphql/types/error_tracking/sentry_detailed_error_type.rb

@@ -79,6 +79,9 @@ module Types
       field :gitlab_commit, GraphQL::STRING_TYPE,
             null: true,
             description: "GitLab commit SHA attributed to the Error based on the release version"
+      field :gitlab_commit_path, GraphQL::STRING_TYPE,
+            null: true,
+            description: "Path to the GitLab page for the GitLab commit attributed to the error"
 
       def first_seen
         DateTime.parse(object.first_seen)

app/helpers/application_settings_helper.rb

@@ -284,7 +284,6 @@ module ApplicationSettingsHelper
       :unique_ips_limit_enabled,
       :unique_ips_limit_per_user,
       :unique_ips_limit_time_window,
-      :updating_name_disabled_for_users,
       :usage_ping_enabled,
       :instance_statistics_visibility_private,
       :user_default_external,

app/models/error_tracking/project_error_tracking_setting.rb

@@ -4,6 +4,7 @@ module ErrorTracking
   class ProjectErrorTrackingSetting < ApplicationRecord
     include Gitlab::Utils::StrongMemoize
     include ReactiveCaching
+    include Gitlab::Routing
 
     SENTRY_API_ERROR_TYPE_BAD_REQUEST = 'bad_request_for_sentry_api'
     SENTRY_API_ERROR_TYPE_MISSING_KEYS = 'missing_keys_in_sentry_response'
@@ -141,6 +142,7 @@ module ErrorTracking
 
     def add_gitlab_issue_details(issue)
       issue.gitlab_commit = match_gitlab_commit(issue.first_release_version)
+      issue.gitlab_commit_path = project_commit_path(project, issue.gitlab_commit) if issue.gitlab_commit
 
       issue
     end

app/policies/user_policy.rb

@@ -13,11 +13,6 @@ class UserPolicy < BasePolicy
   desc "The user is blocked"
   condition(:blocked_user, scope: :subject, score: 0) { @subject.blocked? }
 
-  condition(:updating_name_disabled_for_users) do
-    ::Gitlab::CurrentSettings.current_application_settings
-      .updating_name_disabled_for_users
-  end
-
   rule { ~restricted_public_level }.enable :read_user
   rule { ~anonymous }.enable :read_user
 
@@ -27,8 +22,8 @@ class UserPolicy < BasePolicy
     enable :update_user_status
   end
 
-  rule { can?(:update_user) & ( admin | ~updating_name_disabled_for_users ) }.enable :update_name
-
   rule { default }.enable :read_user_profile
   rule { (private_profile | blocked_user) & ~(user_is_self | admin) }.prevent :read_user_profile
 end
+
+UserPolicy.prepend_if_ee('EE::UserPolicy')

app/serializers/error_tracking/detailed_error_entity.rb

@@ -9,6 +9,7 @@ module ErrorTracking
            :first_release_last_commit,
            :first_release_short_version,
            :gitlab_commit,
+           :gitlab_commit_path,
            :first_seen,
            :frequency,
            :gitlab_issue,

app/services/users/update_service.rb

@@ -54,7 +54,6 @@ module Users
 
     def discard_read_only_attributes
       discard_synced_attributes
-      discard_name unless name_updatable?
     end
 
     def discard_synced_attributes
@@ -65,14 +64,6 @@ module Users
       end
     end
 
-    def discard_name
-      params.delete(:name)
-    end
-
-    def name_updatable?
-      can?(current_user, :update_name, @user)
-    end
-
     def assign_attributes
       @user.assign_attributes(params.except(*identity_attributes)) unless params.empty?
     end

app/views/admin/application_settings/_account_and_limit.html.haml

@@ -51,13 +51,8 @@
         = f.check_box :user_show_add_ssh_key_message, class: 'form-check-input'
         = f.label :user_show_add_ssh_key_message, class: 'form-check-label' do
           = _("Inform users without uploaded SSH keys that they can't push over SSH until one is added")
-    .form-group
-      = f.label :updating_name_disabled_for_users, _('User restrictions'), class: 'label-bold'
-      .form-check
-        = f.check_box :updating_name_disabled_for_users, class: 'form-check-input'
-        = f.label :updating_name_disabled_for_users, class: 'form-check-label' do
-          = _("Prevent users from changing their profile name")
 
+    = render_if_exists 'admin/application_settings/updating_name_disabled_for_users', form: f
     = render_if_exists 'admin/application_settings/availability_on_namespace_setting', form: f
 
   = f.submit _('Save changes'), class: 'btn btn-success qa-save-changes-button'

app/views/profiles/_name.html.haml

 - if user.read_only_attribute?(:name)
   = form.text_field :name, required: true, readonly: true, wrapper: { class: 'col-md-9 qa-full-name rspec-full-name' },
   help: s_("Profiles|Your name was automatically set based on your %{provider_label} account, so people you know can recognize you") % { provider_label: attribute_provider_label(:name) }
-- elsif can?(current_user, :update_name, user)
-  = form.text_field :name, label: s_('Profiles|Full name'), required: true, title: s_("Profiles|Using emojis in names seems fun, but please try to set a status message instead"), wrapper: { class: 'col-md-9 qa-full-name rspec-full-name' }, help: s_("Profiles|Enter your name, so people you know can recognize you")
 - else
-  = form.text_field :name, required: true, readonly: true, wrapper: { class: 'col-md-9 qa-full-name rspec-full-name' },
-  help: s_("Profiles|The ability to update your name has been disabled by your administrator.")
+  = form.text_field :name, label: s_('Profiles|Full name'), required: true, title: s_("Profiles|Using emojis in names seems fun, but please try to set a status message instead"), wrapper: { class: 'col-md-9 qa-full-name rspec-full-name' }, help: s_("Profiles|Enter your name, so people you know can recognize you")

changelogs/unreleased/38223-add-gitlab-commit-path.yml

+---
+title: Add gitlab_commit_path to Sentry Error Details Response
+merge_request: 22803
+author:
+type: added

config/application.rb

@@ -229,13 +229,15 @@ module Gitlab
 
     # Allow access to GitLab API from other domains
     config.middleware.insert_before Warden::Manager, Rack::Cors do
+      headers_to_expose = %w[Link X-Total X-Total-Pages X-Per-Page X-Page X-Next-Page X-Prev-Page X-Gitlab-Blob-Id X-Gitlab-Commit-Id X-Gitlab-Content-Sha256 X-Gitlab-Encoding X-Gitlab-File-Name X-Gitlab-File-Path X-Gitlab-Last-Commit-Id X-Gitlab-Ref X-Gitlab-Size]
+
       allow do
         origins Gitlab.config.gitlab.url
         resource '/api/*',
           credentials: true,
           headers: :any,
           methods: :any,
-          expose: %w[Link X-Total X-Total-Pages X-Per-Page X-Page X-Next-Page X-Prev-Page]
+          expose: headers_to_expose
       end
 
       # Cross-origin requests must not have the session cookie available
@@ -245,7 +247,7 @@ module Gitlab
           credentials: false,
           headers: :any,
           methods: :any,
-          expose: %w[Link X-Total X-Total-Pages X-Per-Page X-Page X-Next-Page X-Prev-Page]
+          expose: headers_to_expose
       end
     end
 

doc/api/graphql/reference/gitlab_schema.graphql

@@ -5721,6 +5721,11 @@ type SentryDetailedError {
   """
   gitlabCommit: String
 
+  """
+  Path to the GitLab page for the GitLab commit attributed to the error
+  """
+  gitlabCommitPath: String
+
   """
   ID (global ID) of the error
   """

doc/api/graphql/reference/gitlab_schema.json

@@ -15576,6 +15576,20 @@
               "isDeprecated": false,
               "deprecationReason": null
             },
+            {
+              "name": "gitlabCommitPath",
+              "description": "Path to the GitLab page for the GitLab commit attributed to the error",
+              "args": [
+
+              ],
+              "type": {
+                "kind": "SCALAR",
+                "name": "String",
+                "ofType": null
+              },
+              "isDeprecated": false,
+              "deprecationReason": null
+            },
             {
               "name": "id",
               "description": "ID (global ID) of the error",

doc/api/graphql/reference/index.md

@@ -895,6 +895,7 @@ Autogenerated return type of RemoveAwardEmoji
 | `firstReleaseShortVersion` | String | Release version the error was first seen |
 | `lastReleaseShortVersion` | String | Release version the error was last seen |
 | `gitlabCommit` | String | GitLab commit SHA attributed to the Error based on the release version |
+| `gitlabCommitPath` | String | Path to the GitLab page for the GitLab commit attributed to the error |
 
 ## SentryErrorFrequency
 

doc/user/admin_area/settings/account_and_limit_settings.md

@@ -117,7 +117,7 @@ Once a lifetime for personal access tokens is set, GitLab will:
   allowed lifetime. Three hours is given to allow administrators to change the allowed lifetime,
   or remove it, before revocation takes place.
 
-## Disabling user profile name changes **(CORE ONLY)**
+## Disabling user profile name changes **(PREMIUM ONLY)**
 
 > [Introduced](https://gitlab.com/gitlab-org/gitlab/issues/24605) in GitLab 12.7.
 

lib/api/settings.rb

@@ -142,7 +142,6 @@ module API
         requires :sourcegraph_url, type: String, desc: 'The configured Sourcegraph instance URL'
       end
       optional :terminal_max_session_time, type: Integer, desc: 'Maximum time for web terminal websocket connection (in seconds). Set to 0 for unlimited time.'
-      optional :updating_name_disabled_for_users, type: Boolean, desc: 'Flag indicating if users are permitted to update their profile name'
       optional :usage_ping_enabled, type: Boolean, desc: 'Every week GitLab will report license usage back to GitLab, Inc.'
       optional :instance_statistics_visibility_private, type: Boolean, desc: 'When set to `true` Instance statistics will only be available to admins'
       optional :local_markdown_version, type: Integer, desc: 'Local markdown version, increase this value when any cached markdown should be invalidated'

lib/gitlab/error_tracking/detailed_error.rb

@@ -16,6 +16,7 @@ module Gitlab
                     :first_seen,
                     :frequency,
                     :gitlab_commit,
+                    :gitlab_commit_path,
                     :gitlab_issue,
                     :gitlab_project,
                     :id,

lib/gitlab/experimentation.rb

@@ -16,6 +16,12 @@ module Gitlab
         environment: ::Gitlab.dev_env_or_com?,
         enabled_ratio: 1,
         tracking_category: 'Growth::Acquisition::Experiment::SignUpFlow'
+      },
+      paid_signup_flow: {
+        feature_toggle: :paid_signup_flow,
+        environment: ::Gitlab.dev_env_or_com?,
+        enabled_ratio: 0.1,
+        tracking_category: 'Growth::Acquisition::Experiment::PaidSignUpFlow'
       }
     }.freeze
 

locale/gitlab.pot

@@ -6576,6 +6576,9 @@ msgstr ""
 msgid "Edit Deploy Key"
 msgstr ""
 
+msgid "Edit Geo Node"
+msgstr ""
+
 msgid "Edit Group Hook"
 msgstr ""
 
@@ -8303,6 +8306,9 @@ msgstr ""
 msgid "Geo Designs"
 msgstr ""
 
+msgid "Geo Node Form"
+msgstr ""
+
 msgid "Geo Nodes"
 msgstr ""
 
@@ -11969,6 +11975,9 @@ msgstr ""
 msgid "New Environment"
 msgstr ""
 
+msgid "New Geo Node"
+msgstr ""
+
 msgid "New Group"
 msgstr ""
 

spec/controllers/admin/application_settings_controller_spec.rb

@@ -102,13 +102,6 @@ describe Admin::ApplicationSettingsController do
       expect(ApplicationSetting.current.minimum_password_length).to eq(10)
     end
 
-    it 'updates updating_name_disabled_for_users setting' do
-      put :update, params: { application_setting: { updating_name_disabled_for_users: true } }
-
-      expect(response).to redirect_to(admin_application_settings_path)
-      expect(ApplicationSetting.current.updating_name_disabled_for_users).to eq(true)
-    end
-
     context 'external policy classification settings' do
       let(:settings) do
         {

spec/controllers/admin/users_controller_spec.rb

@@ -257,28 +257,6 @@ describe Admin::UsersController do
   end
 
   describe 'POST update' do
-    context 'updating name' do
-      context 'when the ability to update their name is disabled for users' do
-        before do
-          stub_application_setting(updating_name_disabled_for_users: true)
-        end
-
-        it 'updates the name' do
-          params = {
-            id: user.to_param,
-            user: {
-              name: 'New Name'
-            }
-          }
-
-          put :update, params: params
-
-          expect(response).to redirect_to(admin_user_path(user))
-          expect(user.reload.name).to eq('New Name')
-        end
-      end
-    end
-
     context 'when the password has changed' do
       def update_password(user, password, password_confirmation = nil)
         params = {

spec/controllers/profiles_controller_spec.rb

@@ -81,54 +81,6 @@ describe ProfilesController, :request_store do
       expect(ldap_user.location).to eq('City, Country')
     end
 
-    context 'updating name' do
-      subject { put :update, params: { user: { name: 'New Name' } } }
-
-      context 'when the ability to update thier name is not disabled for users' do
-        before do
-          stub_application_setting(updating_name_disabled_for_users: false)
-          sign_in(user)
-        end
-
-        it 'updates the name' do
-          subject
-
-          expect(response.status).to eq(302)
-          expect(user.reload.name).to eq('New Name')
-        end
-      end
-
-      context 'when the ability to update their name is disabled for users' do
-        before do
-          stub_application_setting(updating_name_disabled_for_users: true)
-        end
-
-        context 'as a regular user' do
-          it 'does not update the name' do
-            sign_in(user)
-
-            subject
-
-            expect(response.status).to eq(302)
-            expect(user.reload.name).not_to eq('New Name')
-          end
-        end
-
-        context 'as an admin user' do
-          it 'updates the name' do
-            admin = create(:admin)
-
-            sign_in(admin)
-
-            subject
-
-            expect(response.status).to eq(302)
-            expect(admin.reload.name).to eq('New Name')
-          end
-        end
-      end
-    end
-
     it 'allows setting a user status' do
       sign_in(user)
 

spec/controllers/projects/error_tracking_controller_spec.rb

@@ -224,7 +224,12 @@ describe Projects::ErrorTrackingController do
         let(:error) { build(:detailed_error_tracking_error) }
 
         it 'returns an error' do
-          expected_error = error.as_json.except('first_release_version').merge({ 'gitlab_commit' => nil })
+          expected_error = error.as_json.except('first_release_version').merge(
+            {
+              'gitlab_commit' => nil,
+              'gitlab_commit_path' => nil
+            }
+          )
 
           expect(response).to have_gitlab_http_status(:ok)
           expect(response).to match_response_schema('error_tracking/issue_detailed')

spec/factories/error_tracking/detailed_error.rb

@@ -34,7 +34,7 @@ FactoryBot.define do
     last_release_last_commit { '9ad419c86' }
     first_release_short_version { 'abc123' }
     last_release_short_version { 'abc123' }
-    first_release_version { '123456' }
+    first_release_version { '12345678' }
 
     skip_create
   end

spec/fixtures/api/schemas/error_tracking/error_detailed.json

@@ -56,7 +56,8 @@
     "last_release_last_commit": { "type": ["string", "null"] },
     "first_release_short_version": { "type": ["string", "null"] },
     "last_release_short_version": { "type": ["string", "null"] },
-    "gitlab_commit": { "type": ["string", "null"] }
+    "gitlab_commit": { "type": ["string", "null"] },
+    "gitlab_commit_path": { "type": ["string", "null"] }
   },
   "additionalProperties": false
 }

spec/graphql/types/error_tracking/sentry_detailed_error_type_spec.rb

@@ -31,6 +31,7 @@ describe GitlabSchema.types['SentryDetailedError'] do
       firstReleaseShortVersion
       lastReleaseShortVersion
       gitlabCommit
+      gitlabCommitPath
     ]
 
     is_expected.to have_graphql_fields(*expected_fields)

spec/models/error_tracking/project_error_tracking_setting_spec.rb

@@ -4,6 +4,7 @@ require 'spec_helper'
 
 describe ErrorTracking::ProjectErrorTrackingSetting do
   include ReactiveCachingHelpers
+  include Gitlab::Routing
 
   let_it_be(:project) { create(:project) }
 
@@ -213,7 +214,7 @@ describe ErrorTracking::ProjectErrorTrackingSetting do
   describe '#issue_details' do
     let(:issue) { build(:detailed_error_tracking_error) }
     let(:sentry_client) { double('sentry_client', issue_details: issue) }
-    let(:commit_id) { '123456' }
+    let(:commit_id) { issue.first_release_version }
 
     let(:result) do
       subject.issue_details
@@ -230,6 +231,7 @@ describe ErrorTracking::ProjectErrorTrackingSetting do
       it { expect(result).to eq(issue: issue) }
       it { expect(result[:issue].first_release_version).to eq(commit_id) }
       it { expect(result[:issue].gitlab_commit).to eq(nil) }
+      it { expect(result[:issue].gitlab_commit_path).to eq(nil) }
 
       context 'when release version is nil' do
         before do
@@ -237,6 +239,7 @@ describe ErrorTracking::ProjectErrorTrackingSetting do
         end
 
         it { expect(result[:issue].gitlab_commit).to eq(nil) }
+        it { expect(result[:issue].gitlab_commit_path).to eq(nil) }
       end
 
       context 'when repo commit matches first relase version' do
@@ -248,6 +251,7 @@ describe ErrorTracking::ProjectErrorTrackingSetting do
         end
 
         it { expect(result[:issue].gitlab_commit).to eq(commit_id) }
+        it { expect(result[:issue].gitlab_commit_path).to eq("/#{project.namespace.path}/#{project.path}/commit/#{commit_id}") }
       end
     end
 

spec/policies/user_policy_spec.rb

@@ -48,36 +48,4 @@ describe UserPolicy do
   describe "updating a user" do
     it_behaves_like 'changing a user', :update_user
   end
-
-  describe "updating a user's name" do
-    context 'when the ability to update their name is not disabled for users' do
-      before do
-        stub_application_setting(updating_name_disabled_for_users: false)
-      end
-
-      it_behaves_like 'changing a user', :update_name
-    end
-
-    context 'when the ability to update their name is disabled for users' do
-      before do
-        stub_application_setting(updating_name_disabled_for_users: true)
-      end
-
-      context 'for a regular user' do
-        it { is_expected.not_to be_allowed(:update_name) }
-      end
-
-      context 'for a ghost user' do
-        let(:current_user) { create(:user, :ghost) }
-
-        it { is_expected.not_to be_allowed(:update_name) }
-      end
-
-      context 'for an admin user' do
-        let(:current_user) { create(:admin) }
-
-        it { is_expected.to be_allowed(:update_name) }
-      end
-    end
-  end
 end

spec/requests/api/settings_spec.rb

@@ -136,14 +136,6 @@ describe API::Settings, 'Settings' do
       expect(json_response['performance_bar_allowed_group_id']).to eq(group.id)
     end
 
-    it "supports updating_name_disabled_for_users" do
-      put api("/application/settings", admin),
-        params: { updating_name_disabled_for_users: true }
-
-      expect(response).to have_gitlab_http_status(200)
-      expect(json_response['updating_name_disabled_for_users']).to eq(true)
-    end
-
     it "supports legacy performance_bar_enabled" do
       put api("/application/settings", admin),
         params: {

spec/requests/api/users_spec.rb

@@ -645,21 +645,6 @@ describe API::Users do
       expect(response).to have_gitlab_http_status(200)
     end
 
-    context 'updating name' do
-      context 'when the ability to update their name is disabled for users' do
-        before do
-          stub_application_setting(updating_name_disabled_for_users: true)
-        end
-
-        it 'updates the user with new name' do
-          put api("/users/#{user.id}", admin), params: { name: 'New Name' }
-
-          expect(response).to have_gitlab_http_status(200)
-          expect(json_response['name']).to eq('New Name')
-        end
-      end
-    end
-
     it "updates user with new bio" do
       put api("/users/#{user.id}", admin), params: { bio: 'new test bio' }
 

spec/services/users/update_service_spec.rb

@@ -6,46 +6,6 @@ describe Users::UpdateService do
   let(:user) { create(:user) }
 
   describe '#execute' do
-    context 'updating name' do
-      context 'when the ability to update their name is not disabled for users' do
-        before do
-          stub_application_setting(updating_name_disabled_for_users: false)
-        end
-
-        it 'updates the name' do
-          result = update_user(user, name: 'New Name')
-
-          expect(result).to eq(status: :success)
-          expect(user.name).to eq('New Name')
-        end
-      end
-
-      context 'when the ability to update their name is disabled for users' do
-        before do
-          stub_application_setting(updating_name_disabled_for_users: true)
-        end
-
-        context 'executing as a regular user' do
-          it 'does not update the name' do
-            result = update_user(user, name: 'New Name')
-
-            expect(result).to eq(status: :success)
-            expect(user.name).not_to eq('New Name')
-          end
-        end
-
-        context 'executing as an admin user' do
-          it 'updates the name' do
-            admin = create(:admin)
-            result = described_class.new(admin, { user: user, name: 'New Name' }).execute
-
-            expect(result).to eq(status: :success)
-            expect(user.name).to eq('New Name')
-          end
-        end
-      end
-    end
-
     it 'updates time preferences' do
       result = update_user(user, timezone: 'Europe/Warsaw', time_display_relative: true, time_format_in_24h: false)