TrueCloudLab/frostfs-contract
22
0
Fork 19
Code Issues 12 Pull requests 1 Projects Releases 2 Activity Actions

[#55] policy: Swap signature check order

Browse source 
While implementing the changes for FrostFS ID it became obvious, that
committee signature check can be rather costly (`getCommittee` call +
multiaddress construction + checking witness). In real scenarious it
will mostly fail, so it makes sense do it last.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2023-11-28 11:51:24 +03:00
parent 0dda536d4a
commit f30fb324ff
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
policy/policy_contract.go
View file

@ -47,14 +47,13 @@ func _deploy(data any, isUpdate bool) {
} }
func checkAuthorization(ctx storage.Context) { func checkAuthorization(ctx storage.Context) {
if runtime.CheckWitness(common.AlphabetAddress()) {
return
}
admin := getAdmin(ctx) admin := getAdmin(ctx)
if admin != nil && runtime.CheckWitness(admin) { if admin != nil && runtime.CheckWitness(admin) {
return return
} }
if runtime.CheckWitness(common.AlphabetAddress()) {
return
}
panic(ErrNotAutorized) panic(ErrNotAutorized)
} }