Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Dec 21, 2022

No description provided.

istream_iterator eats whitespace charactesr which causes parsing
failures for PSBTs that contain the bytes corresponding to those
characters.

Github-Pull: bitcoin-core/gui#687
Rebased-From: bb5ea1d
@DrahtBot
Copy link
Contributor

DrahtBot commented Dec 21, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@fanquake fanquake added this to the 24.1 milestone Dec 21, 2022
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 0662105

@fanquake fanquake merged commit 0da38b6 into bitcoin:24.x Dec 23, 2022
@maflcko maflcko deleted the 2212-backport-gui-psbt-whitespace-fix-🔈 branch December 23, 2022 14:38
fanquake added a commit that referenced this pull request Mar 13, 2023
932a609 doc: add initial release notes for v24.1 (fanquake)
cc4e315 doc: update manual pages for v24.1rc1 (fanquake)
787affb doc: update version in bips.md to v24.1 (fanquake)
5077e02 build: bump version to v24.1rc1 (fanquake)

Pull request description:

  Bump the version number to v24.1rc1.
  Regenerate the man pages.
  Update the version number in bips.md.
  Move the v24.0.1 release notes to doc/release-notes.
  Add initial release notes for v24.1.

  Merged changes to the 24.x branch since v24.0.1:
  - #26457
  - #26735
  - #26878
  - #26880

ACKs for top commit:
  achow101:
    ACK 932a609

Tree-SHA512: b90fd7c8f22c8fb096864e47cb79eaf5878524739a3b5c1d495c8c196b70d08c7b95fbfb1dfcdddf507bd8a72a5d133ecbe6ae898bbe70931f404afd0807b707
@bitcoin bitcoin locked and limited conversation to collaborators Dec 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants