Defect #23019
closedMarkdown doesn't recognize mails
0%
Description
Redmine hides this email in markdown:
<a_bc@def.com>
But it doesn't hide this:
<abc@def.com>
Files
Updated by Go MAEDA over 7 years ago
The latest version of Redcarpet still have this problem (we upgraded Redcarpet to 3.4.0 in r16583).
Updated by Marius BĂLTEANU almost 7 years ago
- File markdown_emails.png markdown_emails.png added
- Status changed from New to Confirmed
kay rus wrote:
Go MAEDA wrote:
The latest version of Redcarpet still have this problem (we upgraded Redcarpet to 3.4.0 in r16583).
Redmine is not affected by the respective issue because we have no_intra_emphasis: true
and the emails with _ are recognised correctly. Please see the below screenshot.
The issue with <a_bc@def.com>
looks to be from the same Redcarpet gem, the text is filtered out because it is recognised wrongly as HTML tag.
Rails console:
With :filter_html => true
:
irb(main):012:0> Redcarpet::Markdown.new(Redmine::WikiFormatting::Markdown::HTML.new(:filter_html => true, :hard_wrap => true), autolink: true, no_intra_emphasis: true).render("<a_bc@def.com>") => ""
With :filter_html => false
:
irb(main):013:0> Redcarpet::Markdown.new(Redmine::WikiFormatting::Markdown::HTML.new(:filter_html => false, :hard_wrap => true), autolink: true, no_intra_emphasis: true).render("<a_bc@def.com>") => "<p><a_bc@def.com></p>\n"
Updated by Go MAEDA 10 months ago
- Status changed from Confirmed to Closed
- Resolution set to Wont fix
Using CommonMark Markdown instead of Redcarpet-based Markdown can avoid the issue.
Please switch to CommonMark Markdown. Redcarpet-based Markdown was marked as "deprecated" (#36807) in Redmine 5.1.0 and will be removed in the future version.