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

Solve escaping issues with difficult py file commands #12

Open
komlenic opened this issue Dec 1, 2015 · 1 comment
Open

Solve escaping issues with difficult py file commands #12

komlenic opened this issue Dec 1, 2015 · 1 comment

Comments

@komlenic
Copy link
Owner

komlenic commented Dec 1, 2015

In particular, see node.drush_sql() which is only necessary to accommodate difficult sql commands that must be able to execute correctly locally and remotely (over ssh).

The present drush_sql command applies different levels of escaping to a query, and does appear to work, but this should be better understood and accommodated if possible.

Complicating this issue is a chain of escaping that goes through: python, fabric, shell, and drush.

@komlenic
Copy link
Owner Author

komlenic commented Dec 1, 2015

The present handling of this requires that the string literal passed to drush_sql() must be triple single quoted, and have all single quotes escaped with . Double quotes within the string literal should be passed unmodified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant