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.

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

Batch query error message contains internal executeQueries in the response #4335

Closed
MichaelBurgess opened this issue Jul 17, 2024 · 0 comments · Fixed by #4337
Closed

Batch query error message contains internal executeQueries in the response #4335

MichaelBurgess opened this issue Jul 17, 2024 · 0 comments · Fixed by #4337
Assignees
@pskrbasu
Labels
bug Something isn't working

Comments

@MichaelBurgess
Copy link
Contributor

Describe the bug
Internal query execution step name returned in error message.

Steampipe version (steampipe -v)
Example: v0.23.2

To reproduce
Perform a query for a non-existent relation e.g. select * from aws_accountz;

steampipe query "select * from aws_accountz;"
Warning: executeQueries: query 1 of 1 failed: ERROR: relation "aws_accountz" does not exist (SQLSTATE 42P01)

executeQueries doesn't seem like it should be in user-facing messages?

Expected behavior
User-friendly message with no reference to executeQueries
@MichaelBurgess MichaelBurgess added the bug Something isn't working label Jul 17, 2024
@pskrbasu pskrbasu self-assigned this Jul 19, 2024
pskrbasu added a commit that referenced this issue Jul 19, 2024
@pskrbasu
Fix query error message to not include internal function name. Closes #…
d794857 
…4335
@pskrbasu pskrbasu linked a pull request Jul 19, 2024 that will close this issue
Fix query error message to not include internal function name. Closes #4335 #4337
Merged
pskrbasu added a commit that referenced this issue Aug 13, 2024
@pskrbasu
Fix query error message to not include internal function name. Closes #…
cb313fb 
…4335 (#4337)
pskrbasu added a commit that referenced this issue Aug 13, 2024
@pskrbasu
Fix query error message to not include internal function name. Closes #…
115522c 
…4335 (#4337)
@pskrbasu pskrbasu mentioned this issue Aug 13, 2024
Closed
5 tasks
@BrewTestBot BrewTestBot mentioned this issue Aug 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pskrbasu pskrbasu

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix query error message to not include internal function name. Closes #4335 turbot/steampipe
2 participants
@MichaelBurgess @pskrbasu