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

Serialisation of BIP32 misplaced address postition #257

Closed
skywinder opened this issue Apr 4, 2020 · 16 comments · Fixed by #278
Closed

Serialisation of BIP32 misplaced address postition #257

skywinder opened this issue Apr 4, 2020 · 16 comments · Fixed by #278
Labels
bug Something isn't working 💸 gitcoin-bounty Earn money by helping with this issue!

Comments

@skywinder
Copy link
Collaborator

Look at the ending of testByBIP32keystoreSaveAndDeriva for details:

// This will fail. It wont fail if use scrypt from pod 'scrypt', '2.0', not from CryptoSwift
        XCTAssert(keystore?.addresses![0] == recreatedStore?.addresses![1])
        XCTAssert(keystore?.addresses![1] == recreatedStore?.addresses![0])

Adressed switched their position after serialisation (by native JSONEncoder().encode(params)
works fine when you run single test and mix them, when run. test suite.

@skywinder skywinder added the bug Something isn't working label Apr 4, 2020
@podkovyrin
Copy link
Contributor

I'm wondering why the assertion is to have [0] == [1] and [1] == [0]?

@skywinder
Copy link
Collaborator Author

@podkovyrin that's why I create this issue. Can't understand why they swaped.

@podkovyrin
Copy link
Contributor

I see. I came up with the following workaround-like solution so far: 1inch@13088a0
If you think that's acceptable I can make a PR.

@skywinder
Copy link
Collaborator Author

@podkovyrin Awesome!
I will also send you bounty un reward for fixing this! 👍

ps.
I see that you have another good fixes for library: I like that you extract "magic constants" from the code. I would be happy to merge PR too 👍
Just want to ask to split this to separate PR.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 0.0512 ETH (17.32 USD @ $338.24/ETH) attached to it.

@gitcoinbot
Copy link

@podkovyrin Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

1 similar comment
@gitcoinbot
Copy link

@podkovyrin Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@podkovyrin
Copy link
Contributor

@gitcoinbot yes, bot. I'm working

@skywinder
Copy link
Collaborator Author

@podkovyrin No worries and no rush. Code&chill👍

@gitcoinbot
Copy link

@podkovyrin Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

1 similar comment
@gitcoinbot
Copy link

@podkovyrin Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@gitcoinbot
Copy link

gitcoinbot commented Oct 19, 2020

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 3 days from now.
Please review their action plans below:

1) podkovyrin has been approved to start work.

Going to do git cherry-pick. Why I need to type 30 chars here???

Learn more on the Gitcoin Issue Details page.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


@podkovyrin due to inactivity, we have escalated this issue to Gitcoin's moderation team. Let us know if you believe this has been done in error!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

1 similar comment
@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


@podkovyrin due to inactivity, we have escalated this issue to Gitcoin's moderation team. Let us know if you believe this has been done in error!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@skywinder
Copy link
Collaborator Author

@podkovyrin thank you for your outstanding work! Since you already made PR, Please submit for this bounty. I would be happy to reward you!
Happy New Year! 🎉

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 0.1 ETH (74.25 USD @ $754.31/ETH) has been submitted by:


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 💸 gitcoin-bounty Earn money by helping with this issue!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants