🚨Use of innerHTML
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 35/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- javascript
Research direction
Open dom.js at line 65 in the linked revision and inspect how the displayed information is inserted into the DOM. Determine a safer approach that preserves the expected output without the flagged innerHTML usage. Done means the information still appears correctly and the security concern is addressed.
Written by the indexing model from the issue text.
Description
https://github.com/fac-14/NDJS-API/blob/df236cfbc24fd346a72ea94df533a134282921c7/dom.js#L65
This can be a security issue - are there any other ways you could have inserted your info into the DOM without using this?
- Dominant language
- JavaScript
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from fac-14/NDJS-API
-
README needs content Open
Difficulty 1/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
-
Readability 📖 Open
Difficulty 2/5 1-3 hours Newbie friendliness 48/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
-
Duplicated code Open
Difficulty 2/5 1-3 hours Newbie friendliness 42/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
HarperFast/skills#96 ·
-
[Block] Latest Posts [Type] Bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Automattic/studio#4908 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
sugarlabs/musicblocks#8847 ·