Skip to content

Fix/checksum address#24

Merged
poly-rodr merged 2 commits intomainfrom
fix/checksum-address
Jun 2, 2022
Merged

Fix/checksum address#24
poly-rodr merged 2 commits intomainfrom
fix/checksum-address

Conversation

@poly-rodr
Copy link
Contributor

Overview

Im getting this error on the arb keeper and I detected that the conditional address for polygon is not checksummed.

2022-06-02 17:44:33,236 ERROR Thread-2 Failed to fetch the order book (('Web3.py only accepts checksum addresses. The software that gave you this non-checksum address should be considered unsafe, please file it as a bug on their platform. Try using an ENS name instead. Or, if you must accept lower safety, use Web3.toChecksumAddress(lower_case_address).', '0x4d97dcd97ec945f40cf65f87097ace5ea0476045'))!

Description

Testing instructions

Types of changes

  • Refactor/enhancement
  • Bug fix/behavior correction
  • New feature
  • Breaking change
  • Other, additional

Notes

Status

  • Prefix PR title with [WIP] if necessary (changes not yet made).
  • Add tests to cover changes as needed.
  • Update documentation/changelog as needed.
  • Verify all tests run correctly in CI and pass.
  • Ready for review/merge.

@poly-rodr poly-rodr merged commit e7c0efe into main Jun 2, 2022
@poly-rodr poly-rodr deleted the fix/checksum-address branch June 2, 2022 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants