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

Update in multi-user account is not being recognized #2743

Closed
iriberri opened this issue Mar 12, 2015 · 4 comments
Closed

Update in multi-user account is not being recognized #2743

iriberri opened this issue Mar 12, 2015 · 4 comments
Milestone

Comments

@iriberri
Copy link
Contributor

update example SET latitude = latitude / 1000000

is transformed to:

update "username".example SET latitude = latitude / 1000000

and:

image

From: SB/4754950

@iriberri
Copy link
Contributor Author

And Explain Analyze :-) From SB/4764650

@iriberri iriberri added this to the ElSoto milestone Mar 12, 2015
@Xatpy Xatpy assigned Xatpy and unassigned javisantana Mar 12, 2015
@Xatpy
Copy link
Contributor

Xatpy commented Mar 13, 2015

It only fails with users that has an "-" in his username.
In my local environment, I tried with user xatpy and it worked. But I created an user called "ey-tu", and it fails.
Kudos to @iriberri , because she pointed it. 👏

@Xatpy
Copy link
Contributor

Xatpy commented Mar 13, 2015

@javisantana I think that the error could be here

But my question is: this would be an error from cartodb or CartoDB-SQL-API ??
I mean: I don't know if the error here is that frontend is sending data wrongly, or cartodb-sql-api should accept that kind of data. I think this because of the error stack:

{
  error: [
    "syntax error at or near ".""
  ],
  stack: "error: syntax error at or near "." at Connection.parseE (/CartoDB-SQL-API/node_modules/cartodb-psql/node_modules/pg/lib/connection.js:566:11) at Connection.parseMessage (/CartoDB-SQL-API/node_modules/cartodb-psql/node_modules/pg/lib/connection.js:395:17) at null.<anonymous> (/CartoDB-SQL-API/node_modules/cartodb-psql/node_modules/pg/lib/connection.js:104:20) at Socket.EventEmitter.emit (events.js:95:17) at Socket.<anonymous> (_stream_readable.js:746:14) at Socket.EventEmitter.emit (events.js:92:17) at emitReadable_ (_stream_readable.js:408:10) at emitReadable (_stream_readable.js:404:5) at readableAddChunk (_stream_readable.js:165:9) at Socket.Readable.push (_stream_readable.js:127:10)"
}

Also, I've been "playing" with several URL (without - or ., encoding, etc.) but I couldn't find a correct request with a username who contains -.

@Xatpy
Copy link
Contributor

Xatpy commented Mar 18, 2015

Closed with commit 057904a

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

3 participants