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

Question | SQL injection #2

Closed
Nirlah opened this issue Jan 23, 2015 · 4 comments
Closed

Question | SQL injection #2

Nirlah opened this issue Jan 23, 2015 · 4 comments
Assignees
Labels

Comments

@Nirlah
Copy link

Nirlah commented Jan 23, 2015

Hi,

What is the preferred method to prevent SQL injections using this library?

Thanks 😀

@Nirlah Nirlah changed the title Question | sql injection Question | SQL injection Jan 23, 2015
@Ostico Ostico self-assigned this Jan 24, 2015
@Ostico
Copy link
Collaborator

Ostico commented Feb 4, 2015

In the next version of OrientDB it should be present.

@cqcwillard
Copy link

I believe OrientDB only supports client side prepared statements at the moment, is this correct? The NodeJS library appears to be encoding the values on the client side as well.

We're using 2.1.

References:
https://github.com/codemix/oriento/blob/master/lib/transport/binary/protocol28/operations/command.js
https://github.com/orientechnologies/orientdb/blob/master/core/src/main/java/com/orientechnologies/orient/core/sql/OCommandExecutorSQLSelect.java

@Ostico
Copy link
Collaborator

Ostico commented May 13, 2015

Tes @cqcwillard , you're right.

There are no way to perform prepared statement and server side sanitization like PDO and MySQL does.

I'm planning to add a PDO like interface to this driver and implement client side the prepared statement object.

@TACIXAT
Copy link

TACIXAT commented Jun 3, 2017

I was really excited to try out OrientDB. This is pretty much a non-starter for me though. Security features are super important, they should be supported server side and not reimplemented in each client.

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

4 participants