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

Get command does not only uses local cache #15

Open
Arkaeriit opened this issue Dec 14, 2021 · 3 comments
Open

Get command does not only uses local cache #15

Arkaeriit opened this issue Dec 14, 2021 · 3 comments
Labels
documentation Improvements or additions to documentation

Comments

@Arkaeriit
Copy link

In the readme it is mentioned that using skate sync only reads data from the local cache. But when I do a skate get XXX, a call to my remote charm server is made.

I do not know if this is a bug or if the readme should be changed.

@toby
Copy link
Contributor

toby commented Dec 14, 2021

Hi, the data is only synced on write, but on read we're currently making a network call to fetch your encryption keys. The keys are fully encrypted for your ssh public key before being stored on our server, then decrypted on fetch and used to decrypt the BadgerDB. You're correct that the documentation should be clearer though! We also plan to cache the encryption keys locally to make skate and charm kv work in a truly offline, read-only mode. For now, we'll update the docs. Thanks for the catch!

@toby toby added the documentation Improvements or additions to documentation label Dec 14, 2021
@sheldonhull
Copy link

I had to stop using a way to store some local tokens because it introduced seconds of delays to my profile load when pulling several entries. I hope this local caching becomes an option eventually.

@dezren39
Copy link

yes this is a big blocker to usage anywhere other than on a lan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

4 participants