Safari - Invalid Regular Expression - Invalid Group Specifier Names #197
Comments
|
Thanks for reporting this @CodesAreHonest. I'll work on getting a fix out for this very soon. I'll post back with an update once I have a new release published to NPM that fixes this. |
|
Appreciate your response on this issue, looking forward for the fixes and thanks for the awesome library ! |
mrodrig
added a commit
to mrodrig/doc-path
that referenced
this issue
Jun 13, 2021
As reported in mrodrig/json-2-csv#197, many browsers don't support negative look behind RegExps, which causes both calls to this module and dependency modules to fail when that logic was invoked. In order to fix that, this commit reworks the RegExp to be use a string character parser to find first index where a non-escaped '.' character is found.
mrodrig
added a commit
to mrodrig/doc-path
that referenced
this issue
Jun 13, 2021
As reported in mrodrig/json-2-csv#197, many browsers don't support negative look behind RegExps, which causes both calls to this module and dependency modules to fail when that logic was invoked. In order to fix that, this commit reworks the RegExp to be use a string character parser to find first index where a non-escaped '.' character is found.
mrodrig
added a commit
to mrodrig/doc-path
that referenced
this issue
Jun 13, 2021
* Remove negative lookbehind regexp logic. As reported in mrodrig/json-2-csv#197, many browsers don't support negative look behind RegExps, which causes both calls to this module and dependency modules to fail when that logic was invoked. In order to fix that, this commit reworks the RegExp to be use a string character parser to find first index where a non-escaped '.' character is found. * chore(release): 3.0.1
mrodrig
added a commit
that referenced
this issue
Jun 13, 2021
As reported in #197, there was a compatibility issue introduced in doc-path@3.0.0 where a negative lookbehind Regexp was being used despite not being supported by many browsers. This caused runtime errors when this library's functionality was called due to the underlying dependency compatibility issue. This commit bumps the version to 3.0.1 which fixes the compatibility issue in the doc-path module. Fixes #197
1 task
mrodrig
added a commit
that referenced
this issue
Jun 13, 2021
* Update doc-path to fix negative lookbehind regexp. As reported in #197, there was a compatibility issue introduced in doc-path@3.0.0 where a negative lookbehind Regexp was being used despite not being supported by many browsers. This caused runtime errors when this library's functionality was called due to the underlying dependency compatibility issue. This commit bumps the version to 3.0.1 which fixes the compatibility issue in the doc-path module. Fixes #197 * chore(release): 3.14.1
|
Thanks again for reporting this @CodesAreHonest. The fixed version should be up on NPM now as version |
|
My pleasure to help, the latest version work like charm in the Safari environment. Much appreciated. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Background Information
3.14.016.1.0The issue I'm reporting is with:
I have...
Expected Behavior
Actual Behavior
The browser complains the error (refer screenshot above) due to Lookbehind Regex does not support in Safari.
Discussion
The library use
doc-pathas dependencies.Refer this line, the function
stateuse Negative Lookbehind Regex to performexecleads to the mentioned errors and break in the browser.The text was updated successfully, but these errors were encountered: