[Django] #35533: urlize() makes a bit of a mess of links embedded in Markdown

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
------------------------------------------+------------------------
Reporter: Simon Willison | Owner: nobody
Type: Bug | Status: new
Component: Utilities | Version: 5.1
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
------------------------------------------+------------------------
I have this input text:

{{{
Annotated versions of talks I have given, with extensive notes and
additional links. Here's [how I make
these](https://simonwillison.net/2023/Aug/6/annotated-presentations/).
}}}

After running through Django's urlize helper function I got this:
{{{
... Here&#x27;s [how I make <a
href="http://these](https://simonwillison.net/2023/Aug/6/annotated-
presentations/)"
rel="nofollow">these](https://simonwillison.net/2023/Aug/6/annotated-
presentations/)</a>
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/35533>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown

--------------------------------+--------------------------------------

Reporter: Simon Willison | Owner: nobody
Type: Bug | Status: new
Component: Utilities | Version: 5.1

Severity: Normal | Resolution:

Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0

--------------------------------+--------------------------------------
Comment (by Simon Willison):

The ideal output for this example would be:

{{{
Annotated versions of talks I have given, with extensive notes and

additional links. Here's [how I make these](<a
href="https://simonwillison.net/2023/Aug/6/annotated-presentations/"
rel="nofollow">https://simonwillison.net/2023/Aug/6/annotated-
presentations/</a>).
}}}
Alternatively, not URLizing at all would be preferable to URLizing in a
way that produces broken links.

I think what's happening here is the logic that looks for non-protocol
links that include or end in .net may be kicking in, and deciding that the
following is the URL that should be linked:
{{{
these](https://simonwillison.net/2023/Aug/6/annotated-presentations/)
}}}
It's hard to suggest a fix for this. Ideally the code would "notice" that
`these](https://simonwillison.net/2023` is not a valid URL, but instead
the logic is deciding that it's probably valid but should have `http://`
glued on the start.

Maybe we could have code that notices that
`http://these](https://simonwillison.net/2023/Aug/6/annotated-
presentations/)` is NOT a valid URL - you cannot have `](` in the middle
of the hostname portion - and hence decides not to URLize it at all.

(Background: the reason I'm seeing this is that my Django SQL Dashboard
software tries to URLize text it displays, but has no way of knowing if a
database column contains Markdown - this broken example came from
https://simonwillison.net/dashboard/tags-with-descriptions/ )
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:1>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
--------------------------------+--------------------------------------
Reporter: Simon Willison | Owner: nobody
Type: Bug | Status: new
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------

Changes (by Simon Willison):

* Attachment "screenshot-of-markdown.jpg" added.

--
Ticket URL: <https://code.djangoproject.com/ticket/35533>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
--------------------------------+--------------------------------------
Reporter: Simon Willison | Owner: nobody
Type: Bug | Status: new
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------

Comment (by Simon Willison):

Current URLizer code is here:
https://github.com/django/django/blob/2719a7f8c161233f45d34b624a9df9392c86cc1b/django/utils/html.py#L258-L413
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:2>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
--------------------------------+------------------------------------

Reporter: Simon Willison | Owner: nobody
Type: Bug | Status: new
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:

Keywords: | Triage Stage: Accepted

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+------------------------------------

Changes (by Sarah Boyce):

* stage: Unreviewed => Accepted

Comment:

Thank you for the report! Replicated 👍
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:3>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
--------------------------------+------------------------------------
Reporter: Simon Willison | Owner: nobody
Type: Bug | Status: new
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+------------------------------------
Changes (by Sarah Boyce):

* cc: Adam Johnson (added)

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:4>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown

-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned

Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0

-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* owner: nobody => DongwookKim0823
* status: new => assigned

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:5>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Vaarun Sinha):

Hey DongwookKim0823? Are you actively working on this? Do you require any
help? Would love to help
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:6>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by DongwookKim0823):

Replying to [comment:6 Vaarun Sinha]:

I'm currently working on this. I'll leave a comment if I need any help.
Thank you! :)
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:7>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted

Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by DongwookKim0823):

* has_patch: 0 => 1

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:8>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "0001-Fixed-35533-Improved-urlize-function-to-handle-
markd.patch" added.

--
Ticket URL: <https://code.djangoproject.com/ticket/35533>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "0001-Fixed-markdown-hyperlink-handling-in-urlize-
function.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "0001-Fixed-35533-Improved-urlize-function-to-handle-
markd.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "0001-Fixed-markdown-hyperlink-handling-in-urlize-
function.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 1

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by Mariusz Felisiak):

* needs_better_patch: 0 => 1

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:9>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

--
Ticket URL: <https://code.djangoproject.com/ticket/35533>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" removed.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by DongwookKim0823):

* Attachment "changes.patch" added.

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| DongwookKim0823
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by DongwookKim0823):

Replying to [comment:6 Vaarun Sinha]:

I made some progress on this issue, but I think collaborating on a better
solution would be great. I’d love to work together and find the best
approach.
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:10>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown

--------------------------------+--------------------------------------
Reporter: Simon Willison | Owner: JaeHyuckSa

Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0

--------------------------------+--------------------------------------
Changes (by JaeHyuckSa):

* needs_better_patch: 1 => 0
* owner: DongwookKim0823 => JaeHyuckSa

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:11>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
--------------------------------+--------------------------------------
Reporter: Simon Willison | Owner: JaeHyuckSa
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 1

Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------

Changes (by Sarah Boyce):

* needs_better_patch: 0 => 1

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:12>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
--------------------------------+--------------------------------------
Reporter: Simon Willison | Owner: JaeHyuckSa
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------

Changes (by JaeHyuckSa):

* needs_better_patch: 1 => 0

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:13>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown

-------------------------------------+-------------------------------------

Reporter: Simon Willison | Owner:
| JaeHyuckSa
Type: Bug | Status: assigned
Component: Utilities | Version: 5.1
Severity: Normal | Resolution:

Keywords: | Triage Stage: Ready for
| checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0

-------------------------------------+-------------------------------------
Changes (by Sarah Boyce):

* stage: Accepted => Ready for checkin

--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:14>

[Django]

#35533: urlize() makes a bit of a mess of links embedded in Markdown
-------------------------------------+-------------------------------------
Reporter: Simon Willison | Owner:
| JaeHyuckSa

Type: Bug | Status: closed
Component: Utilities | Version: 5.1
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by Sarah Boyce <42296566+sarahboyce@…>):

* resolution: => fixed
* status: assigned => closed

Comment:

In [changeset:"a9fe98d5bd4212d069afe8316101984aadecfbb2" a9fe98d5]:
{{{#!CommitTicketReference repository=""
revision="a9fe98d5bd4212d069afe8316101984aadecfbb2"
Fixed #35533 -- Prevented urlize creating broken links given a markdown
link input.

Signed-off-by: SaJH <wogur...@gmail.com>
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/35533#comment:15>