3
5
Fork
You've already forked DNSSEC-DANE_Padlock
0

Minor security warning in blocked.js:12:1 #15

Closed
opened 2025年08月13日 13:15:30 +02:00 by Seb35 · 0 comments
Owner
Copy link

The automatic review by AMO on version 0.2.1 reports the following warning:

Unsafe assignment to innerHTML
Warning: Due to both security and performance concerns, this may not be set using dynamic values which have not been adequately sanitized. This can lead to security issues or fairly serious performance degradation.
blocked.js line 12 column 1

Indeed, this should be better sanitised to avoid a XSS coming from a rogue translation (even if the translation are obviously reviewed before submission).

The automatic review by AMO on [version 0.2.1](https://codeberg.org/Seb35/DNSSEC-DANE_Padlock/src/tag/0.2.1) reports the following warning: > Unsafe assignment to innerHTML > Warning: Due to both security and performance concerns, this may not be set using dynamic values which have not been adequately sanitized. This can lead to security issues or fairly serious performance degradation. > [blocked.js line 12 column 1](https://codeberg.org/Seb35/DNSSEC-DANE_Padlock/src/commit/e2b0c3e99ac8ff117e5b48e900ced6ded6393bd1/src/blocked.js#L12) Indeed, this should be better sanitised to avoid a XSS coming from a rogue translation (even if the translation are obviously reviewed before submission).
Sign in to join this conversation.
No Branch/Tag specified
main
0.3.1
0.3.0
0.2.5
0.2.4
0.2.3
0.2.2
0.2.1
0.2.0
0.1.0
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Seb35/DNSSEC-DANE_Padlock#15
Reference in a new issue
Seb35/DNSSEC-DANE_Padlock
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?