-
Notifications
You must be signed in to change notification settings - Fork 48
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
replace json-glib with jansson #50
Comments
This is now high prio, moving to 1.1.0. The export feature has been postponed. |
An alternative would be to store the secret encrypted and then it will be decrypted only when needed. This, of course, will add an overhead, especially if a lot of tokens have to be updated every 30s... |
I'm currently rewriting the code to use jansson. Using this library instead of json-glib gives also more flexibility and helps to reduce the amount of code :) I will also take this opportunity to carefully review all the memory usage and allocation 👍 |
porting done, testing it right now |
jansson supports custom memory allocation, meaning that
gcry_calloc
could be used to allocate secure memory. Right now, the data parsed by json-glib is not stored in a secure memory location.http:https://jansson.readthedocs.io/en/latest/apiref.html#custom-memory-allocation
The text was updated successfully, but these errors were encountered: