Skip to content

Commit 45872a2

Browse files
authored
Merge pull request #2819 from UltimateHackingKeyboard/fix-escape-macro-name
fix: escape some html content
2 parents 4826b6b + 72e52b3 commit 45872a2

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

packages/uhk-web/src/app/components/device/firmware/device-firmware.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ <h1>
1010
<ul class="list-unstyled">
1111
<li *ngFor="let state of firmwareUpgradeStates.modules; trackBy:firmwareUpgradeStateTrackByFn;">
1212
<ng-template #htmlTooltipTemplate>
13-
<div [innerHTML]="state.checksumTooltip | safeHtml"></div>
13+
<div [innerHTML]="state.checksumTooltip | escapeHtml | safeHtml"></div>
1414
</ng-template>
1515
<span class="upgrading"
1616
[ngbTooltip]="htmlTooltipTemplate"

packages/uhk-web/src/app/store/index.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ import {
5757
import { PrivilagePageSate } from '../models/privilage-page-sate';
5858
import { SelectOptionData } from '../models/select-option-data';
5959
import { defaultUhkThemeColors } from '../util/default-uhk-theme-colors';
60+
import { escapeHtml } from '../util/escape-html';
6061
import { parseStatusBuffer } from '../util/status-buffer-parser';
6162
import { addMissingModuleConfigs } from './reducers/add-missing-module-configs';
6263

@@ -769,7 +770,7 @@ export const upgradeAgentTooltip = createSelector(
769770
return '';
770771
}
771772

772-
return `rightHalf.firmware.userConfigVersion ${hardwareModules.rightModuleInfo.userConfigVersion} minor version is larger than agent.userConfigVersion ${VERSIONS.userConfigVersion}. <br /> <br /> rightHalf.flashedUserConfigVersion: ${newUserConfiguration.newUserConfigurationVersion}`;
773+
return `rightHalf.firmware.userConfigVersion ${escapeHtml(hardwareModules.rightModuleInfo.userConfigVersion)} minor version is larger than agent.userConfigVersion ${VERSIONS.userConfigVersion}. <br /> <br /> rightHalf.flashedUserConfigVersion: ${escapeHtml(newUserConfiguration.newUserConfigurationVersion)}`;
773774
});
774775
export const upgradeFirmwareTooltip = createSelector(
775776
getHardwareModules,

packages/uhk-web/src/app/util/status-buffer-parser.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ function transformToErrorBlock(macros: Macro[], block: string): any {
6767
}
6868

6969
const url = `#/macro/${macro.id}?actionIndex=${macroActionIndex}&lineNr=${lineNr}&columnNr=${columnNr}&inlineEdit=true`;
70-
const newLine2 = `${line1Result[1]}<a href="${url}">${escapeHtml(line1Result[2])}</a>`;
70+
const newLine2 = `${escapeHtml(line1Result[1])}<a href="${url}">${escapeHtml(line1Result[2])}</a>`;
7171

72-
return `${lines[0]}\n${newLine2}\n${lines[2]}\n`;
72+
return `${escapeHtml(lines[0])}\n${newLine2}\n${escapeHtml(lines[2])}\n`;
7373
}

0 commit comments

Comments
 (0)