Redis parameters

[danitseitlin]

Redis parameters

[danitseitlin]

@FallingSnow Hi, I am looking on https://oss.redislabs.com/redisgraph/commands/#graphquery and It doesn't seem aligned with what you shared. I will try to check what should be the correct syntax and update the code accordingly.
Thanks for your issue.

[danitseitlin]

@FallingSnow Can you share your exact usage?

[FallingSnow]

I'm trying to escape user input in queries I submit to the database. Some libraries provide this escape function. Considering redis returns Missing parameters I think it might be native to redis.

Query:

MATCH (p:user) WHERE p.email=$email RETURN count(p) as count

Response:

Error: RedisGraph class (GRAPH.QUERY): ReplyError: Missing parameters

[danitseitlin]

@FallingSnow Can you share your usage with the redis-modules-sdk?

[FallingSnow]

That is.

export default class Database extends RedisGraph {
  constructor(...args) {
    super({ port: options.databasePort, host: options.databaseHost, ...args });
  }

  async query(query: string) {
    try {
     // Throws "Error: RedisGraph class (GRAPH.QUERY): ReplyError: Missing parameters"
      return await super.query(options.databaseName, query);
    } catch (error) {
      log.error("Failed to run database query: \"%s\"\n%s", query, error);
      throw new Error("Database error. See server logs.");
    }
  }

  static escape(str) {
    return str.replace(/'/g, "\\'");
  }
}

// Doesn't work
const matches = await database.query(`MATCH (p:user) WHERE p.email=$email RETURN count(p) as count`);

// Works
const matches = await database.query(`MATCH (p:user) WHERE p.email='[email protected]' RETURN count(p) as count`);

[FallingSnow]

Are you unable to replicate the error with a query that has a $ (dollar sign) variable?

[danitseitlin]

@FallingSnow hi, yes i am able. Sorry for the delay i will be working on this and many issues this weekend and next week.

[danitseitlin]

@FallingSnow Hi, should have a fix soon, just running some last tests.

[FallingSnow]

Great! Thank you!