status-im
/
status-go
mirror of https://github.com/status-im/status-go.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fix: the validation of collectibles should not be dependent on capitalization (#4917)

This commit is contained in:
Mikhail Rogachev 2024-03-13 15:19:33 +01:00 committed by GitHub
parent 9db149d4f6
commit b987e8d6f5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
protocol/messenger_profile_showcase.go
View File

@ -9,6 +9,7 @@ import (
"math/big" "math/big"
"reflect" "reflect"
"sort" "sort"
"strings"
"github.com/golang/protobuf/proto" "github.com/golang/protobuf/proto"
"go.uber.org/zap" "go.uber.org/zap"
@ -73,10 +74,11 @@ func (m *Messenger) validateCollectiblesOwnership(accounts []*identity.ProfileSh
accountsMap := make(map[string]identity.ProfileShowcaseVisibility) accountsMap := make(map[string]identity.ProfileShowcaseVisibility)
for _, accountProfile := range accounts { for _, accountProfile := range accounts {
if _, ok := accountsMap[accountProfile.Address]; ok { addressCapitalized := strings.ToUpper(accountProfile.Address)
if _, ok := accountsMap[addressCapitalized]; ok {
return errorDublicateAccountAddress return errorDublicateAccountAddress
} }
accountsMap[accountProfile.Address] = accountProfile.ShowcaseVisibility accountsMap[addressCapitalized] = accountProfile.ShowcaseVisibility
} }
for _, collectibleProfile := range collectibles { for _, collectibleProfile := range collectibles {
@ -90,7 +92,8 @@ func (m *Messenger) validateCollectiblesOwnership(accounts []*identity.ProfileSh
// but ERC1155 which can be supported later can have more than one holder and balances > 1 // but ERC1155 which can be supported later can have more than one holder and balances > 1
found := false found := false
for _, balance := range balances { for _, balance := range balances {
if accountShowcaseVisibility, ok := accountsMap[balance.Address.String()]; ok { addressCapitalized := strings.ToUpper(balance.Address.String())
if accountShowcaseVisibility, ok := accountsMap[addressCapitalized]; ok {
if accountShowcaseVisibility < collectibleProfile.ShowcaseVisibility { if accountShowcaseVisibility < collectibleProfile.ShowcaseVisibility {
return errorAccountVisibilityLowerThanCollectible return errorAccountVisibilityLowerThanCollectible
} }