Setter crashes because of immutable emptyList() (#117)

Signed-off-by: S m, Aruna <aruna.mohan@walmart.com>
hyperledger-labs · Dec 28, 2023 · ff1c679 · ff1c679
1 parent c395dd1
commit ff1c679
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/main/java/hlf/java/rest/client/util/FabricChannelUtil.java b/src/main/java/hlf/java/rest/client/util/FabricChannelUtil.java
@@ -57,15 +57,15 @@ private static Configtx.ConfigPolicy getDefaultRoleConfigPolicyForMSP(
     // set rules
     // create those roles
     Policies.SignaturePolicyEnvelope.Builder signaturePolicyEnvelopeBuilder =
-        Policies.SignaturePolicyEnvelope.newBuilder();
+        Policies.SignaturePolicyEnvelope.newBuilder().addAllIdentities(mspPrincipals);
     Policies.SignaturePolicy.Builder signaturePolicyBuilder = Policies.SignaturePolicy.newBuilder();
     Policies.SignaturePolicy.NOutOf.Builder signatureNOutOfBuilder =
         Policies.SignaturePolicy.NOutOf.newBuilder().setN(1); // expect just one signature always
+    List<Policies.SignaturePolicy> signaturePolicies = new ArrayList<>();
     for (int idx = 0; idx < mspPrincipals.size(); idx++) {
-      signaturePolicyEnvelopeBuilder.setIdentities(idx, mspPrincipals.get(idx));
-      signatureNOutOfBuilder.setRules(
-          idx, Policies.SignaturePolicy.newBuilder().setSignedBy(idx).build());
+      signaturePolicies.add(Policies.SignaturePolicy.newBuilder().setSignedBy(idx).build());
     }
+    signatureNOutOfBuilder.addAllRules(signaturePolicies);
     signaturePolicyBuilder.setNOutOf(signatureNOutOfBuilder.build());
     signaturePolicyEnvelopeBuilder.setRule(signaturePolicyBuilder.build());
     // get the policy