Signal-android: URLs in messages fail to include the trailing slash

[idupree]

I have:

• [x] searched open and closed issues for duplicates
• [x] read https://github.com/WhisperSystems/Signal-Android/wiki/Submitting-useful-bug-reports

---

Bug description

URLs in messages fail to include the trailing slash. (In contrast, Signal Desktop does include the slash.)

Steps to reproduce

• Send someone the message Test link: https://www.example.com/foo/bar/ Yay!
• look at the link; click the link
• notice that Signal treated the trailing slash as not part of the link

Actual result: Signal treated the trailing slash as not part of the link
Expected result: Signal should underline the trailing slash and include it in the URL when you click it

Screenshots

Device info

Device: Google Nexus 5X
Android version: 7.0
Signal version: 3.28.4

Link to debug log

[2-4601]

Duplicate of #6019? (cc @nikwen)

[nikwen]

@2-4601 Will check that when I have a few minutes.

[nikwen]

Had a look at this. It does in fact look like a duplicate of #6019 to me.

It is notable that this only happens if the message contains some text after the link:

@idupree commented on how one could fix this manually here, but I do not think that this is a critical issue (as the link still works) and believe that this is something that should be fixed in Linkify instead of adding workaround code to every app out there using it (remember, it's part of Android).

[idupree]

To be clear, I noticed this because a site had a page which was broken without the trailing slash. (This isn't very good web practice, but it happens sometimes.) I don't disagree about this issue being noncritical.

[2-4601]

Closing as duplicate of #6019.