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.

Sign up for GitHub

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

Jump to bottom

feat: add basic escaping to literals and identifiers #4

Merged

danstarns merged 2 commits into main from feat/utils-sql-escape

Jan 18, 2025

Collaborator

danstarns commented Jan 17, 2025

Related:

Need to sanitize strings from user input #1

This PR initialises the @timescaledb/utils package that contains extracts from the node-postgres lib:

https://github.com/brianc/node-postgres/blob/master/packages/pg/lib/utils.js

SQL generation and escaping will change over time as we converge on the examples, but this gives us a tested platform to iterate on.

Thanks for the pointer @murrayju

danstarns added 2 commits

January 17, 2025 15:53


          feat: add basic escaping to literals and identifiers

d661f18


          feat: check name of hypertable

0119fa4

danstarns mentioned this pull request

Need to sanitize strings from user input #1

Closed

danstarns merged commit ac3db69 into main

2 checks passed

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

Labels

None yet