-
Notifications
You must be signed in to change notification settings - Fork 2.5k
SqlString.escapeId and arrays #524
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
Comments
No one needed before. I'll accept a pull request for it, maybe similar to what |
Yup, I was thinking of something along those lines, though initially I didn't think it would need to support objects, just arrays and strings. Will look into it, thanks! |
JS arrays are converted to SQL tuples (by SqlString.escape()), no? Reproducing that, the above would produce: SELECT ( |
If CLIENT_PLUGIN_AUTH capability is not supported, no new cipher is sent have to keep using the cipher sent in the init packet. Fixes mysqljs#518
I've noticed that SqlString.escapeId does not work with arrays. I'd like to do something like:
to produce:
Is there a specific reason for this, or has the need just not arisen previously?
The text was updated successfully, but these errors were encountered: