Skip to content

Commit

Permalink
Merge pull request #50 from levonyak/PAN-2254-remove-salt-from-eip712…
Browse files Browse the repository at this point in the history
…-domain

[PAN-2254] Remove salt from EIP-712 domain
  • Loading branch information
markuslevonyak authored Oct 10, 2024
2 parents 43d222d + 3d0a21c commit d9c6024
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 1 addition & 3 deletions pantos/client/library/blockchains/ethereum.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
from pantos.client.library.constants import TOKEN_SYMBOL_PAN

_EIP712_DOMAIN_NAME = 'Pantos'
_EIP712_DOMAIN_SALT = b''

_TRANSFER_MESSAGE_TYPES = {
'TransferRequest': [{
Expand Down Expand Up @@ -351,8 +350,7 @@ def __get_eip712_domain_data(self) -> dict[str, typing.Any]:
'name': _EIP712_DOMAIN_NAME,
'version': str(self.protocol_version.major),
'chainId': self._get_config()['chain_id'],
'verifyingContract': self._get_config()['forwarder'],
'salt': _EIP712_DOMAIN_SALT
'verifyingContract': self._get_config()['forwarder']
}

def __get_transfer_message_data(
Expand Down
4 changes: 1 addition & 3 deletions tests/blockchains/test_ethereum.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
from pantos.common.blockchains.enums import Blockchain

from pantos.client.library.blockchains.ethereum import _EIP712_DOMAIN_NAME
from pantos.client.library.blockchains.ethereum import _EIP712_DOMAIN_SALT
from pantos.client.library.blockchains.ethereum import \
_TRANSFER_FROM_MESSAGE_TYPES
from pantos.client.library.blockchains.ethereum import _TRANSFER_MESSAGE_TYPES
Expand All @@ -30,8 +29,7 @@ def eip712_domain_data(protocol_version, chain_id, forwarder_address):
'name': _EIP712_DOMAIN_NAME,
'version': str(protocol_version.major),
'chainId': chain_id,
'verifyingContract': forwarder_address,
'salt': _EIP712_DOMAIN_SALT
'verifyingContract': forwarder_address
}


Expand Down

0 comments on commit d9c6024

Please sign in to comment.