MacDownApp / MacDownApp/macdown
Markdown script Injection problem?
Open
Nobody has claimed this yet.
on hold
- Dominant language
- Rich Text Format
- Stars
- 9.8k
- Forks
- 1.2k
- PR merge metrics
- No merged PRs in 30d
Description
Hi Tzu-ping,
It seems some script injection problem in <pre> block, Can Macdown sanitize it?
For example：
<pre>
<script>
window.location.href='http://example.com';
</script>
</pre>
BTW, MacDown is the best markdown editor on OS X. I love it!
Thanks.
Contributor guide
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.
Research direction
Start by reproducing the supplied Markdown example in MacDown and inspect how its Markdown-to-HTML rendering handles script content inside a pre block. Done means the example renders as inert text and cannot redirect the page or execute script.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- markdown
- Domain
- desktop, security
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100