hyperledger-labs / fabric-smart-client / 28500805867

77%

push

github

feat(fabric/config): make PKCS11 SessionCacheSize configurable (#1547)

* feat(fabric/config): make PKCS11 SessionCacheSize configurable

The PKCS11 session cache size was hardcoded to the fabric-lib-go default
of 10, with no way to set it from the MSP PKCS11 config. On nodes that
drive a large, concurrent PKCS11/HSM signing workload this caps the
session pool and serializes signing under load.

fabric-lib-go v1.1.4 exposes SessionCacheSize on bccsp/pkcs11.PKCS11Opts,
so bump to it and plumb the value through:

- add SessionCacheSize to config.PKCS11 and to the !pkcs11 build-tag stub
  so both build variants compile;
- pass it into PKCS11Opts in the x509 ToPKCS11OptsOpts converters (the
  package-level call site in config.go and the pkcs11-tag sub-package).

Default (omitted) keeps current behavior: zero, which fabric-lib-go maps
to its default cache size of 10.

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

* feat(integration/nwo): wire PKCS11 SessionCacheSize through the NWO converter

The NWO test-network PKCS11 converter (ToPKCS11Opts) read config.PKCS11 but
dropped SessionCacheSize, so a configured value was silently lost when
spinning up integration test networks. Bump this module's fabric-lib-go to
v1.1.4 (matching the root module) and forward the field, mirroring the
production x509 path.

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

* chore(comm/libp2p): bump fabric-lib-go to v1.1.4 to keep modules aligned

This module does not use the PKCS11 changes; the bump only keeps the
fabric-lib-go version consistent across all modules in the repo.

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

* docs(fabric/config): document PKCS11 SessionCacheSize in core.yaml

Add the SessionCacheSize option to the PKCS11 section of the core.yaml
reference in docs/configuration.md, per review feedback on #1547.

Refs #1546

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

---------

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

9 of 18 new or added lines in 2 files covered. (50.0%)

29713 of 39159 relevant lines covered (75.88%)

3058.12 hits per line