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

Support Bech32M, Segwit v1, BIP350 #1345

Merged
merged 3 commits into from
Apr 26, 2021
Merged

Support Bech32M, Segwit v1, BIP350 #1345

merged 3 commits into from
Apr 26, 2021

Conversation

optout21
Copy link
Contributor

Description

Support BIP350, New Bech32M checksum for segwit addresses with version >= 1. Fixes #1331 .

Testing instructions

Unit tests (Bech32, Segwit)

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • I have read the guidelines for adding a new blockchain
  • If there is a related Issue, mention it in the description (e.g. Fixes #<issue_number> ).

@codecov
Copy link

codecov bot commented Mar 29, 2021

Codecov Report

Merging #1345 (3a9cdf5) into master (0f2c369) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 3a9cdf5 differs from pull request most recent head e9f957d. Consider uploading reports for the commit e9f957d to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1345      +/-   ##
==========================================
- Coverage   95.41%   95.40%   -0.01%     
==========================================
  Files         443      443              
  Lines       13401    13376      -25     
==========================================
- Hits        12786    12762      -24     
+ Misses        615      614       -1     
Impacted Files Coverage Δ
src/Bech32.cpp 100.00% <100.00%> (ø)
src/Bech32.h 100.00% <100.00%> (ø)
src/Bech32Address.cpp 88.67% <100.00%> (ø)
src/Bitcoin/SegwitAddress.cpp 98.30% <100.00%> (+0.47%) ⬆️
src/Bitcoin/SegwitAddress.h 62.50% <100.00%> (ø)
src/Cardano/AddressV3.cpp 91.21% <100.00%> (-0.12%) ⬇️
src/Bitcoin/CashAddress.cpp 82.22% <0.00%> (-2.40%) ⬇️
src/interface/TWStoredKey.cpp 91.57% <0.00%> (-1.06%) ⬇️
src/NEO/Witness.h 85.71% <0.00%> (-0.96%) ⬇️
src/NEO/Address.cpp 77.77% <0.00%> (-0.80%) ⬇️
... and 34 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f2c369...e9f957d. Read the comment docs.

Copy link
Contributor

@hewigovens hewigovens left a comment

Choose a reason for hiding this comment

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

looks good, how to use this new address in swift or kotlin?

src/Bech32.cpp Show resolved Hide resolved
src/Bech32.cpp Outdated Show resolved Hide resolved
src/Bech32.cpp Outdated Show resolved Hide resolved
src/Bech32.h Outdated Show resolved Hide resolved
@optout21 optout21 requested a review from vikmeup March 31, 2021 09:15
@optout21 optout21 requested a review from madcake April 22, 2021 22:36
@optout21 optout21 merged commit 0b4acd9 into master Apr 26, 2021
@optout21 optout21 deleted the ar/bip350 branch April 26, 2021 22:25
@optout21 optout21 mentioned this pull request Jul 22, 2021
4 tasks
SHH-HaoZhang pushed a commit to monacohq/wallet-core that referenced this pull request Oct 15, 2021
* Support Bech32M, Segwit v1, BIP350

* Review findings

Co-authored-by: Catenocrypt <catenocrypt@users.noreply.github.com>
cornbread78 pushed a commit to cornbread78/wallet-core that referenced this pull request Dec 22, 2021
* Support Bech32M, Segwit v1, BIP350

* Review findings

Co-authored-by: Catenocrypt <catenocrypt@users.noreply.github.com>
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.

Support BIP350; Bech32m address checksum for v1 segwit addresses
3 participants