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.

Sign up for GitHub

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

Jump to bottom

False alarm detecting URLs #36

Open

LeoSarabi opened this issue Jan 4, 2023 · 1 comment

LeoSarabi commented Jan 4, 2023

if i have a text contains 10.00hr, it is consider as a URL

runTest("10.00hr,", UrlDetectorOptions.Default);
it should return empty, but the results is [http://10.00hr]

yl-ang commented Jan 18, 2023

Valid but readme stated that it is better to over detect URL for security reasons.

Note: Keep in mind that for security purposes, its better to overdetect urls and check more against blacklists than to not detect a url that was submitted...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment