fix: use estimated gas rate for bitcoin outTx and put a cap on total fee#1240
fix: use estimated gas rate for bitcoin outTx and put a cap on total fee#1240ws4charlie wants to merge 1 commit intodevelopfrom
Conversation
|
!!!WARNING!!! Be very careful about using Only suppress a single rule (or a specific set of rules) within a section of code, while continuing to scan for other problems. To do this, you can list the rule(s) to be suppressed within the #nosec annotation, e.g: /* #nosec G401 */ or //#nosec G201 G202 G203 Pay extra attention to the way |
|
Is it still relevant @ws4charlie or does #1243 replaces it? |
|
Closing, please feel free to reopen |
Description
We paid up to 3M satoshis ($800) in mock mainnet for bitcoin outTx which is way more than enough. A more reasonable gas price needs to be used.
0.001 BTCas the cap of fee. (in case of accident)1000 satoshisacross TSS signers.Closes:
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.
Checklist: