Don't escape plus signs in URLs

Crozzers · Crozzers · commit fe0b1f2896aa · 2025-03-16T12:00:29.000Z
diff --git a/lib/markdown2.py b/lib/markdown2.py
@@ -4099,8 +4099,6 @@ def _html_escape_url(
         .replace('<', '&lt;')
         .replace('>', '&gt;'))
     if safe_mode:
-        if charset != 'base64':
-            escaped = escaped.replace('+', ' ')
         escaped = escaped.replace("'", "&#39;")
     return escaped
 
diff --git a/test/tm-cases/safe_mode_issue621.html b/test/tm-cases/safe_mode_issue621.html
@@ -0,0 +1 @@
+<p><a href="https://chromium.googlesource.com/v8/v8.git/+/refs/heads/beta">Chromium</a></p>
diff --git a/test/tm-cases/safe_mode_issue621.opts b/test/tm-cases/safe_mode_issue621.opts
@@ -0,0 +1 @@
+{'safe_mode': 'escape'}
diff --git a/test/tm-cases/safe_mode_issue621.text b/test/tm-cases/safe_mode_issue621.text
@@ -0,0 +1 @@
+[Chromium](https://chromium.googlesource.com/v8/v8.git/+/refs/heads/beta)

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+<p><a href="https://chromium.googlesource.com/v8/v8.git/+/refs/heads/beta">Chromium</a></p>`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+[Chromium](https://chromium.googlesource.com/v8/v8.git/+/refs/heads/beta)`