Skip to content

Commit 15084c5

Browse files
chintankavathiaspike-rabbit
authored andcommitted
fix: bind empty message string as plain text to avoid xss
BREAKING CHANGE: `emptyMessage` is no longer interpreted as HTML to prevent XSS attacks. Use content projection for displaying an HTML empty content message: ``` <ngx-datatable> <div empty-content> My rich <i>html</i> content. </div> </ngx-datatable> ```
1 parent 1f49101 commit 15084c5

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

projects/ngx-datatable/src/lib/components/datatable.component.html

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -79,11 +79,9 @@
7979
</ng-content>
8080
<ng-content select="[empty-content]" ngProjectAs="[empty-content]">
8181
<div role="row">
82-
<div
83-
role="cell"
84-
class="empty-row"
85-
[innerHTML]="messages.emptyMessage ?? 'No data to display'"
86-
></div>
82+
<div role="cell" class="empty-row">
83+
{{ messages.emptyMessage ?? 'No data to display' }}
84+
</div>
8785
</div>
8886
</ng-content>
8987
</datatable-body>

0 commit comments

Comments
 (0)