Commit d32802c7 authored by Jamie Schembri's avatar Jamie Schembri

Fix #49272 - Sanitize git URL in import errors

parent 25af7197
...@@ -23,7 +23,7 @@ module Projects ...@@ -23,7 +23,7 @@ module Projects
success success
rescue => e rescue => e
error("Error importing repository #{project.import_url} into #{project.full_path} - #{e.message}") error("Error importing repository #{project.safe_import_url} into #{project.full_path} - #{e.message}")
end end
private private
......
---
title: Sanitize git URL in import errors
merge_request:
author: Jamie Schembri
type: fixed
...@@ -69,7 +69,7 @@ describe Projects::ImportService do ...@@ -69,7 +69,7 @@ describe Projects::ImportService do
result = subject.execute result = subject.execute
expect(result[:status]).to eq :error expect(result[:status]).to eq :error
expect(result[:message]).to eq "Error importing repository #{project.import_url} into #{project.full_path} - The repository could not be created." expect(result[:message]).to eq "Error importing repository #{project.safe_import_url} into #{project.full_path} - The repository could not be created."
end end
context 'when repository creation succeeds' do context 'when repository creation succeeds' do
...@@ -141,7 +141,7 @@ describe Projects::ImportService do ...@@ -141,7 +141,7 @@ describe Projects::ImportService do
result = subject.execute result = subject.execute
expect(result[:status]).to eq :error expect(result[:status]).to eq :error
expect(result[:message]).to eq "Error importing repository #{project.import_url} into #{project.full_path} - Failed to import the repository" expect(result[:message]).to eq "Error importing repository #{project.safe_import_url} into #{project.full_path} - Failed to import the repository"
end end
context 'when repository import scheduled' do context 'when repository import scheduled' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment