Skip to content
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 flags supplied to RegExp constructor 'u' #156

Closed
vigneswaranm opened this issue May 9, 2017 · 8 comments
Closed

Invalid flags supplied to RegExp constructor 'u' #156

vigneswaranm opened this issue May 9, 2017 · 8 comments
Labels

Comments

@vigneswaranm
Copy link

We are getting the issue "Invalid flags supplied to RegExp constructor 'u'" in lib/XMLStringifier.js:140

in the below line
str.match(/[\u0000\ud800-\udfff\ufffe-\uffff]/u);

The /u modifier in PHP is for unicode support. This modifier is not supported in JavaScript.

@EDais
Copy link

EDais commented May 10, 2017

Had the same issue here, updating to latest LTS version of node.js seemed to do the trick

@vigneswaranm
Copy link
Author

i have tested in 4.2.2 version but still, it failed which version did you use

@EDais
Copy link

EDais commented May 10, 2017

Still getting the error here on version 4.4.7, it's working on the current LTS version of 6.10.3

@oozcitak
Copy link
Owner

node.js 6.0 is required starting from xmlbuilder v9.

@ds82
Copy link

ds82 commented May 18, 2017

Please consider to support all node LTS versions as long as they are in maintenance. Humans are bad at semver. Dropping node4 support broke our builds, because another library author just depended on xmlbuilder >= 2 in her library. I know thats not your fault, but if important packages like yours would at least not drop support for active/in maintenance node LTS versions, it would make life a lot easier ..

@oozcitak
Copy link
Owner

oozcitak commented May 18, 2017

Fair point. I'll see what I can do.

@oozcitak oozcitak reopened this May 18, 2017
@formspoint
Copy link

Just upgraded and ran into this issue. Please consider support for all LTS versions.

@tregagnon
Copy link
Contributor

PR #161 should restore the compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants