Backports:
23.x backports #25316
pull fanquake wants to merge 6 commits into bitcoin:23.x from fanquake:23_x_backports changing 8 files +145 −106-
fanquake commented at 11:26 am on June 9, 2022: member
-
rpc: fix inappropriate warning for address type p2sh-segwit in createmultisig and addmultisigaddress
Github-Pull: #25220 Rebased-From: eaf6f630c0190c634b5f1c85f749437f4209cc36
-
test: ensure createmultisig and addmultisigaddress are not returning any warning for expected cases
Github-Pull: #25220 Rebased-From: 3a9b9bb38e653c8ff7220b9af6e337a90c2c22dc
-
windeploy: Renewed windows code signing certificate
Github-Pull: #25201 Rebased-From: 7e9fe6d800ee8f3381e8f6ad2371f7775c68fad9
-
tests: Calculate input weight more accurately
The external input test with specifying input weight would make a pessimistic estimate of the input weight. However this would result in a test failure as it is sometimes too pessimistic when an ECDSA signature ends up being smaller than usual. To correct this, we can calculate the input weight more accurately. Github-Pull: #24454 Rebased-From: 8a04a386f75d099f4b9864b0cdf7f26442b5801f
-
tests: Use descriptor that requires both legacy and segwit
Github-Pull: #24454 Rebased-From: 9f5ab670e7c8165f161ec44dbd95778c5515ece0
-
fanquake added the label Backport on Jun 9, 2022
-
fanquake added this to the milestone 23.1 on Jun 9, 2022
-
LarryRuane commented at 9:20 pm on June 9, 2022: contributorutACK 039ef215bc1268cefb645472d411993e5582486b I compared all 5 commit diffs with the source commit diffs, and the only one that differs is 32fa522a80c87231dc6194b982f64440c801b7d4 (this PR’s second commit) due to merge conflicts with
rpc_createmultisig.py
(specifically, the change to use MiniWallet, which isn’t being backported). But the conflicts look to be resolved correctly. -
jonatack commented at 4:32 pm on June 11, 2022: contributorPrepared a commit for backport, if helpful, in https://github.com/jonatack/bitcoin/commits/v23-backport-fix-rpcwait-with-netinfo-printing-negative-time-durations.
-
p2p: always set nTime for self-advertisements
If we self-advertised to an inbound peer with the address they gave us, nTime was left default-initialized, so that our peer wouldn't relay it any further along. Github-Pull: #25314 Rebased-From: 99b9e5f3a9fa29bbc1e45fc958470fbcc207ef23
-
LarryRuane commented at 7:33 pm on June 23, 2022: contributorre-utACK 4ebf6e35dcac936342525ec7b4b77a82c71693e7 Compared the latest commit, “p2p: always set nTime for self-advertisements” to the source commit, and they match.
-
achow101 commented at 8:34 pm on July 7, 2022: memberACK 4ebf6e35dcac936342525ec7b4b77a82c71693e7
-
MarcoFalke merged this on Jul 8, 2022
-
MarcoFalke closed this on Jul 8, 2022
-
fanquake deleted the branch on Jul 8, 2022
-
bitcoin locked this on Jul 8, 2023
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 21:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 21:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me