diff --git a/lib/gitlab/ci/pipeline/chain/validate/external.rb b/lib/gitlab/ci/pipeline/chain/validate/external.rb index b569e3444a1421c4c77840c8bbc63e7ec1f81d27..e006bd2ab3fce580e478fbf49c251e096780b01f 100644 --- a/lib/gitlab/ci/pipeline/chain/validate/external.rb +++ b/lib/gitlab/ci/pipeline/chain/validate/external.rb @@ -18,7 +18,7 @@ def perform! pipeline_authorized = validate_external log_message = pipeline_authorized ? 'authorized' : 'not authorized' - Gitlab::AppLogger.info(message: "Pipeline #{log_message}", project_id: project.id, user_id: current_user.id) + Gitlab::AppLogger.info(message: "Pipeline #{log_message}", project_id: project.id) return if pipeline_authorized diff --git a/spec/lib/gitlab/ci/pipeline/chain/validate/external_spec.rb b/spec/lib/gitlab/ci/pipeline/chain/validate/external_spec.rb index 5f22c342265a30e85ae835625e00a0d7d20b148d..11ba2d15fec94663454dffb46fbb7496c7f1a0d2 100644 --- a/spec/lib/gitlab/ci/pipeline/chain/validate/external_spec.rb +++ b/spec/lib/gitlab/ci/pipeline/chain/validate/external_spec.rb @@ -142,7 +142,7 @@ end it 'logs the authorization' do - expect(Gitlab::AppLogger).to receive(:info).with(message: 'Pipeline authorized', project_id: project.id, user_id: user.id) + expect(Gitlab::AppLogger).to receive(:info).with(message: 'Pipeline authorized', project_id: project.id) perform! end @@ -330,7 +330,7 @@ it 'logs the authorization' do allow(Gitlab::AppLogger).to receive(:info) - expect(Gitlab::AppLogger).to receive(:info).with(message: 'Pipeline not authorized', project_id: project.id, user_id: user.id) + expect(Gitlab::AppLogger).to receive(:info).with(message: 'Pipeline not authorized', project_id: project.id) perform! end @@ -354,7 +354,7 @@ end it 'logs the authorization' do - expect(Gitlab::AppLogger).to receive(:info).with(message: 'Pipeline not authorized', project_id: project.id, user_id: user.id) + expect(Gitlab::AppLogger).to receive(:info).with(message: 'Pipeline not authorized', project_id: project.id) perform! end