Page MenuHomePhabricator
Feed Advanced Search

Jul 2 2024

PiRK accepted D16405: test: use MiniWallet for feature_csv_activation.py.
Jul 2 2024, 14:31
PiRK updated the diff for D16393: [electrum] remove server.relay_fee API call.

only remove 'relay_fee', keep 'banner'

Jul 2 2024, 14:19
PiRK updated the diff for D16406: [electrum] support trezor==0.13.9.

move the imports to compat.py

Jul 2 2024, 14:11
PiRK accepted D16404: Consistently pad the txs in feature_csv_activation.
Jul 2 2024, 12:06
PiRK updated the diff for D16406: [electrum] support trezor==0.13.9.

revert an unwanted change. I initially tried to not define RECOVERY_TYPE_MATRIX in trezor.py, because it is not used in this module, but after testing the application interactively it turned out to be imported and used elsewhere.

Jul 2 2024, 11:59
PiRK requested review of D16406: [electrum] support trezor==0.13.9.
Jul 2 2024, 11:55
PiRK accepted D16403: [validation] return MempoolAcceptResult for every tx on PCKG_TX failure.
Jul 2 2024, 11:12
PiRK accepted D16402: [refactor] rename variables in AcceptPackage for clarity.
Jul 2 2024, 11:08
PiRK accepted D16401: [doc] release note effective-feerate and effective-includes RPC results.
Jul 2 2024, 11:06
PiRK accepted D16400: [validation] remove PackageMempoolAcceptResult::m_package_feerate.
Jul 2 2024, 11:05
PiRK closed D16391: [electrum] fix handling of unsupported p2sh scripts in the TransactionDialog.
Jul 2 2024, 11:03
PiRK committed rABC928d1f039a5c: [electrum] fix handling of unsupported p2sh scripts in the TransactionDialog (authored by PiRK).
[electrum] fix handling of unsupported p2sh scripts in the TransactionDialog
Jul 2 2024, 11:03

Jul 1 2024

PiRK accepted D16394: Bump version to 0.29.8.
Jul 1 2024, 20:24
PiRK added a comment to D16393: [electrum] remove server.relay_fee API call.

I can imagine a use case for the banner, which is checking you're connecting the expected server. Any strong reason to remove this, which seems harmless ?

Jul 1 2024, 20:21
PiRK requested review of D16393: [electrum] remove server.relay_fee API call.
Jul 1 2024, 18:54
PiRK added a comment to D16391: [electrum] fix handling of unsupported p2sh scripts in the TransactionDialog.
In D16391#371074, @PiRK wrote:

Screenshot from 2024-07-01 11-02-16.png (708×822 px, 88 KB)

Jul 1 2024, 09:13
PiRK added a comment to D16391: [electrum] fix handling of unsupported p2sh scripts in the TransactionDialog.

Screenshot from 2024-07-01 11-02-16.png (708×822 px, 88 KB)

Jul 1 2024, 09:02
PiRK updated the summary of D16391: [electrum] fix handling of unsupported p2sh scripts in the TransactionDialog.
Jul 1 2024, 08:59
PiRK requested review of D16391: [electrum] fix handling of unsupported p2sh scripts in the TransactionDialog.
Jul 1 2024, 08:58

Jun 29 2024

PiRK accepted D16390: add RPC (-regtest only) for testing package policy.
Jun 29 2024, 20:58
PiRK requested changes to D16390: add RPC (-regtest only) for testing package policy.

mostly LGTM, just one debugging print to be removed

Jun 29 2024, 16:50
PiRK accepted D16389: validation: followups for de-duplication of packages.
Jun 29 2024, 16:38
PiRK added a comment to D16388: [Chronik] Add `ParseChronikParams` and use it in chronik::Start.

This looks like a good first step to remove most gArgs access (eventually we want the ArgsManager to not be a global).

Jun 29 2024, 11:55
PiRK requested changes to D16388: [Chronik] Add `ParseChronikParams` and use it in chronik::Start.
Jun 29 2024, 11:43

Jun 26 2024

PiRK accepted D16370: Introduce MemPoolAccept::CheckFeeRate.
Jun 26 2024, 08:24

Jun 25 2024

PiRK accepted D16369: [avalanche] Add a getflakyproofs RPC to retrieve the list of flaky proofs.
Jun 25 2024, 09:31
PiRK accepted D16368: [avalanche] Add an RPC to manually declare a flaky proof.
Jun 25 2024, 09:26

Jun 24 2024

PiRK accepted D16367: [avalanche] Return the staking reward winner proofid in getstakingreward.
Jun 24 2024, 19:11
PiRK accepted D16367: [avalanche] Return the staking reward winner proofid in getstakingreward.
Jun 24 2024, 13:44
PiRK accepted D16366: [avalanche] Remove the deprecatedrpc=getstakingreward option.
Jun 24 2024, 13:37

Jun 21 2024

PiRK closed D16363: [electrum] Bump version and add release notes.
Jun 21 2024, 14:47
PiRK committed rABCe3c3fd5f9272: [electrum] Bump version and add release notes (authored by PiRK).
[electrum] Bump version and add release notes
Jun 21 2024, 14:47
PiRK requested review of D16363: [electrum] Bump version and add release notes.
Jun 21 2024, 11:51
PiRK closed D16362: [electrum] fix support for Ledger app versions >= 2.4.0.
Jun 21 2024, 08:13
PiRK closed D16360: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver mode.
Jun 21 2024, 08:13
PiRK committed rABCad005c298d17: [electrum] fix support for Ledger app versions >= 2.4.0 (authored by PiRK).
[electrum] fix support for Ledger app versions >= 2.4.0
Jun 21 2024, 08:13
PiRK committed rABC2e3c49b5123a: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver… (authored by PiRK).
[electrum] handle 0x5515 status when a Ledger device is locked in screensaver…
Jun 21 2024, 08:13
PiRK closed D16359: [electrum] print a less alarming error when the wrong app is open on a Ledger device.
Jun 21 2024, 08:13
PiRK committed rABC5a4cbddd8c97: [electrum] print a less alarming error when the wrong app is open on a Ledger… (authored by PiRK).
[electrum] print a less alarming error when the wrong app is open on a Ledger…
Jun 21 2024, 08:13
PiRK requested review of D16362: [electrum] fix support for Ledger app versions >= 2.4.0.
Jun 21 2024, 08:04
PiRK updated the diff for D16359: [electrum] print a less alarming error when the wrong app is open on a Ledger device.

remove debugging related line

Jun 21 2024, 07:33
PiRK planned changes to D16359: [electrum] print a less alarming error when the wrong app is open on a Ledger device.
Jun 21 2024, 07:32
PiRK updated the diff for D16359: [electrum] print a less alarming error when the wrong app is open on a Ledger device.

handle the 0x6d00 status separately, show a full exception traceback in the dialog.

Jun 21 2024, 07:32

Jun 20 2024

PiRK retitled D16360: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver mode from [electrum] handle 0x5515 status when device is locked in screensaver mode to [electrum] handle 0x5515 status when a Ledger device is locked in screensaver mode.
Jun 20 2024, 16:25
PiRK requested review of D16360: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver mode.
Jun 20 2024, 16:23
PiRK requested review of D16359: [electrum] print a less alarming error when the wrong app is open on a Ledger device.
Jun 20 2024, 16:13
PiRK committed rABC7abe30a0a8b1: [electrum] minor cleanup in Transaction.verify_signature (authored by PiRK).
[electrum] minor cleanup in Transaction.verify_signature
Jun 20 2024, 15:26
PiRK closed D16302: [electrum] minor cleanup in Transaction.verify_signature.
Jun 20 2024, 15:26
PiRK updated the diff for D16302: [electrum] minor cleanup in Transaction.verify_signature.

rebase

Jun 20 2024, 12:25
PiRK closed D16356: [electrum] fix imports in digitalbitbox.
Jun 20 2024, 10:20
PiRK committed rABC0f5f33a17f15: [electrum] fix imports in digitalbitbox (authored by PiRK).
[electrum] fix imports in digitalbitbox
Jun 20 2024, 10:20
PiRK closed D16355: [electrum] move pubkey_from_signature -> ECPubkey.from_signature65.
Jun 20 2024, 10:20
PiRK closed D16352: [electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address.
Jun 20 2024, 10:19
PiRK closed D16354: [electrum] remove broken PaymentRequestBitPay20.
Jun 20 2024, 10:19
PiRK committed rABC2065f00accec: [electrum] move pubkey_from_signature -> ECPubkey.from_signature65 (authored by PiRK).
[electrum] move pubkey_from_signature -> ECPubkey.from_signature65
Jun 20 2024, 10:19
PiRK committed rABCd46492c45532: [electrum] remove broken PaymentRequestBitPay20 (authored by PiRK).
[electrum] remove broken PaymentRequestBitPay20
Jun 20 2024, 10:19
PiRK committed rABC2f9c5287d613: [electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address (authored by PiRK).
[electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address
Jun 20 2024, 10:19
PiRK closed D16351: [electrum] use ECPubkey in verify_message.
Jun 20 2024, 10:19
PiRK closed D16350: [electrum] cleanup point_to_ser and stop leaking ecdsa implementation details for ECPubkey.
Jun 20 2024, 10:19
PiRK committed rABC398a5f60947e: [electrum] use ECPubkey in verify_message (authored by PiRK).
[electrum] use ECPubkey in verify_message
Jun 20 2024, 10:19
PiRK committed rABC74b0434cd708: [electrum] cleanup point_to_ser and stop leaking ecdsa implementation details… (authored by PiRK).
[electrum] cleanup point_to_ser and stop leaking ecdsa implementation details…
Jun 20 2024, 10:19
PiRK closed D16349: [electrum] introduce a new ECPubkey class.
Jun 20 2024, 10:19
PiRK closed D16348: [electrum] cleanup ser_to_point.
Jun 20 2024, 10:19
PiRK committed rABCa3595882e2a7: [electrum] introduce a new ECPubkey class (authored by PiRK).
[electrum] introduce a new ECPubkey class
Jun 20 2024, 10:19
PiRK committed rABC1e5fc68f200a: [electrum] cleanup ser_to_point (authored by PiRK).
[electrum] cleanup ser_to_point
Jun 20 2024, 10:19
PiRK committed rABC83565a45b310: [electrum] cleanup ECC_YfromX (authored by PiRK).
[electrum] cleanup ECC_YfromX
Jun 20 2024, 10:19
PiRK closed D16347: [electrum] cleanup ECC_YfromX.
Jun 20 2024, 10:19
PiRK updated the diff for D16356: [electrum] fix imports in digitalbitbox.

rebase

Jun 20 2024, 07:11
PiRK updated the diff for D16355: [electrum] move pubkey_from_signature -> ECPubkey.from_signature65.

rebase

Jun 20 2024, 07:10
PiRK updated the diff for D16354: [electrum] remove broken PaymentRequestBitPay20.

rebase

Jun 20 2024, 07:10
PiRK updated the diff for D16352: [electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address.

rebase

Jun 20 2024, 07:08
PiRK updated the diff for D16351: [electrum] use ECPubkey in verify_message.

rebase

Jun 20 2024, 07:08
PiRK updated the diff for D16350: [electrum] cleanup point_to_ser and stop leaking ecdsa implementation details for ECPubkey.

rebase

Jun 20 2024, 07:06
PiRK updated the diff for D16349: [electrum] introduce a new ECPubkey class.

fix f-string, rebase

Jun 20 2024, 07:06
PiRK added inline comments to D16351: [electrum] use ECPubkey in verify_message.
Jun 20 2024, 07:05
PiRK updated the diff for D16348: [electrum] cleanup ser_to_point.

rename ser_to_point -> ser_to_coordinates and ser_to_ecdsa_python_point -> ser_to_point
Use a NamedTuple for the output of ser_to_point (better type-safety than a generic 2-tuple)

Jun 20 2024, 06:49

Jun 19 2024

PiRK added inline comments to D16348: [electrum] cleanup ser_to_point.
Jun 19 2024, 21:08
PiRK updated the test plan for D16348: [electrum] cleanup ser_to_point.
Jun 19 2024, 15:47
PiRK accepted D16353: [doc] package deduplication.
Jun 19 2024, 15:42
PiRK requested review of D16356: [electrum] fix imports in digitalbitbox.
Jun 19 2024, 15:41
PiRK requested review of D16355: [electrum] move pubkey_from_signature -> ECPubkey.from_signature65.
Jun 19 2024, 15:23
PiRK requested review of D16354: [electrum] remove broken PaymentRequestBitPay20.
Jun 19 2024, 15:18
PiRK added inline comments to D16352: [electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address.
Jun 19 2024, 12:42
PiRK updated the summary of D16351: [electrum] use ECPubkey in verify_message.
Jun 19 2024, 12:39
PiRK requested review of D16352: [electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address.
Jun 19 2024, 12:35
PiRK added inline comments to D16351: [electrum] use ECPubkey in verify_message.
Jun 19 2024, 12:35
PiRK requested review of D16351: [electrum] use ECPubkey in verify_message.
Jun 19 2024, 12:32
PiRK added a comment to D16349: [electrum] introduce a new ECPubkey class.

Also future backports of electrum will depend on these changes (PSBT for instance)

Jun 19 2024, 12:30
PiRK requested review of D16350: [electrum] cleanup point_to_ser and stop leaking ecdsa implementation details for ECPubkey.
Jun 19 2024, 12:29
PiRK requested review of D16349: [electrum] introduce a new ECPubkey class.
Jun 19 2024, 12:28
PiRK requested review of D16348: [electrum] cleanup ser_to_point.
Jun 19 2024, 12:24
PiRK requested review of D16347: [electrum] cleanup ECC_YfromX.
Jun 19 2024, 12:23

Jun 18 2024

PiRK accepted D16346: [refactor] consolidate valid MempoolAcceptResult processing.
Jun 18 2024, 15:05
PiRK added inline comments to D16345: Add test coverage for oversized block messages.
Jun 18 2024, 07:12
PiRK accepted D16345: Add test coverage for oversized block messages.

The code looks goods. For future maintenance and ease of review, some additional comments would help.

Jun 18 2024, 07:09

Jun 17 2024

PiRK closed D16333: [electrum] start implementing the PSBT format.
Jun 17 2024, 15:31
PiRK committed rABCeb0a68ec66d0: [electrum] start implementing the PSBT format (authored by PiRK).
[electrum] start implementing the PSBT format
Jun 17 2024, 15:31
PiRK closed D16336: [electrum] make read_compact_size return None at EOF instead of raising an error.
Jun 17 2024, 15:31
PiRK committed rABCe1f67687b71c: [electrum] make read_compact_size return None at EOF instead of raising an error (authored by PiRK).
[electrum] make read_compact_size return None at EOF instead of raising an error
Jun 17 2024, 15:31
PiRK updated the diff for D16333: [electrum] start implementing the PSBT format.

fix a comment in the tests

Jun 17 2024, 09:19