-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
//www.google.com cannot find such type of links #94
Comments
trying to fetch all URLs from a html response and some scripts.src have values like <script src="//www.google.com/somejsfile.js"> 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 '//' |
@akshayanandraut Thank you for reporting this issue. |
Great
Thanks man
…On Wed, Oct 6, 2021, 2:43 AM Jan Lipovský ***@***.***> wrote:
Closed #94 <#94>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#94 (comment)>, 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>.
|
No description provided.
The text was updated successfully, but these errors were encountered: