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

Escape ltree input #178

Open
jjeising opened this issue Jun 25, 2018 · 1 comment
Open

Escape ltree input #178

jjeising opened this issue Jun 25, 2018 · 1 comment

Comments

@jjeising
Copy link
Member

The ltree extension only supports A-Za-z0-9_. Other characters, even in conditions cause Syntax errors which cause exceptions. We need to filter input at least for search, probably also for property creation.

Or maybe just drop ltree and use strings? I'm currently not sure if we use any features.

@pegro
Copy link
Member

pegro commented Jun 25, 2018

We are not using ltree features as far as I can tell. In the API handler I only use the model and I didn't find any operators specific to ltree there. So I guess using strings is fine.
Maybe defining an own regex in a check contraint would be a good idea, e.g. for prohibiting spaces or equal signs, which might break things somewhere else.

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

No branches or pull requests

2 participants