Skip to content

Commit 2453c27

Browse files
author
Jason Yellick
committed
FAB-10471 Cleanup configtx.yaml defaults
The suggested default policies don't really mesh with the default ACLs. This CR cleans them up and updates the e2e_cli to use the suggested defaults. Change-Id: I294bfc59d543d7c6191237e1cd34957efdeb64dc Signed-off-by: Jason Yellick <jyellick@us.ibm.com>
1 parent 6b34fbf commit 2453c27

File tree

2 files changed

+19
-5
lines changed

2 files changed

+19
-5
lines changed

examples/e2e_cli/configtx.yaml

+16-2
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ Organizations:
5757
Policies:
5858
Readers:
5959
Type: Signature
60-
Rule: "OR('Org1MSP.admin', 'Org1MSP.peer')"
60+
Rule: "OR('Org1MSP.admin', 'Org1MSP.peer', 'Org1MSP.client')"
6161
Writers:
6262
Type: Signature
6363
Rule: "OR('Org1MSP.admin', 'Org1MSP.client')"
@@ -88,7 +88,7 @@ Organizations:
8888
Policies:
8989
Readers:
9090
Type: Signature
91-
Rule: "OR('Org2MSP.admin', 'Org2MSP.peer')"
91+
Rule: "OR('Org2MSP.admin', 'Org2MSP.peer', 'Org2MSP.client')"
9292
Writers:
9393
Type: Signature
9494
Rule: "OR('Org2MSP.admin', 'Org2MSP.client')"
@@ -115,6 +115,20 @@ Organizations:
115115

116116
MSPDir: crypto-config/idemix/idemix-config
117117

118+
# Policies defines the set of policies at this level of the config tree
119+
# For organization policies, their canonical path is usually
120+
# /Channel/<Application|Orderer>/<OrgName>/<PolicyName>
121+
Policies:
122+
Readers:
123+
Type: Signature
124+
Rule: "OR('Org3MSP.admin', 'Org3MSP.peer', 'Org3MSP.client')"
125+
Writers:
126+
Type: Signature
127+
Rule: "OR('Org3MSP.admin', 'Org3MSP.client')"
128+
Admins:
129+
Type: Signature
130+
Rule: "OR('Org3MSP.admin')"
131+
118132
AnchorPeers:
119133
# AnchorPeers defines the location of peers which can be used
120134
# for cross org gossip communication. Note, this value is only

sampleconfig/configtx.yaml

+3-3
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,13 @@ Organizations:
3838
Rule: "OR('SampleOrg.member')"
3939
# If your MSP is configured with the new NodeOUs, you might
4040
# want to use a more specific rule like the following:
41-
# Rule: "OR('SampleOrg.admin', 'SampleOrg.peer')"
41+
# Rule: "OR('SampleOrg.admin', 'SampleOrg.peer', 'SampleOrg.client')"
4242
Writers:
4343
Type: Signature
4444
Rule: "OR('SampleOrg.member')"
4545
# If your MSP is configured with the new NodeOUs, you might
4646
# want to use a more specific rule like the following:
47-
# Rule: "OR('SampleOrg.admin', 'SampleOrg.client'')"
47+
# Rule: "OR('SampleOrg.admin', 'SampleOrg.client')"
4848
Admins:
4949
Type: Signature
5050
Rule: "OR('SampleOrg.admin')"
@@ -189,7 +189,7 @@ Application: &ApplicationDefaults
189189
cscc/GetConfigTree: /Channel/Application/Readers
190190

191191
# ACL policy for cscc's "SimulateConfigTreeUpdate" function
192-
cscc/SimulateConfigTreeUpdate: /Channel/Application/Writers
192+
cscc/SimulateConfigTreeUpdate: /Channel/Application/Readers
193193

194194
#---Miscellanesous peer function to policy mapping for access control---#
195195

0 commit comments

Comments
 (0)