The “Discount/GFM” processor does not correctly handled embedded “)” characters in URLs.

John Hart's Avatar

John Hart

13 Sep, 2017 04:21 PM

// Please enter a detailed description of the issue
// Description of the issue:

The “Discount/GFM” processor does not correctly handled embedded “)” characters in URLs.

// Please enter a list of steps required to reproduce the issue
// Steps to reproduce:

Use this text in a markdown document:

[link](http://www.google.com?q=()&foo=bar)

The display text should read only “link” and should link to “http://www.google.com?q=()&foo=bar”.

If you use the MultiMarkdown processor, that is what happens = good.

If you use the Discount/GFM processor, the URL is truncated at the first “)”, so the display looks like this:

link&foo=bar)

With “link” an HREF that points to “http://www.google.com?q=(“

NOTE: if you enter the same markdown text into github (eg, on a github wiki page) it does the right thing. So this is a bug in the “Discount” processor, not in the behavior/specification of GFM itself.

// If applicable please attach a sample document that causes the issue for you.

// Configuration (do not edit)
Marked 2 930 (Direct)
OS X Version: 10.11.6
---
additionalMathJaxConfig:
allowPageBreakInCode: 1
bookTxtIsLeanpub: 0
summaryMdIsGitBook: 0
codeIsPoetry: 0
collapsibleHeadlines: 0
convertGithubCheckboxes: 1
convertYAMLToMMD: 0
defaultMathJaxConfig: TeX-AMS-MML_HTMLorMML
defaultPreviewStyle: GitHub
defaultProcessor: MultiMarkdown
enableMiniMap: 0
excludeBlockQuotesFromCounts: 0
excludeFootnotesFromCounts: 0
excludeVerbatimFromCounts: 0
excludeCaptionsFromCounts: 0
folderPreviewExtensionsArray: md, markdown, txt, mdown, html, opml
githubNewlines: 1
h1IsPageBreak: 0
h2IsPageBreak: 0
hrIsPageBreak: 0
includeMathJax: 1
limitTextWidth: 0
minimalJavaScript: 0
outlineModeExtensions:
permissionGranted: 0
printHeaderBottomLeft:
printHeaderBottomRight:
printHeaderFooterFirstPage: 0
printHeaderTopLeft:
printHeaderTopRight:
processHTMLFiles: 1
removeMetadataForPreview: 0
removeYAMLHeaders: 0
scrollToEdit: 1
shouldDetectCritic: 1
shouldHighlightMarkupErrors: 0
shouldLoadCritic: 0
syntaxHighlight: 1

  1. Support Staff 1 Posted by Brett on 13 Sep, 2017 05:20 PM

    Brett's Avatar

    Thanks for the report, I'll look into it.

    -Brett

Reply to this discussion

Internal reply

Formatting help / Preview (switch to plain text) No formatting (switch to Markdown)

Attaching KB article:

»

Already uploaded files

  • sample.md 44 Bytes

Attached Files

You can attach files up to 10MB

If you don't have an account yet, we need to confirm you're human and not a machine trying to post spam.

Keyboard shortcuts

Generic

? Show this help
ESC Blurs the current field

Comment Form

r Focus the comment reply box
^ + ↩ Submit the comment

You can use Command ⌘ instead of Control ^ on Mac