[Bug] Timestamps in YouTube descriptions not working

[ermac0]

Checklist (Your issue will be automatically closed if you delete this part)

• I make sure that there are no existing issues - open or closed - which I could contribute my information to.
• I understand that issues with limited impact, such as those occurring on specific devices or under specific network conditions, will not be fixed.
• I am able to reproduce the bug with the latest version given here: CLICK THIS LINK.
• I have attached the error report in the issue.

Describe the bug
In version 4.4.2 timestamps in the YouTube descriptions not displayed as links and cannot be clicked on.
Worked fine in the version prior to 4.4.2

Frequency
Always

Steps to reproduce the bug
Open a video such as
https://m.youtube.com/watch?v=C5lWtpkJuzs
Timestamps work when opened in the browser.

Device (e.g. Pixel 9 Pro)
OnePlus 7 Pro

Error report

Additional context
The timestamps start with 0:00 which is permissable by YouTube.

[InfinityLoop1308]

It works fine here with the video you post.
Maybe you can post a screen recording.

[ermac0]

Here's a link to the video https://jumpshare.com/s/IPBuEDnQKTeh19ZNXvu2
It's not just on this channel that the links don't work.

Let me know if you need more info.

[InfinityLoop1308]

No idea what's happening... Maybe you can try if it works with a fresh install.

[ermac0]

It seems that the new Pattern introduced in commit isn't detecting timestamps for me. Is it possible to fall back onto the old pattern if no matches are found, or make it configurable which one to use?

Weird that it's working for you, I wonder if that was a bug in Android libs. I'm on android 9.

[InfinityLoop1308]

    private void linkify() {
        LinkifyCompat.addLinks(itemContentView, Linkify.WEB_URLS);
        LinkifyCompat.addLinks(itemContentView, TimestampExtractor.TIMESTAMPS_PATTERN, null, null,
                (match, url) -> {
                    try { //here url is like 15:38 or 01:00:00
                        final var timestampMatch = TimestampExtractor
                                .getTimestampFromMatcher(match, commentText);
                        if (timestampMatch == null) {
                            return url;
                        }
                        return "internal://timestamp/" + timestampMatch.seconds();
                    } catch (final Exception ex) {
                        Log.e(TAG, "Unable to process url='" + url + "' as timestampLink", ex);
                        return url;
                    }
                });
    }

if adding a fallback pattern it will conflict with the existed pattern which works for most users.

[InfinityLoop1308]

Close since it is device specific and no workaround.