-
Notifications
You must be signed in to change notification settings - Fork 134
Labels
Blockchain / DLTIssues engineering distributed ledger functionalityIssues engineering distributed ledger functionalityenhancementNew feature or requestNew feature or requestintermediaterequires some knowledge of the codebase with some defined steps to implement or examplesrequires some knowledge of the codebase with some defined steps to implement or examples
Description
The AccountID protobuf definition allows the alias field to contain either:
- an EVM address (20 bytes), or
- an alias public key (ED25519 or ECDSA)
Currently, the Python SDK does not reliably distinguish between these formats when decoding alias. This leads to inconsistencies with other SDKs (Java, JavaScript, etc.) and may cause confusion when working with accounts that use alias_key or EVM addresses.
Proposed Solutions:
- Properly decode alias as either a
public keyorEVM address - Add helper methods to populate missing account numbers or EVM addresses from the mirror node
- Add serialization helpers (to_bytes / from_bytes)
- Update from_string and add from_evm_address to simplify creation
- Make AccountId behavior consistent with other SDKs (Java, JavaScript, etc.)
Metadata
Metadata
Assignees
Labels
Blockchain / DLTIssues engineering distributed ledger functionalityIssues engineering distributed ledger functionalityenhancementNew feature or requestNew feature or requestintermediaterequires some knowledge of the codebase with some defined steps to implement or examplesrequires some knowledge of the codebase with some defined steps to implement or examples