* changes:
  collACL: use cache to avoid repetative ACL check
  collACL: perform ACL on private data access
  collACL: add simpleCollectionStore to txContext
  collACL: add `MemberOnlyRead` in collectionConf

This CR introduces a cache (i.e., Golang map) in the txContext
so that we can avoid repeating the same collection ACL check
for a given <ns, coll> during privateData access.

FAB-13040 #done

Change-Id: I3093c2079e78b2b349276a275b22e77d7bd9c871
Signed-off-by: senthil <cendhu@gmail.com>

this change set is to rewrite private data e2e tests using the new test framework.
as part of this change, will also fix pvtdata e2e tests flakes.

FAB-12181 #done

Change-Id: Ib01765ca76a553d960e144e736042be8b6aa9eab
Signed-off-by: nirro <nirro@il.ibm.com>

Change-Id: Ifa793a3a9346432a6a164a63bbcb45075e29541a
Signed-off-by: joe-alewine <Joe.Alewine@ibm.com>

It is not actually necessary to specify the
CC type when instantiating chaincode as
CC name and version are actually used to
get the installed chaincode bytes from the
filesytem.

With this change, CCType is no longer checked
at all during instantiate and is therefore no
longer required (which will make things easier
for clients)

Change-Id: Ic1decbf98cf28689f85af94931fa121461ef1e4d
Signed-off-by: Gari Singh <gari.r.singh@gmail.com>

Pin to v1.0.0 rather than master

FAB-13092 #done

Change-Id: I8c7f2137a9c8a578b9f2786aede61b9c0813476c
Signed-off-by: Gari Singh <gari.r.singh@gmail.com>

using the simpleCollectionStore in transactionContext,
get the policy and evaluate it against the signedData to
decide whether a private data access should be allowed or not.

FAB-13039 #done

Change-Id: I9f66ae70aac5c439ec5dc943407c59d365ef5d84
Signed-off-by: senthil <cendhu@gmail.com>

This CR adds a mutex to guard parallel access to `SendSubmit`
rpc call in etcdraft chain, due to thread-unsafe of gRPC.

This is going to be tested by subsequent benchmark tests.
Without this serialization, loading etcdraft orderer with
many parallel requests would most likely cause a dead-lock
in communication, which causes benchmark test to stuck.

Change-Id: Id2f6e8f86f7465880725e1aecc7374c3e9e4106e
Signed-off-by: Jay Guo <guojiannan1101@gmail.com>

-- define proto messages needed for expectation request
-- add RequestExpectation method to prover server
-- add RequestExpectation method to the transactor interface
-- add policy checking for expectation request

Change-Id: I66d4c5099f423a937b4d969a3272ca76f2c2363c
Signed-off-by: Wenjian Qiao <wenjianq@gmail.com>

We need to add simpleCollectionStore interface in the
transactionContext so that we can check whether the creator
has access to the collection or not.

FAB-13038 #done

Change-Id: I5b150bb751bd22451b546b8c1028e54f1f1ca884
Signed-off-by: senthil <cendhu@gmail.com>

Before v1.4 cut, disable prover service and invalidate token tx
After v1.4 cut, restore the code.

Change-Id: I5e123c59b8fd0416335ec44ceee5f3fa75394495
Signed-off-by: Wenjian Qiao <wenjianq@gmail.com>

This CR adds a new parameter named `MemberOnlyRead` in the
collectionConfig.

FAB-13041 #done

Change-Id: Ie80d81130e61f71ef219e5c370b8059647979feb
Signed-off-by: senthil <cendhu@gmail.com>

FAB-13083 #done

Change-Id: I6ee6b1e4e30b5e3b7fdb7b0b1c145f33acfd92af
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

As per KeyEndorsementPolicy
Removed DelAllOrgs() and corrected return type for ListOrgs() in docs

Change-Id: Iad70f0ca7b10df76377329ae3b8793b687e2d892
Signed-off-by: Sheetal Sharma <sheetal_sharma@persistent.co.in>

CI process checks existence of trailing spaces.
If trailing spaces exist in modified ASCII-text file, VerifyBuild fails.
On the other hand, local build does not check it.

This patch adds checking for trailing spaces existence in local build time.

Change-Id: I31b35c7782d0aa71366e3dd907073302d5d5b286
Signed-off-by: Daisuke IIZUKA <daisuke.iizuka.ag@hitachi.com>

In preparation of adding the blockcutter metric, this CR refactors the
blockcutter tests to use ginkgo.  This simplifies the upcoming metrics
CR because of the shared setup across the tests which would have made
the previous test approach balloon in size and repetition.

Change-Id: I0951bf16766e3a24ced2631cf3fbc425ea4ce8a3
Signed-off-by: Jason Yellick <jyellick@us.ibm.com>

Make linter is failing with an error against:

 core/ledger/kvledger/txmgmt/txmgr/lockbasedtxmgr/txmgr_test.go:1272

This CR simply fixes it.

Change-Id: Icaf80ba46ca81659e4f2f233ebbe04ae73302c9e
Signed-off-by: Jason Yellick <jyellick@us.ibm.com>

channel_name --> channel
chaincode_name --> chaincode

Use "unknown" as the sentinel for an unknown label value.

Change-Id: I03a5abaae7b84e228fb809a3559c179a04f64caf
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Address linter errors and fix bug where check of error result while
reading metadata was omitted.

Change-Id: I1bf87c6da5273c26916ca735b32ae0b7dd4bad8c
Signed-off-by: Artem Barger <bartem@il.ibm.com>