From cb05a4ae1bb9f0875b75b48c530767db1031ebae Mon Sep 17 00:00:00 2001
From: Yunwen Zheng <yunwen.zheng@grafana.com>
Date: Thu, 27 Nov 2025 01:03:36 -0500
Subject: [PATCH] usePullRequestParam: Provisioned dashboard preview banner,
 decode pull request url before sanitize (#114516)

usePullRequestParam: decode url before sanitize
---
 .../components/Shared/PreviewBannerViewPR.tsx         | 11 ++++++++---
 .../provisioning/hooks/usePullRequestParam.ts         |  8 ++++----
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/public/app/features/provisioning/components/Shared/PreviewBannerViewPR.tsx b/public/app/features/provisioning/components/Shared/PreviewBannerViewPR.tsx
index 8a796ee049c..15b814a14db 100644
--- a/public/app/features/provisioning/components/Shared/PreviewBannerViewPR.tsx
+++ b/public/app/features/provisioning/components/Shared/PreviewBannerViewPR.tsx
@@ -107,9 +107,14 @@ export function PreviewBannerViewPR({ prParam, isNewPr, behindBranch, repoUrl, b
       {/* when repo type is not local, we show branch information */}
       {showBranchInfo(repoType, branchInfo) && (
         <Box marginTop={1}>
-          <Trans i18nKey="provisioned-resource-preview-banner.preview-banner.branch-text">branch: </Trans>
-          <TextLink href={getBranchUrl(repoBaseUrl!, targetBranch!, repoType)}>{targetBranch}</TextLink> {'\u2192'}{' '}
-          <TextLink href={getBranchUrl(repoBaseUrl!, configuredBranch!, repoType)}>{configuredBranch}</TextLink>
+          <Trans i18nKey="provisioned-resource-preview-banner.preview-banner.branch-text">branch:</Trans>{' '}
+          <TextLink href={getBranchUrl(repoBaseUrl!, targetBranch!, repoType)} external>
+            {targetBranch}
+          </TextLink>{' '}
+          {'\u2192'}{' '}
+          <TextLink href={getBranchUrl(repoBaseUrl!, configuredBranch!, repoType)} external>
+            {configuredBranch}
+          </TextLink>
         </Box>
       )}
     </Alert>
diff --git a/public/app/features/provisioning/hooks/usePullRequestParam.ts b/public/app/features/provisioning/hooks/usePullRequestParam.ts
index 81d10210503..8ad124953a4 100644
--- a/public/app/features/provisioning/hooks/usePullRequestParam.ts
+++ b/public/app/features/provisioning/hooks/usePullRequestParam.ts
@@ -9,9 +9,9 @@ export const usePullRequestParam = () => {
   const repoType = params.get('repo_type');
 
   return {
-    prURL: prParam ? textUtil.sanitizeUrl(prParam) : undefined,
-    newPrURL: newPrParam ? textUtil.sanitizeUrl(newPrParam) : undefined,
-    repoURL: repoUrl ? textUtil.sanitizeUrl(repoUrl) : undefined,
-    repoType: repoType ? textUtil.sanitizeUrl(repoType) : undefined,
+    prURL: prParam ? textUtil.sanitizeUrl(decodeURIComponent(prParam)) : undefined,
+    newPrURL: newPrParam ? textUtil.sanitizeUrl(decodeURIComponent(newPrParam)) : undefined,
+    repoURL: repoUrl ? textUtil.sanitizeUrl(decodeURIComponent(repoUrl)) : undefined,
+    repoType: repoType ? textUtil.sanitizeUrl(decodeURIComponent(repoType)) : undefined,
   };
 };