Skip to content

Commit 9c4b115

Browse files
committed
Merged PR 913588: Whitelist canaryhigh allowed powerbi sov host
Whitelist canaryhigh allowed powerbi sov host ---- #### AI description (iteration 1) #### PR Classification Enhancement – updating the allowed PowerBI SOV host whitelist. #### PR Summary This PR updates the regular expression in `src/util.ts` to include the `canaryhigh` domain in the allowed PowerBI SOV whitelist, ensuring that requests from the canaryhigh host are validated correctly. - `src/util.ts`: Modified the regex `allowedPowerBiHostsSovRegex` to add a new option for `.canaryhigh.` among the allowed host patterns. <!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot --> Related work items: #2011901
1 parent 188918b commit 9c4b115

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

src/util.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const allowedPowerBiHostsRegex =
1212
/**
1313
* @hidden
1414
*/
15-
const allowedPowerBiHostsSovRegex = new RegExp(/^app\.powerbi\.cn$|^app(\.mil\.|\.high\.|\.)powerbigov\.us$|^app\.powerbi\.eaglex\.ic\.gov$|^app\.powerbi\.microsoft\.scloud$/);
15+
const allowedPowerBiHostsSovRegex = new RegExp(/^app\.powerbi\.cn$|^app(\.mil\.|\.canaryhigh\.|\.high\.|\.)powerbigov\.us$|^app\.powerbi\.eaglex\.ic\.gov$|^app\.powerbi\.microsoft\.scloud$/);
1616

1717
/**
1818
* @hidden

test/util.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ describe('util', function () {
1111
"https://daily.powerbi.com",
1212
"https://app.powerbi.cn",
1313
"https://app.powerbigov.us/",
14+
"https://app.canaryhigh.powerbigov.us",
1415
"https://app.powerbigov.us/reportEmbed?someQueryParam=x",
1516
"https://msit.fabric.microsoft.com/groups/someGroup/reports/",
1617
"HTTPS://APP.powErbi.com",

0 commit comments

Comments
 (0)