-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
Invalid regular expression on Japanese site #65
Comments
I don't see any problems. Your issue can be caused by improper encoding and so on - not related to this project. Could you provide a reproducible samples via http://markdown-it.github.io/linkify-it/ ? |
Sorry, didn't get what do you mean by "improper encoding" in this case. I've only discovered that error happens in this project code while constructing some RegExps dependent on site location which brokes library relied on you. |
Please, remove everything, not related to this project to leave minimal possible reproducible sample. That will be a good proof, that problem is really related to this package. Could you do that? The ideal way would be to reproduce problem in http://markdown-it.github.io/linkify-it/. |
Ok, I'll try. Do you have a CDN version of the library? I can't find it. Because I must add it onto the website to which I haven't access. |
https://unpkg.com/ everything pulished in npm is available on cdn |
Yeah, I saw this. But you haven't UMD section. It's impossible just add this js file to the browser. |
Ah, ok. You could use node.js then. This package is platform-agnostic. Browser is not mandatory to create sample. |
Close - timed out with no details. |
@Amethystafyy282 see #15, i think it's the same or very similar to your case. If you have idea how to formalize condition, that will help. Need someone with experience of "asian groop languages" to discuss. |
As I can see, the different is those are not unicode characters(except for the link) and some asian languages doesn`t has any spaces between words. |
@Amethystafyy282 you are right. Some languages not use spaces. Problem is, how to define condition without false positives. For example, is it possible to have links with hieroglyphs in real world? How to define border then? I suggest to continue in #15 if you have ideas. It's still open to collect useful info. |
Hi!
I used react-linkify which rely on your module.
And I faced with a critical error on one of japanise sites. It was real cause of error because bug reproduces only on this website and was totally okay on english ones.
It happens in RegExp construction at
linkify-it/index.js
Line 170 in 3abf523
the error itself
please note
Unmatched ')'
.I suppose it happen somewhere here
linkify-it/lib/re.js
Line 168 in 3abf523
Feel free to contact me if you need additional info.
The text was updated successfully, but these errors were encountered: