loopbackio / loopback-connector-redis

EXPERIMENTAL: Redis connector for LoopBack.
http://loopback.io/doc/en/lb2/Redis-connector.html
Other
36 stars 46 forks source link

handle ObjectId from mongodb #18

Closed makeros closed 8 years ago

makeros commented 8 years ago

When using as datasource for User mongodb and for accessToken redis then there is an issue when we want to store the accessToken and we got the principalId as ObjectId from mongodb directly. The id is then stored with additional quotes.

slnode commented 8 years ago

Can one of the admins verify this patch? To accept patch and trigger a build add comment ".ok\W+to\W+test."

devntd commented 8 years ago

I tried to follow the instructions but still valuable userId is "NaN"

slnode commented 8 years ago

Can one of the admins verify this patch?

superkhau commented 8 years ago

Can you add some tests to verify your changes and prevent regressions in the future?

superkhau commented 8 years ago

Closing due to inactivity. If you are still running into problems, feel free to leave a comment and I will reopen the issue.

evseevnn-zz commented 8 years ago

+1 i have same problem

superkhau commented 8 years ago

@evseevnn Would you like to submit another PR based on this PR with some tests? We can get it landed quicker that way.