-
Notifications
You must be signed in to change notification settings - Fork 14
feat: CoinJoin #592
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
Closed
feat: CoinJoin #592
Changes from all commits
Commits
Show all changes
210 commits
Select commit
Hold shift + click to select a range
780a384
feat: coinjoin integration draft
Syn-McJ 56b2015
Merge pull request #525 from Syn-McJ/feat/coinjoin-base-classes
Syn-McJ ed43497
feat: callbacks for WalletEx
Syn-McJ 161159d
chore: cleanup
Syn-McJ 466f203
Merge pull request #526 from Syn-McJ/feat/coinjoin-wallet-ex-callbacks
Syn-McJ 88bd849
feat: availableCoins & hasCollateralInputs methods
Syn-McJ 2702712
Merge pull request #528 from Syn-McJ/feat/coinjoin-available-coins
Syn-McJ bc74776
feat: selectCoinsGroupedByAddresses Obj-C part and callbacks
Syn-McJ 19a798c
Merge pull request #529 from Syn-McJ/feat/coinjoin-select-coins
Syn-McJ 76df1f4
feat: create_denominated callbacks
Syn-McJ ff0ba82
Merge pull request #530 from Syn-McJ/feat/coinjoin-create-denominated
Syn-McJ 5703274
feat: coinjoin derivation path and commit transaction
Syn-McJ 181fce9
Merge pull request #531 from Syn-McJ/feat/coinjoin-commit-tx
Syn-McJ 6b7056f
chore: refactor signtransaction without auth prompt
Syn-McJ 1288411
feat: expose availableCoins to Rust
Syn-McJ e776c38
fix: white background
Syn-McJ af91ac6
Merge pull request #532 from Syn-McJ/feat/coinjoin-auto-denom
Syn-McJ 30079d3
Merge branch 'develop' into feat/coinjoin-wait-utxos
Syn-McJ 0999127
fix: wait for denom tx to be published before making collateral
Syn-McJ 668bbbb
Merge pull request #535 from Syn-McJ/feat/coinjoin-wait-utxos
Syn-McJ 4c11654
feat: CoinJoinClientManager callbacks
Syn-McJ 8e4f8b0
Merge pull request #536 from Syn-McJ/feat/coinjoin-client-manager
Syn-McJ 8fe3bab
chore: refactor code into CoinJoinManager
Syn-McJ bd9c95c
Merge pull request #537 from Syn-McJ/feat/coinjoin-manager
Syn-McJ 5e65d71
feat: send accept message
Syn-McJ eecc368
chore: switching manager and wrapper
Syn-McJ 62c4986
Merge pull request #538 from Syn-McJ/feat/coinjoin-send-message
Syn-McJ 6daf3f9
fix: recalculate balance
Syn-McJ ebd7c4d
Merge branch 'feat/coinjoin' into feat/coinjoin-recalculate-balance
Syn-McJ 9185e19
Merge pull request #541 from Syn-McJ/feat/coinjoin-recalculate-balance
Syn-McJ b5025f5
fix: get script from an actuall output if empty in initWithTransaction
Syn-McJ 9709b9a
Merge pull request #542 from Syn-McJ/fix/coinjoin-sign-collateral
Syn-McJ 46cd8b7
feat: sending DSA to a connected masternode
Syn-McJ 4f4a9bf
Merge pull request #543 from Syn-McJ/feat/coinjoin-dsa
Syn-McJ 49507d9
feat: senddsq & MasternodeGroup draft
Syn-McJ 8377689
Merge pull request #544 from Syn-McJ/feat/coinjoin-senddsq
Syn-McJ a934f5c
feat: different message types support & disconnect logic for masterno…
Syn-McJ 79bf495
Merge pull request #546 from Syn-McJ/feat/coinjoin-send-dsi
Syn-McJ 25f0f93
feat: process dsq
Syn-McJ 90a183e
chore: cleanup
Syn-McJ 29fe732
fix: concurrent borrow issues
Syn-McJ 9c22d40
Merge pull request #547 from Syn-McJ/feat/coinjoin-dsq
Syn-McJ caa81f8
feat: process dssu & dsc, masternode group logic
Syn-McJ 158aadb
Merge pull request #548 from Syn-McJ/feat/coinjoin-dssu-dsc
Syn-McJ 828e9da
feat: masternodegroup improvements
Syn-McJ c9cca9d
Merge branch 'feat/coinjoin' into feat/coinjoin-dstx
Syn-McJ fd47f05
feat: connected/disconnected
Syn-McJ 63c38c5
Merge pull request #549 from Syn-McJ/feat/coinjoin-dstx
Syn-McJ 772daee
feat: process dsf
Syn-McJ b5e42b5
feat: connectivity management
Syn-McJ ff080d8
fix: remove redundant synchronize
Syn-McJ 544391d
Merge pull request #550 from Syn-McJ/feat/coinjoin-dsf
Syn-McJ 4b4c6fd
fix: reorg and bug fixes
Syn-McJ c202e47
Merge pull request #552 from Syn-McJ/feat/coinjoin-messaging
Syn-McJ 26834dd
fix: synchronization fixes
Syn-McJ 79846e1
fix: reducing synchronizations
Syn-McJ d2f4a88
Merge pull request #554 from Syn-McJ/fix/coinjoin-issues
Syn-McJ dfd36b3
chore: logs
Syn-McJ f336045
fix: dsa & dsi sending
Syn-McJ 13573d9
chore: cleanup
Syn-McJ c16bb8e
Merge branch 'develop' into fix/coinjoin-anyonecanpay
Syn-McJ 3fb4013
fix: anyonecanpay & masternode group improvements
Syn-McJ d66de8f
Merge pull request #556 from Syn-McJ/feat/coinjoin-server
Syn-McJ 4762b93
Merge pull request #558 from Syn-McJ/fix/coinjoin-anyonecanpay
Syn-McJ 53b5344
chore: fixes
Syn-McJ e372154
fix: empty inputs for anyonecanpay
Syn-McJ a3adbc3
fix: signed inputs
Syn-McJ 78044d7
feat: remove getIdentityIdsByKeyHashes and edit getIdentitiesByKeyHashes
pauldelucia b293048
fix: address dpns contract changes
pauldelucia 76d6899
fix: change misspelling serivce to service
pauldelucia c4ec591
Merge pull request #559 from Syn-McJ/fix/coinjoin-anyonecanpay
Syn-McJ 545c440
removed DAPIGRPC
QuantumExplorer 7cf821f
feat: event listeners
Syn-McJ 03f3d45
Merge pull request #561 from Syn-McJ/feat/coinjoin-events
Syn-McJ 1784155
fix: multisession mixing & refresh unused keys
Syn-McJ f1bab5b
fix: skip registerAddressesWithGapLimit if addresses aren't loaded
Syn-McJ 01e0c60
Merge branch 'fix/coinjoin-deriv-path' into fix/coinjoin-multisession
Syn-McJ 0d5175a
chore: wallet integration changes
Syn-McJ 91769f7
Merge pull request #562 from Syn-McJ/fix/coinjoin-multisession
Syn-McJ ff27ac1
Merge pull request #564 from Syn-McJ/fix/coinjoin-deriv-path
Syn-McJ 3790d39
fix: coinjoin balance calculation
Syn-McJ 732ba71
Merge branch 'fix/coinjoin-balance-calculations' into fix/coinjoin-send
Syn-McJ 0d183ad
Merge pull request #565 from Syn-McJ/fix/coinjoin-balance-calculations
Syn-McJ f3835f9
fix: use DSCoinControl to only include selected inputs
Syn-McJ 315879b
Merge branch 'fix/coinjoin-send' into feat/coinjoin-wallet-integration
Syn-McJ 41f025d
fix: wallet integration fixes
Syn-McJ 1b812f4
Merge pull request #566 from Syn-McJ/fix/coinjoin-send
Syn-McJ c43a04e
Merge pull request #567 from Syn-McJ/feat/coinjoin-wallet-integration
Syn-McJ 49b001b
fix: simplify MasternodeGroup
Syn-McJ 397c870
Merge pull request #568 from Syn-McJ/chore/coinjoin-connectivity-impr…
Syn-McJ f9e21b8
feat: coincontrol improvements
Syn-McJ f43c411
fix: MasternodeGroup crashes
Syn-McJ b865fe4
fix: change state to enabled when starting CoinJoin
Syn-McJ de6320d
fix: xcode 16 build
ogabrielides 371128a
Merge pull request #571 from ogabrielides/xcode-16-fix
Syn-McJ 16cf6b2
Merge pull request #570 from Syn-McJ/fix/coinjoin-mngroup-fixes
Syn-McJ b6db74b
fix: set from the caller to account for coinjoin being enabled, but …
Syn-McJ 03698d4
fix: stuck syncing
Syn-McJ 26e9aed
fix: take parameter from the caller
Syn-McJ 40ddb7b
Merge pull request #573 from Syn-McJ/fix/coinjoin-send-anonymized
Syn-McJ c15ff1d
chore: reduce logs
Syn-McJ 09eee81
fix: align derivation path limit gap with Android
Syn-McJ 5527d76
fix: NPE at DSChainBlockWasLockedNotification and logs
Syn-McJ 1b5af0d
fix: crash & logging fixes
Syn-McJ 17c1a92
fix: NPE at DSChainBlockWasLockedNotification and logs
Syn-McJ 8b8df3a
fix: crash & logging fixes
Syn-McJ c9d9c4f
Merge pull request #574 from Syn-McJ/feat/coinjoin-crash-fixes
Syn-McJ 23ebb5c
Merge branch 'feat/coinjoin' into fix/coinjoin-gap-limit
Syn-McJ fad1b1a
chore: cleanup
Syn-McJ 618d981
fix: report new blocks even if not started
Syn-McJ 4d36bc2
fix: example build
Syn-McJ 14de3ab
fix: incorrect polymorphism
pankcuf 805290b
Merge pull request #576 from dashpay/fix/special-tx-hashing
Syn-McJ 7f9026f
Merge branch 'feat/coinjoin' into fix/coinjoin-gap-limit
Syn-McJ e8143c3
fix: synchronization fixes
Syn-McJ 668ca4c
fix: private logs & memory management
Syn-McJ e7a8ea0
Merge pull request #575 from Syn-McJ/fix/coinjoin-gap-limit
Syn-McJ de07238
Merge pull request #577 from Syn-McJ/fix/coinjoin-crashes
Syn-McJ 343de62
Merge branch 'develop' into feat/coinjoin
Syn-McJ 02f184c
fix: improve error mode detection
Syn-McJ 6b74d4a
Merge branch 'develop' into feat/migrate-v0.22-to-v1.0-methods
pankcuf 849ce95
Merge pull request #560 from dashpay/feat/migrate-v0.22-to-v1.0-methods
pankcuf 0ad0a1c
Merge branch 'develop' into fix/coinjoin-error-mode
Syn-McJ 895a56b
Revert "feat: migrations for Platform v0.22 to v1"
pankcuf 629ca9d
Merge pull request #581 from dashpay/revert-560-feat/migrate-v0.22-to…
Syn-McJ adc69ac
Merge branch 'develop' into feat/coinjoin
Syn-McJ 2a64061
Merge branch 'feat/coinjoin' into fix/coinjoin-error-mode
Syn-McJ 68ccdd7
fix: adjust coinjoin gap limit when forming the bloom filter
Syn-McJ 0edaa96
chore: add session status to logs
Syn-McJ e37280a
chore: cleanup
Syn-McJ a4e3242
Merge pull request #582 from Syn-McJ/fix/coinjoin-gap-limit
Syn-McJ 54e7567
feat: compress logs
Syn-McJ 0b17e79
Merge branch 'feat/coinjoin-zip-logs' into fix/coinjoin-cache-tx-info
Syn-McJ 957a8e9
Merge pull request #583 from Syn-McJ/feat/coinjoin-zip-logs
Syn-McJ 9b57d27
fix: cache dashAmount and direction in DSTransaction
Syn-McJ a064058
fix: sync fixes
Syn-McJ aebadbc
Merge pull request #584 from Syn-McJ/fix/coinjoin-cache-tx-info
Syn-McJ 9e286c7
fix: asset lock payload
pankcuf dbf4a27
fix: add dest_change to coin control
Syn-McJ 6c7a5d3
Merge pull request #585 from dashpay/fix/asset-lock-payload
Syn-McJ a7e815f
chore: dirty
pankcuf e62dcdb
Merge pull request #586 from Syn-McJ/feat/coinjoin-dest-change
Syn-McJ 91855ab
chore: new portion of requests
pankcuf d6f726d
chore: switch onto remote dash-shared-core
pankcuf b5a52c9
chore: fix dtor
pankcuf dfc86c6
fix: asset lock payload
pankcuf d92d814
fix: asset lock tx change
Syn-McJ 3743e9a
Merge branch 'develop' into feat/coinjoin
Syn-McJ 81cec50
Merge pull request #587 from dashpay/feat/coinjoin
Syn-McJ 50c8926
feat: doc requests
pankcuf 66a868a
chore: remove old DAPI deps
pankcuf 60ac6c2
fix: memory & lock issues in sortTransactions
Syn-McJ 2090ec4
Merge branch 'fix/sort-transactions' into fix/coinjoin-double-spend
Syn-McJ a2905f3
chore: dashpay continuation (sync with identities/usernames/profiles …
pankcuf fb036d9
fix: notify UI of double spent or unconfirmed tx
Syn-McJ 2c6d6f6
chore: cleanup
pankcuf c2c571e
fix: polymorphism in DSAssetLockTransaction
Syn-McJ 160d7e0
Merge pull request #588 from dashpay/fix/sort-transactions
Syn-McJ 51f1515
Merge pull request #589 from Syn-McJ/fix/coinjoin-double-spend
Syn-McJ d077de5
fix: polymorphism in DSAssetUnlockTransaction
Syn-McJ 3445c8f
Merge branch 'develop' into fix/coinjoin-polymorphism
Syn-McJ f13c8ad
chore: continuation (fix identity key dictionary)
pankcuf 9535a57
chore: fix flag descriptors
pankcuf ee94d4a
Merge pull request #590 from Syn-McJ/fix/coinjoin-polymorphism
Syn-McJ 3056edf
chore: stream contact requests
pankcuf 463eb05
fix: check for valid data in ecdsaKeyAddressFromPublicKeyData
Syn-McJ 707eae9
fix: remove unnesesary synchronizations
Syn-McJ 40e83d1
chore: tmp commit
pankcuf 212b8a5
fix: process DSTX
Syn-McJ fd05825
feat: unlock coins from recently arrived mixing tx
Syn-McJ f2c0a1d
feat: coinjoin finishing mode
Syn-McJ b473d7e
Merge branch 'fix/coinjoin-polymorphism' into fix/coinjoin-finishing
Syn-McJ 6ee715c
Merge branch 'fix/coinjoin-crashes' into fix/coinjoin-finishing
Syn-McJ c3ecb68
Merge pull request #591 from Syn-McJ/fix/coinjoin-crashes
Syn-McJ af4cf45
chore: start (before merge with bindings)
pankcuf 0f9b5dc
chore: restore and bump DashSharedCore version
Syn-McJ f2adb2f
chore: cleanup Example podfile
Syn-McJ b391188
Merge pull request #593 from Syn-McJ/fix/coinjoin-finishing
Syn-McJ 1b087bc
chore: continuation
pankcuf a11fbef
fix: downgrade cocoapods for CI
Syn-McJ 4431af1
chore: continuation
pankcuf e1e27bd
chore: bump dash-shared-core version
Syn-McJ 93fb368
chore: continuation
pankcuf b868973
Merge branch 'develop' into feat/merge-with-coinjoin
pankcuf 7705823
chore: compilation sucess
pankcuf c3aeb2d
chore: warnings
pankcuf 51dcda8
chore: remove unused
pankcuf 487bf1f
chore: cleanup
pankcuf d3ca8b4
chore: cleanup
pankcuf 18c655c
chore: cleanup
pankcuf baacd7a
chore: cleanup
pankcuf 43de8f6
chore: cleanup
pankcuf 45a1e73
chore: cleanup
pankcuf 3a18462
chore: some dtors/cleanup
pankcuf f085f08
fix: restore incoming funds account flow
pankcuf ef8c110
fix: revive coinjoin test
pankcuf 966b4c4
fix: nullable invitationFullLink and error
Syn-McJ fbb7d66
chore: bump dash-shared-core
Syn-McJ b776325
chore: add SMLError conversion
pankcuf 442e433
chore: update masternode stuff as for rust-dashcore v0.39.1
pankcuf c5dbd55
chore: comment block removal logs
pankcuf d9ffbe0
Merge pull request #594 from Syn-McJ/fix/invites-interop
Syn-McJ 83a70f8
chore: some debug options
pankcuf 7293bda
chore: test address with gaps
pankcuf 2484530
Merge branch 'develop' into feat/merge-with-coinjoin
pankcuf e649fc7
chore: update podspec
pankcuf ada4a2b
chore: cleanup
pankcuf f3d6111
chore: cleanup
pankcuf 685afc5
chore: cleanup
pankcuf 3468a89
Merge pull request #595 from dashpay/feat/merge-with-coinjoin
pankcuf File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,3 @@ | ||
| [submodule "dashsync/crypto/secp256k1"] | ||
| path = dashsync/crypto/secp256k1 | ||
| url = https://github.com/bitcoin-core/secp256k1.git | ||
| [submodule "platform"] | ||
| path = platform | ||
| url = https://github.com/dashevo/platform.git |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -14,7 +14,7 @@ Pod::Spec.new do |s| | |
| s.homepage = 'https://github.com/dashevo/dashsync-ios.git' | ||
| s.license = { :type => 'MIT', :file => 'LICENSE' } | ||
| s.author = { 'quantumexplorer' => '[email protected]' } | ||
| s.source = { :git => 'https://github.com/dashevo/dashsync-iOS.git', :tag => s.version.to_s } | ||
| s.source = { :git => 'https://github.com/dashpay/dashsync-iOS.git', :tag => s.version.to_s } | ||
|
|
||
| s.ios.deployment_target = '13.0' | ||
| s.osx.deployment_target = '10.15' | ||
|
|
@@ -34,13 +34,10 @@ Pod::Spec.new do |s| | |
| s.ios.framework = 'UIKit' | ||
| s.macos.framework = 'Cocoa' | ||
| s.compiler_flags = '-Wno-comma' | ||
| s.dependency 'DashSharedCore', '0.4.19' | ||
| s.dependency 'DashSharedCore', '0.5.1' | ||
| s.dependency 'CocoaLumberjack', '3.7.2' | ||
| s.ios.dependency 'DWAlertController', '0.2.1' | ||
| s.dependency 'DSDynamicOptions', '0.1.2' | ||
| s.dependency 'DAPI-GRPC', '0.22.0-dev.8' | ||
| s.dependency 'TinyCborObjc', '0.4.6' | ||
| s.prefix_header_contents = '#import "DSEnvironment.h"' | ||
|
|
||
| end | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review implementation of
dsutxo_hexmacroThe new macro
dsutxo_hexappears to have an implementation issue. It attempts to access a fieldpinDSUTXOstruct (viau.p), but looking at theDSUTXOstruct definition (lines 79-82), there is no field namedp- it only containshashandnfields.The correct implementation should likely be:
📝 Committable suggestion