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

wallet-core: Remove u64 cast when generating the seed #2217

Closed
wants to merge 2 commits into from

Conversation

moCello
Copy link
Member

@moCello moCello commented Aug 27, 2024

Resolves #2209

@moCello moCello force-pushed the mocello/2209_keys_u8 branch from 6581635 to 5df6112 Compare August 27, 2024 15:59
@moCello
Copy link
Member Author

moCello commented Aug 28, 2024

Note: Since this is a very breaking change, please only merge with explicit approval by all involved parties.

Copy link
Member

@herr-seppia herr-seppia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As previously discussed, I am against this change because people would not be able to use the same seed used during ITN (and therefore they would not be able to claim their rewards)

I'm curious to know what's the benefit of this (apart from code clarity)

I'll put a "conservative" RFC while waiting for someone that change my mind

@HDauven
Copy link
Member

HDauven commented Aug 28, 2024

As previously discussed, I am against this change because people would not be able to use the same seed used during ITN (and therefore they would not be able to claim their rewards)

I'm curious to know what's the benefit of this (apart from code clarity)

I'll put a "conservative" RFC while waiting for someone that change my mind

Why can't we create a claim contract, whereby they sign they're the owner for a given key, but get the funds on a newly generated key?

We would need to put in some extra work and still have the old way of generating keys available in some library, but it's not like it would be impossible to do this, right?

@herr-seppia
Copy link
Member

whereby they sign they're the owner for a given key,

And how could they prove it if they are not able to derive the old secret key anymore?

@autholykos
Copy link
Member

Closing as we no longer need to change key generation

@autholykos autholykos closed this Sep 3, 2024
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

Successfully merging this pull request may close these issues.

wallet-core: Remove the cast to u64 when generating the seed for the keys
4 participants