erming/shout

URLs ending with a close parentheses are not linked properly

Open

#255 aberto em 22 de out. de 2014

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (3.640 stars) (293 forks)batch import
bughelp wanted

Description

Not if there are other characters that have this occur, but I noticed it with this url:

image

http://en.wikipedia.org/wiki/The_Martian_(Andy_Weir)

Guia do colaborador

Pilha de tecnologia
javascripthtmlcss
Domain
frontendbackend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
JavaScriptregular expressions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
The issue is that URLs containing parentheses, like Wikipedia links, are not properly linked because the parentheses are not escaped or the URL detection regex does not account for them. To fix, examine the URL parsing logic in shout's source code, likely in a file that handles message rendering or linkification. Look for a regex pattern that extracts URLs and modify it to handle balanced parentheses, or use a library like 'url regex' that handles this. Test with the given example URL. There may be existing comments or PRs for reference.