hyperledger-labs / fabric-smart-client / 26111026165

71%

Build Type

push

github

Committed by web-flow

Commit Message

fix(fabric/config): add missing GetInt prefix wrapper (#1463)

Service exposes prefixed accessors (GetString, GetBool, IsSet, ...) that
wrap the embedded Configuration with "fabric.<prefix>." for all keys. The
GetInt wrapper was missing, so callers like OrdererConnectionPoolSize and
BroadcastNumRetries silently fell through to Configuration.GetInt without
the prefix. The IsSet branch reported the key present (correctly prefixed)
but the value lookup missed and returned 0.

Practical effect: setting fabric.default.ordering.connectionPoolSize: 20
in yaml made OrdererConnectionPoolSize() return 0 instead of 20 - silently
worse than leaving the key unset (which falls back to default 10).

Add the missing wrapper to match the other accessors.

Signed-off-by: Evan <evanyan@sign.global>

Coverage Stats

3 of 3 new or added lines in 1 file covered. (100.0%)

2 existing lines in 1 file now uncovered.

27352 of 39836 relevant lines covered (68.66%)

3039.52 hits per line