Sandro Santilli
Sandro Santilli
Actually, I see the hinting is done by cartodb-redis module, not sure why it doesn't appear in the CartoDB-SQL-API
See https://github.com/CartoDB/node-cartodb-redis/blob/v0.3.0/lib/carto_metadata.js#L81
It actually _is_ handling the error, but it looks like it's intentionally modifying it: https://github.com/CartoDB/CartoDB-SQL-API/blob/master/app/controllers/app.js#L300-L303 The modification was probably meant for the enduser, but in the log we'd still want...
I'd just review if the cartodb-redis module error is "safe" to show to the user (I think it likely is) and confirmed that just drop the err.message override (keeping the...
Maybe one way to deal with log vs. enduser message separation would be to add an optional "user_message" to the Error object thrown. The responder would then show that one...
The failure we'd currently get by simply skipping the override: ``` 1) app.test GET decent error if domain is incorrect: + expected - actual +Sorry, we can't find CartoDB user...
Another idea (probably simpler) would be to only override the error message IFF environment != "development"
NOTE: the regressing feature has been moved into the CDB-2600 branch ( https://github.com/CartoDB/CartoDB-SQL-API/tree/CDB-2600 ) -- the branch will need to be merged the new tests as soon as available, and...
Does this work ? ``` LC_ALL=C LANG=C make check ```
Changing server configuration is not an option. There must be a way to tell the server that the client wants to speak english. Do you still have a de-speaking server...