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

Return the encrypted id-graphs when it's updated #842

Closed
Kailai-Wang opened this issue Sep 17, 2022 · 5 comments
Closed

Return the encrypted id-graphs when it's updated #842

Kailai-Wang opened this issue Sep 17, 2022 · 5 comments
Assignees
Labels
D1-feature A new feature, a piece of functionality that needs to be developed I2-medium should be completed within 10 working days

Comments

@Kailai-Wang
Copy link
Collaborator

Context

proposed by @kaylawangnow, we should return the full encrypted ID-graph when some identity is updated in the ID-graph, e.g. together with the IdentityVerified event.

It's mainly for display purposes.


✔️ Please set appropriate labels and assignees if applicable.

@Kailai-Wang Kailai-Wang added I2-medium should be completed within 10 working days D1-feature A new feature, a piece of functionality that needs to be developed labels Sep 17, 2022
@jingleizhang
Copy link
Contributor

jingleizhang commented Nov 9, 2022

needs:

- [ ] included in Parachain event, and encrypted by user_shielding_key
- [ ] to figure out which functionalities should include this feature, e.g, linking verification?

@Kailai-Wang
Copy link
Collaborator Author

I think link|unlink|verify_identity all can cause ID-graphs to change, thus all of them.

This is probably still meaningful even if the trusted port is exposed, where a user signature is required.

@jingleizhang
Copy link
Contributor

related issue #1010

If we decided to open a trusted RPC port, it's not necessary to include the user's id-graph in Parachain event.

@Kailai-Wang
Copy link
Collaborator Author

related issue litentry/litentry-parachain#1010

If we decided to open a trusted RPC port, it's not necessary to include the user's id-graph in Parachain event.

A second thought leads me to believe it might still be helpful. A trusted query needs user to sign the request while here the id graph is directly returned - better user experience

@Kailai-Wang
Copy link
Collaborator Author

closed via #1080

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D1-feature A new feature, a piece of functionality that needs to be developed I2-medium should be completed within 10 working days
Projects
None yet
Development

No branches or pull requests

4 participants