💬 pinheadmz commented on issue "help i think my core is hacked":
(https://github.com/bitcoin/bitcoin/issues/29944#issuecomment-2073702422)
This is your change output. Look for more information on https://bitcoin.stackexchange.com

✅ pinheadmz closed an issue: "help i think my core is hacked"
(https://github.com/bitcoin/bitcoin/issues/29944)

💬 laanwj commented on pull request "refactor: Use our own implementation of urlDecode":
(https://github.com/bitcoin/bitcoin/pull/29904#discussion_r1577057922)
Even this can go? It's fascinating to see how this little function was such a wedge issue and now we can drop half the build system 😄

👍 laanwj approved a pull request: "refactor: Use our own implementation of urlDecode"
(https://github.com/bitcoin/bitcoin/pull/29904#pullrequestreview-2018558155)
Code review ACK 477c03b42825084ac344050d97ea206a82ba0eb6

💬 luke-jr commented on pull request "Add used balance to overview page":
(https://github.com/bitcoin-core/gui/pull/775#issuecomment-2073729945)
What is the use case here? (And the avoid_reuse flag in general...)

💬 laanwj commented on pull request "doc: Suggest only necessary Qt packages for installation on OpenBSD":
(https://github.com/bitcoin/bitcoin/pull/29947#issuecomment-2073755514)
Concept ACK. It's nice that OpenBSD provides separate modules for Qt nowadays.

💬 HaSan1y commented on issue "help i think my core is hacked":
(https://github.com/bitcoin/bitcoin/issues/29944#issuecomment-2073763151)
> This is your change output. Look for more information on https://bitcoin.stackexchange.com
i dont understand? what do you mean with 'change output'?? what do you mean with look for more information?
elaborate please?
anyway who are you? did you even read my problem?
i know what is inside that link https://explorer.btc.com/btc/transaction/c5069dd52612dd07cfff6a2c587356497a2f8e7489923376a92420850fe27d09
im saying something is wrong with the transaction. did you perheps programm to steal
...

💬 IAmAdamRest commented on issue "Crashes in v27 on Windows 10 and 11":
(https://github.com/bitcoin/bitcoin/issues/29916#issuecomment-2073776190)
Having this same issue. Where do I find my logs?

💬 tdb3 commented on pull request "cli: Detect port errors in rpcconnect and rpcport":
(https://github.com/bitcoin/bitcoin/pull/29521#discussion_r1577085955)
Yes, that sounds reasonable. Removed the "18443" test and pushed.

💬 tdb3 commented on pull request "test: Run framework unit tests in parallel":
(https://github.com/bitcoin/bitcoin/pull/29771#issuecomment-2073796442)
Rebased and pushed.

💬 laanwj commented on pull request "Bugfix: JSON-RPC request Content-Type is application/json":
(https://github.com/bitcoin/bitcoin/pull/29946#issuecomment-2073812562)
IIRC we do *return* `application/json` content type, so this at least makes it symmetric. id be against enforcing this (because so many clients already specify it wrong), but the example would be better this way.

💬 Umiiii commented on pull request "test: add missing comparison of node1's mempool in MempoolPackagesTest":
(https://github.com/bitcoin/bitcoin/pull/29941#issuecomment-2073825129)
@glozow could you review again and see if it meets the requirements now ..?

💬 Umiiii commented on pull request "test: add missing comparison of node1's mempool in MempoolPackagesTest":
(https://github.com/bitcoin/bitcoin/pull/29941#discussion_r1577099806)
Could you please clarify what you meant by "indented wrong"? I believe these lines are intended to follow the same indentation pattern as the rest of the file. If there's a specific indentation style that should be applied here, I'd appreciate your guidance.

🤔 tdb3 reviewed a pull request: "contrib: rpcauth.py - Add new option (-json) to output text in json format"
(https://github.com/bitcoin/bitcoin/pull/29433#pullrequestreview-2018620939)
Concept ACK.

Looks like these commits (4f746716c008216ff01da48ccfda559308b4232e, 1606590bbffa46f88fa6e4333960e82b919efd8a, and f2fb13bb1ac392abc4ccd656b025aa017b12060e) should be squashed to clean up and tell a concise story of update.

Seems to work well:
```
$ ./rpcauth.py
usage: rpcauth.py [-h] [-j] username [password]
rpcauth.py: error: the following arguments are required: username

$ ./rpcauth.py -h
usage: rpcauth.py [-h] [-j] username [password]

Create login credentials for
...

💬 Umiiii commented on pull request "test: add missing comparison of node1's mempool in MempoolPackagesTest":
(https://github.com/bitcoin/bitcoin/pull/29941#discussion_r1577104003)
But I agreed with you. Those lines are self explanatory. I removed the comments.

✅ Umiiii closed a pull request: "test: add missing comparison of node1's mempool in MempoolPackagesTest"
(https://github.com/bitcoin/bitcoin/pull/29941)

📝 Umiiii opened a pull request: "test: add missing comparison of node1's mempool in MempoolPackagesTest"
(https://github.com/bitcoin/bitcoin/pull/29948)
#29941 Recreated a pull request because there was a conflict. Trying to resolve the conflict but the old one automatically closed.

Add missing comparison for TODO comments in `mempool_packages.py`

Also, notice that the ancestor size limits and descendant size limits actually implemented in #21800 , so I removed the todo for those two size limits.

@glozow @instagibbs

:lock: fanquake locked an issue: "help i think my core is hacked"
(https://github.com/bitcoin/bitcoin/issues/29944)

💬 fanquake commented on pull request "JSON-RPC request Content-Type is application/json":
(https://github.com/bitcoin/bitcoin/pull/29946#issuecomment-2073990266)
Removed bugfix given this is a doc/test only change.

💬 maflcko commented on pull request "contrib: rpcauth.py - Add new option (-json) to output text in json format":
(https://github.com/bitcoin/bitcoin/pull/29433#issuecomment-2074059554)
Please squash your commits according to https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#squashing-commits