Skip to content

fix SQL command syntax in Application Setup command #23

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

Merged
merged 1 commit into from
Feb 10, 2019
Merged

fix SQL command syntax in Application Setup command #23

merged 1 commit into from
Feb 10, 2019

Conversation

JNavith
Copy link
Contributor

@JNavith JNavith commented Feb 10, 2019

linuxserver.io

This fixes a SQL syntax error experienced when using the LinuxServer MariaDB container.
Here is an example that shows both the error and that it is fixed by the introduction of ' around %) (I am using hass in place of myuser because I'm using it for Home Assistant -- but that's irrelevant).

@aptalca aptalca merged commit df258f8 into linuxserver:master Feb 10, 2019
@JNavith JNavith deleted the application-setup-command-fix branch February 10, 2019 06:34
@JNavith
Copy link
Contributor Author

JNavith commented Feb 11, 2019

The CI bot undid the changes in bdb1f93.
Was there somewhere else I should've made the change instead/in addition?

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

Successfully merging this pull request may close these issues.

2 participants