You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Markdown.markdown stuck in infinite loop <!-- leading to memory leaks #1586
the package isn't handling content with <!-- essentially i think the < is causing the issue and leading to major memory leak
input_string = '''#Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lore \n\nLorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lore.\n\nALorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an .\n\nLorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an .\n\n## HTML Conditional Statements: A Dual-Edged Sword\n\nUpon closer examination of the email's HTML code, it became evident that the threat actors leveraged HTML conditional comments, specifically <!--[if mso]> and <!--[if !mso]>'''
trying markdown.markdown(input_string)
The processing is stuck and goes into an infinite loop
My understanding of the repo or markdown, in general, might be wrong :)
The infinite loop problem is fixed on master, but the example you've given still doesn't render correctly. Sigh, I'll have to dig in further and figure out how we can ensure consistent results.
No, that's why I referenced the branch and not a release, but it appears that fix, while it doesn't hang, drops data, which is why I opened a new PR that should completely fix it.
Activity
facelessuser commentedon Jan 20, 2026
The infinite loop problem is fixed on
master, but the example you've given still doesn't render correctly. Sigh, I'll have to dig in further and figure out how we can ensure consistent results.More HTML fixes
facelessuser commentedon Jan 20, 2026
I have a fix in #1587.
Neeraj-Palliyali commentedon Jan 20, 2026
@facelessuser thanks, is master 3.10?
facelessuser commentedon Jan 21, 2026
No, that's why I referenced the branch and not a release, but it appears that fix, while it doesn't hang, drops data, which is why I opened a new PR that should completely fix it.
Neeraj-Palliyali commentedon Jan 21, 2026
@facelessuser Thanks 👍
More HTML fixes
chore(deps): update dependency markdown to v3.10.1 (#246)