Skip to content

Fix multi_rpc test for hosts that dont default to utf8#10759

Merged
maflcko merged 1 commit intobitcoin:masterfrom
TheBlueMatt:2017-07-fix-mult-rpc-test
Jul 7, 2017
Merged

Fix multi_rpc test for hosts that dont default to utf8#10759
maflcko merged 1 commit intobitcoin:masterfrom
TheBlueMatt:2017-07-fix-mult-rpc-test

Conversation

@TheBlueMatt
Copy link
Copy Markdown
Contributor

Otherwise the utf8 written to bitcoin.conf throws an exception when
read from get_auth_cookie

Otherwise the utf8 written to bitcoin.conf throws an exception when
read from get_auth_cookie
@fanquake fanquake added the Tests label Jul 7, 2017
@meshcollider
Copy link
Copy Markdown
Contributor

Pretty easy utACK

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Jul 7, 2017

obvious utACK

@fanquake
Copy link
Copy Markdown
Member

fanquake commented Jul 7, 2017

utACK bc7d103

@practicalswift
Copy link
Copy Markdown
Contributor

utACK bc7d103

1 similar comment
@maflcko
Copy link
Copy Markdown
Member

maflcko commented Jul 7, 2017

utACK bc7d103

@maflcko maflcko merged commit bc7d103 into bitcoin:master Jul 7, 2017
maflcko pushed a commit that referenced this pull request Jul 7, 2017
bc7d103 Fix multi_rpc test for hosts that dont default to utf8 (Matt Corallo)

Tree-SHA512: 7644b1f50d4010a08aed5d1c87ab7326af9c109ac05dfbfc4bb6d5a19ace7997ef9cdd64d4301072f1106225de6d3aacccb17f53043b55ba61f5723c2b65af30
@jnewbery
Copy link
Copy Markdown
Contributor

jnewbery commented Jul 7, 2017

posthumous tested ACK. Works fine for me.

PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 10, 2019
…o utf8

bc7d103 Fix multi_rpc test for hosts that dont default to utf8 (Matt Corallo)

Tree-SHA512: 7644b1f50d4010a08aed5d1c87ab7326af9c109ac05dfbfc4bb6d5a19ace7997ef9cdd64d4301072f1106225de6d3aacccb17f53043b55ba61f5723c2b65af30
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
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.

7 participants