-
Notifications
You must be signed in to change notification settings - Fork 492
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
Fixing large transaction inputs timeout #12337
Fixing large transaction inputs timeout #12337
Conversation
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.
Interesting. The user was talking about 45 seconds timeout. Are you sure that you are at the right request?
Only at this point the program waits indefinitely |
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.
cACK
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.
Please remove the comments.
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.
ACK! gj
fixes: #12090
When we send transactions with many inputs to HW, sending the transaction to the device takes longer than usual, so I changed the timeout to dynamic.
Opinion:
A transaction with 1 input takes about 7-10 seconds
For 2 inputs, this number is 14-19 seconds
The current default timeout was 3 minutes. So if, say, in the case of 10 inputs, the number is about 100 seconds, given a safety margin, this should be good.