Closed akshayanandraut closed 2 years ago
@akshayanandraut Thank you for reporting this issue. It should be fixed in next release (probably this week).
Great Thanks man
On Wed, Oct 6, 2021, 2:43 AM Jan Lipovský @.***> wrote:
Closed #94 https://github.com/lipoja/URLExtract/issues/94.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/lipoja/URLExtract/issues/94#event-5413664563, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD62Y7YJCQ4PBEMMMTTAMV3UFNS7JANCNFSM5CTNHYQA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
trying to fetch all URLs from a html response and some scripts.src have values like
These type of links are not identified
In the html world, this format is quite common and is accepted.
I'm not sure if pre-processing the entire response to fix these links to a format identified by the urlextract lib is something I would do since that would mean that I parse through the entire response first correcting link formats and then use the find method which does the same. That would impact the performance on what I am doing.
Let's work on this as many on stackoverflow face similar issues when links start with a character before '//'