Table data cell SQL should escape columns

observablehq / feedback

Customer submitted bugs and feature requests

42 stars 3 forks source link

Closed domoritz closed 2 years ago

domoritz commented 2 years ago

In https://observablehq.com/d/4eae70d9aa34c86d, the table data cell doesn't escape a column with a - in it so the SQL query is invalid.

mbostock commented 2 years ago

It looks like the DuckDBClient doesn’t implement escape:

If you implement that, it should work. If it doesn’t, it is a bug on our end. Let us know!

domoritz commented 2 years ago

I missed that. Implemented and now it works as expected. Thank you!