-
Notifications
You must be signed in to change notification settings - Fork 358
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
valid URL containing an underscore cannot be added #206
Comments
Yeah, URL validation is not perfect, and from what I understand this is quite opinionated. For now linkding just uses the default Djanog URL validator, so you could also report it there. As a workaround linkding provides an option to disable URL validation: https://github.com/sissbruecker/linkding/blob/master/docs/Options.md#ld_disable_url_validation |
Django won't fix, since 10 years |
I'm here one year later trying linkding again (thanks to Browser Companion app for iOS/macOS) and these two URLs failed to import again, not that I expected it to have changed.
I was able to add them as wayback machine versions. eg 😄 |
Here's another one:
The above link has since been broken (quick google) it's now at: https://github.com/sissbruecker/linkding/blob/master/docs/src/content/docs/options.md |
Switch to use https://github.com/python-validators/validators/ can be the trick, support very well RFC and can be used to expand in future to support another types of url like custom schema and local uri using feature-flag. |
Why not disable the validation to keep using it? |
One important point: this is not fully the same as #957 because the point about the another one is not validation is similarity support to avoid more wide duplicate than strict equal-only check. |
I simply do not want to turn off validation, though I might change my mind about that in the future. |
From #202
Confirming a valid URL containing an _ (underscore) cannot be added to the system
Screenshot
Example URLs
Bug
So there is a problem with the process of checking a URLs, as both of the above are valid.
The text was updated successfully, but these errors were encountered: