Skip to content

Commit c3c3c52

Browse files
Address code review feedback: remove optional chaining and document CSP limitations
Co-authored-by: granatonatalia <[email protected]>
1 parent ad15316 commit c3c3c52

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

src/app/pages/mapping/mapping.component.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ export class MappingComponent implements OnInit, AfterViewInit {
180180
// Auto-fit columns for better readability
181181
worksheet.columns.forEach(column => {
182182
let maxLength = 0;
183-
column.eachCell?.({ includeEmpty: true }, cell => {
183+
column.eachCell({ includeEmpty: true }, cell => {
184184
const cellLength = cell.value ? cell.value.toString().length : 10;
185185
if (cellLength > maxLength) {
186186
maxLength = cellLength;

src/index.html

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55
<title>DSOMM</title>
66
<base href="/" />
77
<meta name="viewport" content="width=device-width, initial-scale=1" />
8+
<!-- Content Security Policy for XSS mitigation (Dependabot #58)
9+
Note: 'unsafe-inline' and 'unsafe-eval' are required for Angular 13.
10+
These will be removed when upgrading to Angular 19+ which supports stricter CSP. -->
811
<meta
912
http-equiv="Content-Security-Policy"
1013
content="default-src 'self';

0 commit comments

Comments
 (0)