code100x / tiplink

MIT License
174 stars 197 forks source link

Feat: added encryption of shares #157

Open lakshayvaishnav opened 2 months ago

lakshayvaishnav commented 2 months ago

Feat: added encryption of shares

I've added AES encryption to the shares generated during the secret splitting process. This ensures that even if someone gets their hands on the shares, they won't be able to access the original secret without the proper keys.

136

🛠️ Type of Change

✅ Checklist

To ensure a smooth review process, please check off each item as you complete it:


We appreciate your contributions! If you have any questions or need further assistance, feel free to reach out.

axatbhardwaj commented 2 months ago

Hi @lakshayvaishnav why are we not using IV over here ?

would like to know if it was a deliberate decision and if not then we need to use IV while encrypting

IV

cb7chaitanya commented 2 months ago

Also we have to handle three shares of the private key right now