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

gun.get('foo').get('bar').put(sea key) store sea key bug #783

Open
Lightnet opened this issue Jul 30, 2019 · 4 comments
Open

gun.get('foo').get('bar').put(sea key) store sea key bug #783

Lightnet opened this issue Jul 30, 2019 · 4 comments

Comments

@Lightnet
Copy link

@Lightnet Lightnet commented Jul 30, 2019

Bug found I guess.

gun.get('foo').get('bar').put(sea key)

This code should but it need to string to work. As for user.get('foo').put(sea key). Which works without need string but json format.

gun.get('foo').get('bar').once()

Will return null or stop working. Notice by amark.

@amark

This comment has been minimized.

Copy link
Owner

@amark amark commented Aug 27, 2019

@Lightnet thanks for the report. I think I also saw this with @QVDev the other day when trying to encrypt video frames. Will need to get this fixed.

@QVDev

This comment has been minimized.

Copy link

@QVDev QVDev commented Aug 28, 2019

@amark

This comment has been minimized.

Copy link
Owner

@amark amark commented Jan 9, 2020

people terribly working around this by saving .put('a'+enc) then on a read, doing enc = enc.slice(1) this is a terrible bug that needs to be fixed, I'm just hammered dealing with HackerNoon etc. (which is public data, not encrypted/private). LOVE IT IF SOMEBODY COULD FIX.

@pszabop

This comment has been minimized.

Copy link

@pszabop pszabop commented Jan 10, 2020

I just did ).put(JSON.stringify(value)) and it works as long as you JSON.parse on subsequent gets . That way you can put anything in including an Object and it'll just work around this and any other potential string parsing issues. If you are paranoid base64 encoded after JSON.stringify()

I do this with lots of databases for similar reasons.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.