link-patterns extra replaces matches inside inline HTML tags and destroys email auto-links
Mantenedores costumam responder em até 3 dias
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 30/100
Direção de pesquisa
Start by reading LinkPatterns.run and is_inside_link in lib/markdown2.py, then inspect the inline HTML tokenization and auto-link tests. Reproduce the three examples with the link-patterns extra and compare them with output without it. Done means matches in prose are linkified while tag attributes and email auto-link tokens remain intact; pull request #753 is already open.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Summary
With the link-patterns extra enabled, a pattern that matches inside an inline HTML tag (its attributes) or inside a <...> auto-link token gets replaced there, corrupting the HTML and destroying email auto-links. The same input renders correctly without the extra.
Repro
import re
import markdown2
lp = [(re.compile(r"\b(word)\b"), r"/w/\1")]
print(markdown2.markdown('See <span title="word">x</span>', extras=["link-patterns"], link_patterns=lp))
print(markdown2.markdown('<[email protected]>', extras=["link-patterns"], link_patterns=lp))
print(markdown2.markdown('<img src="/x.png" alt="word">', extras=["link-patterns"], link_patterns=lp))
Output with the extra:
<p>See <span title="<a href="/w/word">word</a>">x</span></p>
<p><<a href="/w/word">word</a>@example.com></p>
<p><img src="/x.png" alt="<a href="/w/word">word</a>"></p>
Expected (matches what plain markdown2 does for the markup, while prose matches are linkified):
<p>See <span title="word">x</span></p>
<p><a href="mailto:[email protected]">[email protected]</a></p>
<p><img src="/x.png" alt="word"></p>
Three distinct symptoms:
<span title="word">x</span>— an anchor gets inserted inside thetitleattribute value, producing invalid HTML (and, depending on the pattern, allowing attribute content to be rewritten inside a raw tag).<[email protected]>— the mailto auto-link is destroyed entirely: the match splits the token so the auto-linker no longer recognizes it, and the angle brackets end up escaped.<img src="/x.png" alt="word">— thealtattribute is corrupted the same way.
Root cause
LinkPatterns.run (lib/markdown2.py) guards matches with is_inside_link, which checks only _auto_link_re (<http://...> style auto-links) and _basic_link_re (markdown [...](...) syntax) — #385/#446 covered those. It does not check the other tokens that markdown2's own tokenizer _sorta_html_tokenize_re classifies as inline HTML markup: tags, comments, processing instructions and <[email protected]> style auto-link tokens. _do_auto_links runs after _do_links, so at the time the extra runs the email auto-link is still the raw <[email protected]> text and a match inside it destroys it.
Environment
python-markdown2 2.5.6 (current master, 6e89cff), Python 3.13, Linux.
- Linguagem predominante
- Python
- Estrelas
- 2.8k
- Forks
- 477
- Merge médio
- 3d 19h
- PRs com merge (30d)
- 14
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de trentm/python-markdown2
-
Add py.typed markerTalvez já em andamento @mac-agent assumiu há 222 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
trentm/python-markdown2#696 ·
Mantenedores costumam responder em até 3 dias
-
wavedrom extra bypasses safe_mode and allows XSSTalvez já em andamento @adityaanikam assumiu há 1 dia. AbertaBug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 25/100
trentm/python-markdown2#765 ·
Mantenedores costumam responder em até 3 dias
-
Reused `Markdown` instance keeps link URLs and code spans from earlier documentsTalvez já em andamento @HongTin2104 assumiu há 1 dia. AbertaTech debt
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 53/100
trentm/python-markdown2#763 ·
Mantenedores costumam responder em até 3 dias
-
Code spans that wrap onto the next line are no longer recognised (regression from the #702 fix)Talvez já em andamento @awais-ur-rehman assumiu há 7 dias. Aberta
Dificuldade 4/5 1-2 dias Facilidade para iniciantes 55/100
trentm/python-markdown2#756 ·
Mantenedores costumam responder em até 3 dias
-
Quadratic-time inline-link parsing: many unclosed [...]( cause a CPU DoSTalvez já em andamento @ParamTanna assumiu há 9 dias. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 30/100
trentm/python-markdown2#750 ·
Mantenedores costumam responder em até 3 dias
Todas as issues de trentm/python-markdown2
Issues semelhantes
-
Remove the `pull_request_target` workflow before GitHub blocks itTalvez já em andamento @jat255 assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 2 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
IBM/mcp-context-forge#7483 ·
Mantenedores costumam responder em até 2 dias
-
bug good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
-
[Bug Report]: OpenAI 辅助 API 连通性测试因输出上限仅 1 token 误报“未知错误”Talvez já em andamento @fjfanjian assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Project-N-E-K-O/N.E.K.O#3640 ·
Mantenedores costumam responder em até 1 dia
-
author-page correction
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 64/100
acl-org/acl-anthology#10808 · 1 comentário ·
Mantenedores costumam responder em até 1 dia