refactoring (#50)

VirgilSecurity · Jun 12, 2020 · a621ff3 · a621ff3
1 parent 4944e5d
commit a621ff3
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 19 deletions.
diff --git a/sdk/cardverifier.go b/sdk/cardverifier.go
@@ -70,9 +70,9 @@ func VirgilCardVerifierDisableVirgilSignature() VirgilCardVerifierOption {
 	}
 }
 
-func VirgilCardVerifierAddWhitelist(wl Whitelist) VirgilCardVerifierOption {
+func VirgilCardVerifierAddAllowList(wl *AllowList) VirgilCardVerifierOption {
 	return func(v *VirgilCardVerifier) {
-		v.whitelists = append(v.whitelists, wl)
+		v.allowLists = append(v.allowLists, wl)
 	}
 }
 
@@ -86,7 +86,7 @@ type VirgilCardVerifier struct {
 	crypto                *CardCrypto
 	verifySelfSignature   bool
 	verifyVirgilSignature bool
-	whitelists            []Whitelist
+	allowLists            []*AllowList
 	virgilPublicKey       crypto.PublicKey
 
 	// virgilPublicKeySource is used to update Virgil Cards service public key
@@ -137,15 +137,15 @@ func (v *VirgilCardVerifier) VerifyCard(card *Card) error {
 			return errors.NewSDKError(err, "action", "VirgilCardVerifier.VerifyCard", "validate", "virgil")
 		}
 	}
-	return v.verifyCardByWhitelist(card)
+	return v.verifyCardByAllowList(card)
 }
 
-func (v *VirgilCardVerifier) verifyCardByWhitelist(card *Card) error {
-	for _, whitelist := range v.whitelists {
+func (v *VirgilCardVerifier) verifyCardByAllowList(card *Card) error {
+	for _, allowList := range v.allowLists {
 		signatureVerified := false
 		var err error
-		for i := range whitelist.VerifierCredentials {
-			var cred = whitelist.VerifierCredentials[i]
+		for i := range allowList.VerifierCredentials {
+			var cred = allowList.VerifierCredentials[i]
 			if err = v.ValidateSignerSignature(card, cred.Signer, cred.PublicKey); err != nil {
 				continue
 			}
@@ -181,12 +181,12 @@ func (v *VirgilCardVerifier) ValidateSignerSignature(card *Card, signer string,
 	return ErrSignerWasNotFound
 }
 
-type Whitelist struct {
+type AllowList struct {
 	VerifierCredentials []*VerifierCredentials
 }
 
-func NewWhitelist(credentials ...*VerifierCredentials) Whitelist {
-	return Whitelist{
+func NewAllowList(credentials ...*VerifierCredentials) *AllowList {
+	return &AllowList{
 		VerifierCredentials: credentials,
 	}
 }

diff --git a/sdk/cardverifier_test.go b/sdk/cardverifier_test.go
@@ -53,7 +53,7 @@ type testCredentials struct {
 	PrivateKey          crypto.PrivateKey
 }
 
-func TestWhitelist(t *testing.T) {
+func TestAllowList(t *testing.T) {
 	const credsCount = 5
 	creds := make([]testCredentials, 5)
 	for i := 0; i < credsCount; i++ {
@@ -84,8 +84,8 @@ func TestWhitelist(t *testing.T) {
 	verifier := NewVirgilCardVerifier(
 		VirgilCardVerifierSetCrypto(cryptoNative),
 		VirgilCardVerifierDisableVirgilSignature(),
-		VirgilCardVerifierAddWhitelist(NewWhitelist(creds[0].VerifierCredentials, creds[1].VerifierCredentials)),
-		VirgilCardVerifierAddWhitelist(NewWhitelist(creds[2].VerifierCredentials)),
+		VirgilCardVerifierAddAllowList(NewAllowList(creds[0].VerifierCredentials, creds[1].VerifierCredentials)),
+		VirgilCardVerifierAddAllowList(NewAllowList(creds[2].VerifierCredentials)),
 	)
 
 	card, err := ParseRawCard(cryptoNative, model, false)
@@ -95,24 +95,24 @@ func TestWhitelist(t *testing.T) {
 	err = verifier.VerifyCard(card)
 	require.NoError(t, err)
 
-	//check that everything is ok if at least one signature in whitelist is valid
+	//check that everything is ok if at least one signature in allow list is valid
 	// creds[4] doesn't exist
 	verifier = NewVirgilCardVerifier(
 		VirgilCardVerifierSetCrypto(cryptoNative),
 		VirgilCardVerifierDisableVirgilSignature(),
-		VirgilCardVerifierAddWhitelist(NewWhitelist(creds[4].VerifierCredentials, creds[1].VerifierCredentials)),
+		VirgilCardVerifierAddAllowList(NewAllowList(creds[4].VerifierCredentials, creds[1].VerifierCredentials)),
 	)
 
 	err = verifier.VerifyCard(card)
 	require.NoError(t, err)
 
-	//Check that verification fails if no signature exists for whitelist
+	//Check that verification fails if no signature exists for allow list
 	// creds[3] doesn't exist
 	verifier = NewVirgilCardVerifier(
 		VirgilCardVerifierSetCrypto(cryptoNative),
 		VirgilCardVerifierDisableVirgilSignature(),
-		VirgilCardVerifierAddWhitelist(NewWhitelist(creds[2].VerifierCredentials)),
-		VirgilCardVerifierAddWhitelist(NewWhitelist(creds[3].VerifierCredentials)),
+		VirgilCardVerifierAddAllowList(NewAllowList(creds[2].VerifierCredentials)),
+		VirgilCardVerifierAddAllowList(NewAllowList(creds[3].VerifierCredentials)),
 	)
 
 	err = verifier.VerifyCard(card)