Code improvements PoA module #3

[ishantiw]

Description

1. The getAllValidatorsResponseSchema schema specifies that weights should have a 'uint64' format, while the implementation may assign them to be blank strings: This is not a security issue, since there is no formal definition of the 'uint64' format anyway, and since the response is never validated against the schema
2. Unnecessary store lookup: In the PoAEndpoint.getAllValidators function, validatorStore.iterate is used to get a list of validator (address, name) pairs. However, later on, the addresses in the returned list are again looked up using validatorStore.get(context, data.key), when data.value could be used instead.
3. Unnecessary call to getAddressFromPublicKey: The register authority command uses address.getAddressFromPublicKey(context.transaction.senderPublicK ey) in its verify function to get the transaction’s sender address, when it could instead use context.transaction.senderAddress.

Steps to reproduce

N/A

Which version(s) does this affect? (Environment, OS, etc...)

6.1.0-beta.1