-
Notifications
You must be signed in to change notification settings - Fork 50
Comparing changes
Open a pull request
base repository: okx/proof-of-reserves
base: v3.1.15
head repository: okx/proof-of-reserves
compare: v3.1.16
- 18 commits
- 24 files changed
- 3 contributors
Commits on Jul 31, 2026
-
feat(verify): add BIP-322 P2TR verification and ADA payment credentia…
…l matching BIP-322 P2TR: - Add common/bip322.go implementing BIP-322 message hashing and Schnorr signature verification for Taproot key-path spends - VerifyUtxoCoin routes P2TR addresses to VerifyBip322P2TR; Schnorr is not a recoverable signature scheme, so the pubkey-recovery path cannot apply and the signing key is taken from the address instead - GuessUtxoCoinAddressType checks P2TR first: the existing bech32 pattern "^bc1[0-9a-zA-Z]{11,71}$" also matches bc1p... and reported them as P2WSH ADA: - Replace address reconstruction with AdaAddressMatchesPubKey, which decodes the address and compares the payment credential it carries. Base (addr1q...) and Pointer addresses also commit to a staking credential or chain pointer that no payment key can derive, so they can never be rebuilt from the key alone. Enterprise addresses are covered by the same path. - Reject script-hash payment credentials: a script is not a public key, so no key can prove ownership of one. Dependencies (required for Taproot/Schnorr support, cannot be reduced): - btcd 0.23.4 -> 0.24.2, btcec 2.3.2 -> 2.3.4, btcutil -> 1.1.5 - go-ethereum 1.12.2 -> 1.16.1, Go 1.17 -> 1.23 Tests: bip322_test.go covers the official BIP-322 vector, a POR report vector, and rejection cases (tampered signature, wrong message/address, script path). ada_test.go covers base/enterprise addresses, wrong key, script hash, testnet. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>Configuration menu - View commit details
-
Copy full SHA for 8faba08 - Browse repository at this point
Copy the full SHA 8faba08View commit details -
fix(checkbalance): parse 11- and 12-column POR report detail rows
InitPorCsvDataMap only accepted len(args) == 9. The detail section grew to 11 columns (eoa1/eoa2) and then 12 (Type), so every detail row was silently dropped -- loading the current production report yields 0 rows instead of 213087, leaving PorCoinDataMap empty and every report-side balance at 0. Resolve the layout from the header instead. DetectPorFormatOffset is now shared with verifyaddress, which no longer keeps its own copy. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 3cceefb - Browse repository at this point
Copy the full SHA 3cceefbView commit details -
docs: add design spec for network-driven signature verification
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 30f9eab - Browse repository at this point
Copy the full SHA 30f9eabView commit details -
docs: add implementation plan for network-driven verification
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 990a4b4 - Browse repository at this point
Copy the full SHA 990a4b4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ed7161 - Browse repository at this point
Copy the full SHA 8ed7161View commit details -
Configuration menu - View commit details
-
Copy full SHA for 649041e - Browse repository at this point
Copy the full SHA 649041eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 337ab61 - Browse repository at this point
Copy the full SHA 337ab61View commit details -
refactor(verify): resolve msg header and address type from network
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for fe393cf - Browse repository at this point
Copy the full SHA fe393cfView commit details -
test(crypto): pass network names to Verify* after semantic change
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for f0da870 - Browse repository at this point
Copy the full SHA f0da870View commit details -
refactor(csv_verify): look up verification scheme by network table
Switch the Deloitte flow's three PorCoinTypeMap[coin] lookups to NetworkType(coin), keeping the ECDSA fallback in verifyCSVLineInternal. The coin parameter here is always the network column value.
Configuration menu - View commit details
-
Copy full SHA for 484a388 - Browse repository at this point
Copy the full SHA 484a388View commit details -
Configuration menu - View commit details
-
Copy full SHA for 81272e3 - Browse repository at this point
Copy the full SHA 81272e3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 08c7891 - Browse repository at this point
Copy the full SHA 08c7891View commit details -
Configuration menu - View commit details
-
Copy full SHA for b9e6243 - Browse repository at this point
Copy the full SHA b9e6243View commit details -
docs: remove internal design/plan documents from repository
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 83967f4 - Browse repository at this point
Copy the full SHA 83967f4View commit details -
test(csv_verify): detect dq-por chain-column layout in multithread test
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 3d80f78 - Browse repository at this point
Copy the full SHA 3d80f78View commit details
Commits on Aug 3, 2026
-
feat(verify): derive NULS address for ECDSA signature verification
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2Configuration menu - View commit details
-
Copy full SHA for e1b2e7c - Browse repository at this point
Copy the full SHA e1b2e7cView commit details
Commits on Aug 14, 2026
-
2
Configuration menu - View commit details
-
Copy full SHA for 732764f - Browse repository at this point
Copy the full SHA 732764fView commit details -
Merge branch 'feature/ada-bip322-address-verification' into 'main'
feat(verify): add BIP-322 P2TR verification and ADA payment credential matching Closes BIP-322 See merge request sign-github-mirror/proof-of-reserves!6
yong.huang@okg.com committedAug 14, 2026 Configuration menu - View commit details
-
Copy full SHA for dc00a96 - Browse repository at this point
Copy the full SHA dc00a96View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v3.1.15...v3.1.16