> I think if we find more than one question mark we should just assume that's a bad url and not do any further comparisons.
Huh? The URLs cited in the comment occur routinely in MailNews. Sure, we can clean this up one day. This bug is about detecting the patch correctly. And we do this already in IMAP, hence I copied those four lines. Do I need to find a different reviewer?
Bug 1572864 Comment 39 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
> I think if we find more than one question mark we should just assume that's a bad url and not do any further comparisons.
Huh? The URLs cited in the comment occur routinely in MailNews. Sure, we can clean this up one day. This bug is about detecting the path correctly. And we do this already in IMAP, hence I copied those four lines. Do I need to find a different reviewer?