Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Need better error message when appserver not found in execute_query #360

Closed
paxtonhare opened this issue Jan 30, 2015 · 2 comments
Closed

Need better error message when appserver not found in execute_query #360

paxtonhare opened this issue Jan 30, 2015 · 2 comments
Labels
deployer enhancement
Milestone
July 2017

Comments

@paxtonhare
Copy link
Contributor

execute_query can take an appserver as a parameter. properties[:app_name]

if the name provided is not a valid appserver name Roxy will fall back on any old database. Instead, it should throw an error telling the user that the appserver name they provided is not valid. Having just debugged this for 20 minutes I think it will save folks time.
@grtjn
Copy link
Contributor

grtjn commented Jan 30, 2015 •
edited
Loading

+1 :-)

@grtjn
Copy link
Contributor

grtjn commented Jun 14, 2017

I added clearer messages as part of the fix for #661 (see PR #768). Fixed in MASTER

@grtjn grtjn closed this as completed Jun 14, 2017
@grtjn grtjn self-assigned this Jun 14, 2017
@grtjn grtjn added this to the July 2017 milestone Jun 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
deployer enhancement
Projects
None yet
Milestone
July 2017
Development

No branches or pull requests
2 participants
@grtjn @paxtonhare