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

Set kaspawallet daemon send/recv message size limits to 100MB #2117

Draft
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

michaelsutton
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Jul 16, 2022

Codecov Report

Merging #2117 (351a9a7) into dev (b26b9f6) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev    #2117      +/-   ##
==========================================
- Coverage   59.16%   59.16%   -0.01%     
==========================================
  Files         678      678              
  Lines       32897    32905       +8     
==========================================
+ Hits        19464    19467       +3     
- Misses      10618    10621       +3     
- Partials     2815     2817       +2     
Impacted Files Coverage Δ
cmd/kaspawallet/common.go 0.00% <ø> (ø)
cmd/kaspawallet/daemon/client/client.go 0.00% <0.00%> (ø)
cmd/kaspawallet/daemon/server/server.go 0.00% <0.00%> (ø)
...ructure/network/rpcclient/grpcclient/grpcclient.go 76.56% <0.00%> (-4.69%) ⬇️
app/rpc/rpccontext/notificationmanager.go 59.44% <0.00%> (-1.39%) ⬇️
.../protocol/flows/v5/blockrelay/handle_relay_invs.go 52.63% <0.00%> (-0.81%) ⬇️
...selectedchainstore/headers_selected_chain_store.go 75.58% <0.00%> (+6.97%) ⬆️
...ain/consensus/database/binaryserialization/hash.go 78.26% <0.00%> (+8.69%) ⬆️
...tructure/network/rpcclient/rpc_on_chain_changed.go 66.66% <0.00%> (+14.28%) ⬆️

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 b26b9f6...351a9a7. Read the comment docs.

@michaelsutton
Copy link
Collaborator Author

This PR is suppose to allow huge wallet compounding operations to succeed (supporting hundreds of transactions). However I tested this on testnet with the help of @biospb, and it seems that some of the later transactions do not get processed by the network, possibly due to mempool cleanup policies (it takes the network a few minutes to process hundreds of huge transactions). It seems that only the first layer of compounding txs succeeds, while the following layers which depend on the first, are dropped eventually.
Any thoughts @someone235 @svarogg before I close this ?

@michaelsutton michaelsutton marked this pull request as draft August 21, 2022 00:11
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.

2 participants