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

Creation of wallet #32

Open
t-izbassar opened this issue Jul 12, 2018 · 5 comments
Open

Creation of wallet #32

t-izbassar opened this issue Jul 12, 2018 · 5 comments

Comments

@t-izbassar
Copy link
Contributor

The Wallets interface has a method to create a new Wallet instance. In order to create Wallet we need header information:

  • wallet id
  • network name
  • protocol version
  • public RSA key.

I'm not sure, that the Wallets.create() should accept that information through parameters. Who should maintain the ids? From where we will receive it? Through the constructor of WalletsIn()?

@0crat
Copy link
Collaborator

0crat commented Jul 12, 2018

@llorllale/z please, pay attention to this issue

@llorllale
Copy link
Collaborator

@t-izbassar

  • id: My thoughts for now are to make this library generate this randomly and internally somehow.
  • network name and protocol version: I just started asking about network name at Why does a Wallet have a 'network'? papers#44. The X-Zold-Protocol header mentioned in RESTful API is linked to the name. Thus it would seem like the wallet's network name is tied to whichever node it's first pushed to. I think more clarification is needed.
  • RSA key: Wallets.create should accept the RSA key as parameter, to be provided by the user. After all, wallets may or may not share the same RSA key.

@llorllale
Copy link
Collaborator

@t-izbassar can we close this?

@t-izbassar
Copy link
Contributor Author

@llorllale can you provide links on documentation or commits that are improving the understanding of the project?

@llorllale
Copy link
Collaborator

@t-izbassar this is all still a work-in-progress (even the spec itself).

wallet id and protocol version are implementation details that will won't be included in docs.

RSA key will definitely be provided by the user, and it looks like network name will be as well.

This is the documentation we have for now, in this ticket, in the README, in the whitepaper, and in the zold project.

It'll have to do for now.

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

No branches or pull requests

3 participants