diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-09-20 14:18:08 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-09-20 14:18:08 +0300 |
commit | 5afcbe03ead9ada87621888a31a62652b10a7e4f (patch) | |
tree | 9918b67a0d0f0bafa6542e839a8be37adf73102d /app/assets/javascripts/lib/utils/url_utility.js | |
parent | c97c0201564848c1f53226fe19d71fdcc472f7d0 (diff) |
Add latest changes from gitlab-org/gitlab@16-4-stable-eev16.4.0-rc42
Diffstat (limited to 'app/assets/javascripts/lib/utils/url_utility.js')
-rw-r--r-- | app/assets/javascripts/lib/utils/url_utility.js | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/app/assets/javascripts/lib/utils/url_utility.js b/app/assets/javascripts/lib/utils/url_utility.js index 08c98298121..ea0520e3157 100644 --- a/app/assets/javascripts/lib/utils/url_utility.js +++ b/app/assets/javascripts/lib/utils/url_utility.js @@ -1,5 +1,3 @@ -import * as Sentry from '@sentry/browser'; - export const DASH_SCOPE = '-'; export const PATH_SEPARATOR = '/'; @@ -705,11 +703,7 @@ export function visitUrl(destination, external = false) { } if (!isSafeURL(url)) { - // For now log this to Sentry and do not block the execution. - // See https://gitlab.com/gitlab-org/gitlab/-/merge_requests/121551#note_1408873600 - // for more context. Once we're sure that it's not breaking functionality, we can use - // a RangeError here (throw new RangeError('Only http and https protocols are allowed')). - Sentry.captureException(new RangeError(`Only http and https protocols are allowed: ${url}`)); + throw new RangeError(`Only http and https protocols are allowed: ${url}`); } if (external) { |