Added methods to store and fetch by identity #165
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We want to be able to store root Cid of encrypted hamt tree from WNFS in the backend. But it should be retrievable only with identiy (which is something we can recreate). We added two methods for store and get. One receives the encrypted versio of root Cid and returns the encrypted version and JAVA code must handle the encryption and decryption, and one receives a key and handles the decryption and encryption itself
we need to also provide appId on top of identity as the WNFS tree for each appID is different
|
A better approach might be having a link system like identity -> AppID -> encryptedAPPRootCID |
First try at 1- using current Put and Get 2- store it in an interconected link system, which right now is like: `identity -> CID(appID -> encryptedRootCID)`
|
Although it seems we have an error on JAVA side: |
We probabely need to change IPNS to DNSLink
|
I think IPNI only supports valid CIDs and we cannot announce identity-CID pair to it nda later query CID by identity? |
|
That's right identity cids by definition contain the data they represent. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We want to be able to store the root Cid of the encrypted hamt tree from WNFS in the backend. But it should be retrievable only with identity (which is something we can recreate) and app id (because each tree is unique for each app).
We added two methods for store and get. One receives the encrypted version of root Cid and returns the encrypted version and JAVA code must handle the encryption and decryption, and one receives an encryption key input as well and handles the decryption and encryption itself
Update 1: I tried to use the current Get and Put methods and also make it a link system like:
identity -> CID(appID -> encryptedRootCID)