Page MenuHomePhabricator

hazzarust (Harry Grant)
User

Projects

User does not belong to any projects.

User Details

User Since
Nov 17 2024, 14:52 (20 w, 10 h)
Slack Username
Harry

Recent Activity

Sat, Apr 5

hazzarust added inline comments to D17886: [chronik] Add failover_proxy to bitcoinsuite-chronik-client.
Sat, Apr 5, 13:32

Thu, Apr 3

hazzarust added a comment to D17886: [chronik] Add failover_proxy to bitcoinsuite-chronik-client.

Please dont review yet,

Thu, Apr 3, 11:39
hazzarust updated the diff for D17886: [chronik] Add failover_proxy to bitcoinsuite-chronik-client.

lint

Thu, Apr 3, 10:00
hazzarust requested review of D17886: [chronik] Add failover_proxy to bitcoinsuite-chronik-client.
Thu, Apr 3, 09:39

Wed, Mar 26

hazzarust closed D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.
Wed, Mar 26, 11:23
hazzarust committed rABC0c1713d91285: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into… (authored by hazzarust).
[Chronik] Add broadcast_txs_and_validate_rawtxs integration test into…
Wed, Mar 26, 11:23

Tue, Mar 25

hazzarust updated the diff for D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.

Added comments

Tue, Mar 25, 19:04
hazzarust closed D17853: [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client.
Tue, Mar 25, 19:02
hazzarust committed rABC396a501ae4d5: [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client (authored by hazzarust).
[chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client
Tue, Mar 25, 19:02
hazzarust added inline comments to D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.
Tue, Mar 25, 13:25
hazzarust updated the diff for D17853: [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client.

change ordering

Tue, Mar 25, 13:11

Mon, Mar 24

hazzarust updated the diff for D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.

Made change to how we handle status and ready to reduce indentation

Mon, Mar 24, 17:59
hazzarust retitled D17853: [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client from [chronik] Ammend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client to [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client.
Mon, Mar 24, 17:39
hazzarust updated the diff for D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.

Rebased

Mon, Mar 24, 17:39
hazzarust updated the summary of D17853: [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client.
Mon, Mar 24, 17:36
hazzarust updated the summary of D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.
Mon, Mar 24, 17:36
hazzarust requested review of D17853: [chronik] Amend broadcast_tx and broadcast_txs in bitcoinsuite-chronik-client.
Mon, Mar 24, 17:36

Sat, Mar 22

hazzarust requested review of D17847: [Chronik] Add broadcast_txs_and_validate_rawtxs integration test into bitcoinsuite-chronik-client.
Sat, Mar 22, 14:25

Sat, Mar 15

hazzarust closed D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.
Sat, Mar 15, 17:24
hazzarust committed rABC18cf3e52f32d: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite… (authored by hazzarust).
[chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite…
Sat, Mar 15, 17:24

Fri, Mar 14

hazzarust updated the diff for D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.

lint

Fri, Mar 14, 13:06
hazzarust updated the diff for D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.

Remove useless comments

Fri, Mar 14, 13:05

Thu, Mar 13

hazzarust updated the diff for D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.

Added comments from TS implementation into this implementation

Thu, Mar 13, 19:30
hazzarust added a comment to D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.

Still need to add comprehensive comments to explain whats going on - will do this later this evening.

Thu, Mar 13, 15:08
hazzarust updated the diff for D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.

lint

Thu, Mar 13, 15:08
hazzarust updated the diff for D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.

Removed boilerplate and replaced with assert_status_code_eq!()

Thu, Mar 13, 15:04
hazzarust closed D17784: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client.
Thu, Mar 13, 13:29
hazzarust committed rABCe3e84199d89c: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client (authored by hazzarust).
[chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client
Thu, Mar 13, 13:29
hazzarust updated the diff for D17784: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client.

Removed test and changed title + summary

Thu, Mar 13, 11:30
hazzarust retitled D17784: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client from [chronik] Add extract_status_code function to bitcoinsuite-chronik-client to [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client.
Thu, Mar 13, 11:29

Wed, Mar 12

hazzarust updated the diff for D17784: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client.

lint

Wed, Mar 12, 17:08
hazzarust updated the diff for D17784: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client.

Removed the impl function and made into macro

Wed, Mar 12, 17:07
hazzarust requested review of D17784: [chronik] Add assert_status_code_eq! macro to bitcoinsuite-chronik-client.
Wed, Mar 12, 14:13
hazzarust updated the summary of D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.
Wed, Mar 12, 12:07
hazzarust updated the summary of D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.
Wed, Mar 12, 12:07
hazzarust requested review of D17783: [chronik] Add blocktxs_and_tx_and_rawtx.rs integration test to bitcoinsuite-chronik-client.
Wed, Mar 12, 12:03

Mar 5 2025

hazzarust closed D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.
Mar 5 2025, 15:04
hazzarust committed rABC35b43b04dbd2: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite… (authored by hazzarust).
[Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite…
Mar 5 2025, 15:04
hazzarust updated the diff for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.

Linter

Mar 5 2025, 13:20
hazzarust updated the diff for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.

fixed error

Mar 5 2025, 12:38
hazzarust added inline comments to D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.
Mar 5 2025, 11:33

Mar 4 2025

hazzarust updated the diff for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.

Downcast the Report into ChronikClientError, so that we can now match on status code being returned (thanks Fabien!). Updated variable names from load to get, updated panic messages

Mar 4 2025, 23:25

Feb 27 2025

hazzarust abandoned D17272: [CHRONIK][WEB][MODULES] Updated every out of date dependency in Cargo.toml that does not require code change.
Feb 27 2025, 19:23
hazzarust committed rABC344ee6e4aeb3: [chronik] Add an integration test for the blockchain_info endpoint in… (authored by hazzarust).
[chronik] Add an integration test for the blockchain_info endpoint in…
Feb 27 2025, 13:08
hazzarust closed D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.
Feb 27 2025, 13:08
hazzarust committed rABC0f4a80e06575: [chronik] Add an integration test for the chronik_info endpoint in bitcoinsuite… (authored by hazzarust).
[chronik] Add an integration test for the chronik_info endpoint in bitcoinsuite…
Feb 27 2025, 13:07
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

rebase

Feb 27 2025, 12:54

Feb 24 2025

hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Updated comment

Feb 24 2025, 22:02
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Updating comment (linter plz dont kill me )

Feb 24 2025, 21:58
hazzarust added inline comments to D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.
Feb 24 2025, 21:52
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Updated !unreachable() comment and removed assert comments

Feb 24 2025, 20:32
hazzarust retitled D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client from [Chronik] Add block_and_blocks integration test into bitcoinsuite-chronik-client to [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.
Feb 24 2025, 17:52
hazzarust updated the diff for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.

Merged imports

Feb 24 2025, 17:39
hazzarust updated the test plan for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.
Feb 24 2025, 17:12
hazzarust updated the diff for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.

Updated the assert!() so that it will now anticipate 404 errors, renamed function names to be explicit

Feb 24 2025, 17:12
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Rename blockchain_info_ipc

Feb 24 2025, 12:47

Feb 22 2025

hazzarust updated the diff for D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.

lint

Feb 22 2025, 14:04
hazzarust requested review of D17687: [Chronik] Add integration test for block_and_blocks endpoint into bitcoinsuite-chronik-client.
Feb 22 2025, 14:02
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

lint

Feb 22 2025, 12:39

Feb 21 2025

hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Added unreachable panic, renamed test_blockchain_info to blockchain_info and added comments

Feb 21 2025, 14:11

Feb 20 2025

hazzarust abandoned D17653: [chronik] Add block_and_blocks to bitcoinsuite-chronik-client.
Feb 20 2025, 23:03
hazzarust updated the summary of D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.
Feb 20 2025, 22:37
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Renamed blockchain_info_new to blockchain_info

Feb 20 2025, 22:36
hazzarust retitled D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client from [chronik] Add blockchain_info to bitcoinsuite-chronik-client to [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.
Feb 20 2025, 22:36
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

sorry relint

Feb 20 2025, 19:38
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

lint

Feb 20 2025, 19:37
hazzarust added a comment to D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

I will make the changes from Pierre's comment in a separate diff to all current integration tests.

Feb 20 2025, 19:36
hazzarust updated the diff for D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.

Revamped blockchain_info to include the new trait changes.

Feb 20 2025, 19:34
hazzarust accepted D17680: [chronik-client] Fix a bug in the plugins integration test.
Feb 20 2025, 14:43

Feb 8 2025

hazzarust updated the test plan for D17653: [chronik] Add block_and_blocks to bitcoinsuite-chronik-client.
Feb 8 2025, 20:57
hazzarust updated the diff for D17653: [chronik] Add block_and_blocks to bitcoinsuite-chronik-client.

lint

Feb 8 2025, 20:01
hazzarust updated the summary of D17653: [chronik] Add block_and_blocks to bitcoinsuite-chronik-client.
Feb 8 2025, 20:00
hazzarust requested review of D17653: [chronik] Add block_and_blocks to bitcoinsuite-chronik-client.
Feb 8 2025, 19:59

Feb 4 2025

hazzarust updated the summary of D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.
Feb 4 2025, 23:11
hazzarust requested review of D17638: [chronik] Add an integration test for the blockchain_info endpoint in bitcoinsuite-chronik-client.
Feb 4 2025, 23:05

Jan 27 2025

hazzarust closed D17598: [ci] Add specific packages to run when calling cargo test.
Jan 27 2025, 16:16
hazzarust committed rABC78fba5d584cd: [ci] Add specific packages to run when calling cargo test (authored by hazzarust).
[ci] Add specific packages to run when calling cargo test
Jan 27 2025, 16:16
hazzarust updated the test plan for D17598: [ci] Add specific packages to run when calling cargo test.
Jan 27 2025, 16:07
hazzarust requested review of D17598: [ci] Add specific packages to run when calling cargo test.
Jan 27 2025, 15:53

Jan 25 2025

hazzarust closed D17565: [Chronik] Add unix socket listener to test_runner.rs.
Jan 25 2025, 16:03
hazzarust committed rABC1d3c803c723d: [Chronik] Add unix socket listener to test_runner.rs (authored by hazzarust).
[Chronik] Add unix socket listener to test_runner.rs
Jan 25 2025, 16:03
hazzarust closed D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.
Jan 25 2025, 16:01
hazzarust committed rABC7ea1a14a1618: [chronik] Add test runner to bitcoinsuite-chronik-client (authored by hazzarust).
[chronik] Add test runner to bitcoinsuite-chronik-client
Jan 25 2025, 16:01

Jan 24 2025

hazzarust updated the diff for D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.

Asserted at the end of test to make sure we return an error. Killing child process won't return an error, so we get the returned error from assert

Jan 24 2025, 17:32
hazzarust updated the diff for D17565: [Chronik] Add unix socket listener to test_runner.rs.

Removed timeout

Jan 24 2025, 12:42
hazzarust updated the diff for D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.

Added error handling

Jan 24 2025, 12:39

Jan 21 2025

hazzarust updated the diff for D17565: [Chronik] Add unix socket listener to test_runner.rs.

New line added + updated server to make explicit

Jan 21 2025, 15:52
hazzarust updated the diff for D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.

Removed assert_eq

Jan 21 2025, 15:37

Jan 20 2025

hazzarust updated the diff for D17565: [Chronik] Add unix socket listener to test_runner.rs.

Removed useless comment

Jan 20 2025, 22:07
hazzarust updated the diff for D17565: [Chronik] Add unix socket listener to test_runner.rs.

Deleted second test, made socket error handling verbose

Jan 20 2025, 22:04
hazzarust updated the test plan for D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.
Jan 20 2025, 21:58
hazzarust updated the test plan for D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.
Jan 20 2025, 21:57
hazzarust updated the summary of D17565: [Chronik] Add unix socket listener to test_runner.rs.
Jan 20 2025, 21:57
hazzarust updated the summary of D17565: [Chronik] Add unix socket listener to test_runner.rs.
Jan 20 2025, 21:56
hazzarust updated the test plan for D17565: [Chronik] Add unix socket listener to test_runner.rs.
Jan 20 2025, 17:30
hazzarust updated the diff for D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.

Made changes suggested

Jan 20 2025, 17:26

Jan 19 2025

hazzarust updated the summary of D17565: [Chronik] Add unix socket listener to test_runner.rs.
Jan 19 2025, 22:40
hazzarust requested review of D17565: [Chronik] Add unix socket listener to test_runner.rs.
Jan 19 2025, 22:40
hazzarust requested review of D17564: [chronik] Add test runner to bitcoinsuite-chronik-client.
Jan 19 2025, 11:54

Jan 15 2025

hazzarust closed D17528: [Chronik] Add chronik_info method into lib.rs.
Jan 15 2025, 11:00