Page MenuHomePhabricator
Feed All Stories

Wed, Dec 18

roqqit requested review of D17337: [avalanche] Set local stake winner in the contender cache.
Wed, Dec 18, 17:26
roqqit updated the diff for D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.
  • just use getStakeContenderStatus instead of a new accessor
  • hasstakecontender -> getstakecontendervote per feedback
  • pass in prevblockhash and proofid instead of stakecontenderid
Wed, Dec 18, 17:23
roqqit updated the diff for D17368: Fix race to check chaintips in p2p_headers_sync_with_minchainwork.

feedback

Wed, Dec 18, 16:50
bytesofman published D17369: [mock-chronik-client] Add support for Agora ws subscriptions for review.
Wed, Dec 18, 15:32
teamcity closed T3706: Build Bitcoin-ABC / Master / Coverage is broken. as Resolved.
Wed, Dec 18, 08:56
Fabien requested changes to D17368: Fix race to check chaintips in p2p_headers_sync_with_minchainwork.
Wed, Dec 18, 08:35
PiRK closed D17366: [chronik] use custom jsonrpc error codes for application specific errors.
Wed, Dec 18, 08:06
PiRK committed rABC387990ca0d72: [chronik] use custom jsonrpc error codes for application specific errors (authored by PiRK).
[chronik] use custom jsonrpc error codes for application specific errors
Wed, Dec 18, 08:06

Tue, Dec 17

roqqit requested review of D17368: Fix race to check chaintips in p2p_headers_sync_with_minchainwork.
Tue, Dec 17, 22:35
bytesofman published D17357: [Cashtab] Do not airdrop XEC to agora scripts for review.
Tue, Dec 17, 22:06
bytesofman published D17367: [Cashtab] Show a spinner while parsed txs are loading for review.
Tue, Dec 17, 22:06
teamcity triaged T3706: Build Bitcoin-ABC / Master / Coverage is broken. as Unbreak Now! priority.
Tue, Dec 17, 21:55
roqqit closed D17349: [seeder] Bump thread stacksize.
Tue, Dec 17, 21:02
roqqit committed rABC61b79c87cb21: [seeder] Bump thread stacksize (authored by roqqit).
[seeder] Bump thread stacksize
Tue, Dec 17, 21:02
Fabien accepted D17366: [chronik] use custom jsonrpc error codes for application specific errors.
Tue, Dec 17, 20:51
Fabien added a comment to D17366: [chronik] use custom jsonrpc error codes for application specific errors.
In D17366#394875, @PiRK wrote:

Unrelated. Looks like a permission error when creating a directory

20:03:09     File "/work/abc-ci-builds/ecash-lib-integration-tests/test/functional/test_runner.py", line 1094, in <module>
20:03:09       main()
20:03:09     File "/work/abc-ci-builds/ecash-lib-integration-tests/test/functional/test_runner.py", line 361, in main
20:03:09       os.makedirs(tmpdir)
20:03:09     File "<frozen os>", line 225, in makedirs
20:03:09     FileExistsError: [Errno 17] File exists: '/work/abc-ci-builds/ecash-lib-integration-tests/test/tmp/test_runner_₿₵_🏃_20241217_200307'
Tue, Dec 17, 20:48
Fabien accepted D17349: [seeder] Bump thread stacksize.
Tue, Dec 17, 20:46
roqqit retitled D17349: [seeder] Bump thread stacksize from [seeder] Respect resource limit for stacksize as set by the system to [seeder] Bump thread stacksize.
Tue, Dec 17, 20:38
roqqit updated the diff for D17349: [seeder] Bump thread stacksize.

Just bump the value because predicting it is proving to be difficult and time consuming

Tue, Dec 17, 20:37
PiRK closed D17364: [chronik] check max number of parameters for Electrum commands.
Tue, Dec 17, 20:31
PiRK committed rABCe0dd041c75da: [chronik] check max number of parameters for Electrum commands (authored by PiRK).
[chronik] check max number of parameters for Electrum commands
Tue, Dec 17, 20:31
PiRK added a comment to D17366: [chronik] use custom jsonrpc error codes for application specific errors.

Unrelated. Looks like a permission error when creating a directory

Tue, Dec 17, 20:09
PiRK updated the diff for D17366: [chronik] use custom jsonrpc error codes for application specific errors.

custom error code and test for wrong type for boolean arg

Tue, Dec 17, 20:05
Fabien accepted D17364: [chronik] check max number of parameters for Electrum commands.
Tue, Dec 17, 20:03
teamcity added a comment to D17366: [chronik] use custom jsonrpc error codes for application specific errors.
Tue, Dec 17, 20:03
PiRK planned changes to D17366: [chronik] use custom jsonrpc error codes for application specific errors.

i missed one

Tue, Dec 17, 20:02
PiRK requested review of D17366: [chronik] use custom jsonrpc error codes for application specific errors.
Tue, Dec 17, 19:57
PiRK updated the diff for D17364: [chronik] check max number of parameters for Electrum commands.

feedback

Tue, Dec 17, 19:32
PiRK added inline comments to D17364: [chronik] check max number of parameters for Electrum commands.
Tue, Dec 17, 17:11
Fabien committed rABC48a7ec65fcf9: [CI] Make sure the nobody user account isn't expired on Arch (authored by Fabien).
[CI] Make sure the nobody user account isn't expired on Arch
Tue, Dec 17, 16:29
Fabien closed D17365: [CI] Make sure the nobody user account isn't expired on Arch.
Tue, Dec 17, 16:29
Fabien requested changes to D17364: [chronik] check max number of parameters for Electrum commands.
Tue, Dec 17, 16:28
PiRK retitled D17364: [chronik] check max number of parameters for Electrum commands from [chronik] check max number of parameters to [chronik] check max number of parameters for Electrum commands.
Tue, Dec 17, 16:08
PiRK accepted D17365: [CI] Make sure the nobody user account isn't expired on Arch.
Tue, Dec 17, 16:03
PiRK added a comment to D17365: [CI] Make sure the nobody user account isn't expired on Arch.

TIL chage

Tue, Dec 17, 16:03
Fabien requested review of D17365: [CI] Make sure the nobody user account isn't expired on Arch.
Tue, Dec 17, 15:36
bytesofman accepted D17335: [Modules] Added `bitcoinsuite-chronik-client` to monorepo.
Tue, Dec 17, 15:03
bytesofman closed D17340: [Cashtab] Parse tx history async to resolve UI lock issue on homescreen.
Tue, Dec 17, 14:55
bytesofman committed rABC30a23d5b0dad: [Cashtab] Parse tx history async to resolve UI lock issue on homescreen (authored by bytesofman).
[Cashtab] Parse tx history async to resolve UI lock issue on homescreen
Tue, Dec 17, 14:55
PiRK closed D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 14:28
PiRK committed rABCbdfd881df17c: [chronik] add electrum method blockchain.transaction.get_height (authored by PiRK).
[chronik] add electrum method blockchain.transaction.get_height
Tue, Dec 17, 14:28
PiRK added a comment to D17364: [chronik] check max number of parameters for Electrum commands.

Note that I also tried a form for the macro that check both min and max check_number_of_params!(params, 1, 2); but then I ran into a clippy warning when I added check_number_of_params!(params, 0, 0); in the server.ping handler

Tue, Dec 17, 14:25
PiRK updated the diff for D17364: [chronik] check max number of parameters for Electrum commands.

line length nit

Tue, Dec 17, 14:19
PiRK requested review of D17364: [chronik] check max number of parameters for Electrum commands.
Tue, Dec 17, 14:16
Fabien accepted D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 13:29
PiRK updated the diff for D17356: [chronik] add electrum method blockchain.transaction.get_height.

address clippy's complaint

Tue, Dec 17, 12:41
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 11:32
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 11:30
PiRK updated the summary of D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 11:27
PiRK added inline comments to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 11:25
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 11:22
PiRK updated the diff for D17356: [chronik] add electrum method blockchain.transaction.get_height.

feedback

Tue, Dec 17, 11:22
Fabien accepted D17335: [Modules] Added `bitcoinsuite-chronik-client` to monorepo.
Tue, Dec 17, 10:35
teamcity awarded D17335: [Modules] Added `bitcoinsuite-chronik-client` to monorepo a Haypence token.
Tue, Dec 17, 10:33
Fabien added a comment to D17335: [Modules] Added `bitcoinsuite-chronik-client` to monorepo.

@bot build-bitcoinsuite-chronik-client

Tue, Dec 17, 10:33
Fabien requested changes to D17337: [avalanche] Set local stake winner in the contender cache.
Tue, Dec 17, 10:31
Fabien requested changes to D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.
Tue, Dec 17, 10:14
Fabien requested changes to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Tue, Dec 17, 10:10
Fabien closed D17363: Bump version to 0.30.7.
Tue, Dec 17, 08:44
Fabien committed rABC10d026eb1e67: Bump version to 0.30.7 (authored by Fabien).
Bump version to 0.30.7
Tue, Dec 17, 08:44
emack accepted D17340: [Cashtab] Parse tx history async to resolve UI lock issue on homescreen.
Tue, Dec 17, 08:23
PiRK updated the diff for D17356: [chronik] add electrum method blockchain.transaction.get_height.

so it looks like we don't need to use serde_json::Value; in the module that implement try_from(value: &'_ serde_json::Value), but we still need to have the crate as a depencency (Cargo.[toml/lock])

Tue, Dec 17, 07:56

Mon, Dec 16

bytesofman updated the diff for D17269: [ecash-lib] Improve address handling of ecash dev libraries.

change toLegacyAddress behavior to throw if called with legacy address, move legacy functions to legacyaddr.ts, define toLegacyAddress in address.ts, organize and update tests accordingly

Mon, Dec 16, 23:05
bytesofman added inline comments to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Mon, Dec 16, 23:04
PiRK planned changes to D17356: [chronik] add electrum method blockchain.transaction.get_height.

Not sure why the build claims serde_json:: Value is unused in txid.rs, will investigate

Mon, Dec 16, 18:24
roqqit requested review of D17337: [avalanche] Set local stake winner in the contender cache.
Mon, Dec 16, 18:20
roqqit updated the diff for D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.

Use a debug RPC. Waiting for getavalanchepeerinfo is not sufficient here because its possible for this value to be updated before the contender is added to the cache.

Mon, Dec 16, 17:49
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Mon, Dec 16, 16:30
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Mon, Dec 16, 16:24
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Mon, Dec 16, 16:23
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Mon, Dec 16, 16:17
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Mon, Dec 16, 16:15
PiRK updated the diff for D17356: [chronik] add electrum method blockchain.transaction.get_height.

use is_none() isntead of match

Mon, Dec 16, 16:12
PiRK updated the diff for D17356: [chronik] add electrum method blockchain.transaction.get_height.

take a reference for TryFrom, use it also in transaction_get (minor behavior change in the error message, because now we have one less line that can raise a variant on the same error message)

Mon, Dec 16, 16:07
teamcity closed T3705: Build Bitcoin-ABC / Master / Sanitizers / Thread Sanitizer (TSAN) is broken. as Resolved.
Mon, Dec 16, 15:50
PiRK closed D17347: [chronik] add macros to extract params from a JSONRPC request.
Mon, Dec 16, 15:33
PiRK committed rABC719ef6251e2e: [chronik] add macros to extract params from a JSONRPC request (authored by PiRK).
[chronik] add macros to extract params from a JSONRPC request
Mon, Dec 16, 15:33
PiRK closed D17323: [chronik] add electrum method blockchain.transaction.get.
Mon, Dec 16, 15:32
PiRK closed D17314: [chronik] add a test for the new JSON RPC interface.
Mon, Dec 16, 15:32
PiRK committed rABC5ccdd58eaef1: [chronik] add electrum method blockchain.transaction.get (authored by PiRK).
[chronik] add electrum method blockchain.transaction.get
Mon, Dec 16, 15:32
PiRK committed rABCcedd98bd44e8: [chronik] add a test for the new JSON RPC interface (authored by PiRK).
[chronik] add a test for the new JSON RPC interface
Mon, Dec 16, 15:32
PiRK accepted D17363: Bump version to 0.30.7.
Mon, Dec 16, 15:32
teamcity triaged T3705: Build Bitcoin-ABC / Master / Sanitizers / Thread Sanitizer (TSAN) is broken. as Unbreak Now! priority.
Mon, Dec 16, 15:30
hazzarust updated the test plan for D17335: [Modules] Added `bitcoinsuite-chronik-client` to monorepo.
Mon, Dec 16, 15:20
Fabien requested review of D17363: Bump version to 0.30.7.
Mon, Dec 16, 15:09
teamcity added a comment to D17356: [chronik] add electrum method blockchain.transaction.get_height.
Mon, Dec 16, 15:00
teamcity added a comment to D17323: [chronik] add electrum method blockchain.transaction.get.
Mon, Dec 16, 14:51
bytesofman closed D17362: [ecash-herald] Make sure to build mock-chronik-client.
Mon, Dec 16, 14:44
bytesofman committed rABCfd2a3acba2d3: [ecash-herald] Make sure to build mock-chronik-client (authored by bytesofman).
[ecash-herald] Make sure to build mock-chronik-client
Mon, Dec 16, 14:44
Fabien accepted D17362: [ecash-herald] Make sure to build mock-chronik-client.
Mon, Dec 16, 14:44
bytesofman published D17362: [ecash-herald] Make sure to build mock-chronik-client for review.
Mon, Dec 16, 14:44
hazzarust updated the diff for D17335: [Modules] Added `bitcoinsuite-chronik-client` to monorepo.

Added CI integration (build-configurations.yml), added copyright licenses, changed server and reduced Tokio syntax.

Mon, Dec 16, 14:42
bytesofman abandoned D17361: [ecash-herald] Dockerfile update.

not the issue

Mon, Dec 16, 14:38
bytesofman published D17361: [ecash-herald] Dockerfile update for review.
Mon, Dec 16, 14:24
tobias_ruck requested changes to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Mon, Dec 16, 14:22
PiRK updated the diff for D17356: [chronik] add electrum method blockchain.transaction.get_height.

rebase after reducing the port range in D17314

Mon, Dec 16, 14:06
PiRK updated the diff for D17347: [chronik] add macros to extract params from a JSONRPC request.

rebase after reducing the port range in D17314

Mon, Dec 16, 14:05
PiRK updated the diff for D17323: [chronik] add electrum method blockchain.transaction.get.

rebase after reducing the port range in D17314

Mon, Dec 16, 14:05
teamcity awarded D17314: [chronik] add a test for the new JSON RPC interface a Haypence token.
Mon, Dec 16, 14:04
PiRK added a comment to D17314: [chronik] add a test for the new JSON RPC interface.

@bot build-linux64 build-linux-aarch64 build-chronik build-chronik-plugins

Mon, Dec 16, 14:04