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

URL escaped characters in json values break PUT operation #32

Closed
mooreds opened this issue Jan 24, 2013 · 3 comments
Closed

URL escaped characters in json values break PUT operation #32

mooreds opened this issue Jan 24, 2013 · 3 comments

Comments

@mooreds
Copy link
Contributor

@mooreds mooreds commented Jan 24, 2013

The error message I get when looking at a result is:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'LIKE '&price=%3C=300000&price=200001&lat=40.1',endpoint = 'test@example.com',las' at line 1 :: UPDATE userdb.saved_search SET api_where_clause LIKE '&price=%3C=300000&price=200001&lat=40.1',endpoint = 'test@example.com',last_run = '2013-01-10 10:00:00.0',search_name = 'home search lat price108492551',alert_type_id = 2 WHERE saved_search_id = 981

It looks like the update SQL parsing is getting confused by the % in %3C, which is actually a URL escaped > (or maybe <, I lose track).

@mooreds
Copy link
Contributor Author

@mooreds mooreds commented Jan 24, 2013

I have a test case that fails, but I can't seem to attach it.

@mooreds
Copy link
Contributor Author

@mooreds mooreds commented Jan 24, 2013

Here is a url to the test case, which I put in the json tests directory.

http://www.mooreds.com/scratch/TestUpdate_SingleTable_SingleRow_ByBody_Escaped_Chars.xml

@mooreds
Copy link
Contributor Author

@mooreds mooreds commented Jan 24, 2013

See restsql/restsql-test#2 as well.

@restsql restsql closed this in 08e6831 Mar 6, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants