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

autocomplete does not quote table names that need to be quoted #3065

Closed
johnsmyth opened this issue Feb 2, 2023 · 0 comments · Fixed by #3092
Closed

autocomplete does not quote table names that need to be quoted #3065

johnsmyth opened this issue Feb 2, 2023 · 0 comments · Fixed by #3092
Assignees
Labels
bug Something isn't working

Comments

@johnsmyth
Copy link

Describe the bug
Some table names (especially for dynamic tables) need to be quoted because they contain characters that are not legal postgres identifiers. These appear in the autocomplete list , but they are not quoted when you select them so the auto-completed query does not actually work.

Steampipe version (steampipe -v)
All versions is assume, but definitely 0.18

To reproduce
You can reproduce this by adding a table name that requires quoting, then trying to use autocomplete. The CSV plugin is useful for testing this, as is will use the file name as the table name. Create a file like MyFile.csv, then use the autocomplete to create a query - the table name is not quoted:

> select * from MyFile
Error: relation "myfile" does not exist (SQLSTATE 42P01)

Expected behavior
table names that require quoting should be quoted when added by autocomplete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants