Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix exception when etherscan responds no status #22

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open

Fix exception when etherscan responds no status #22

wants to merge 36 commits into from

Conversation

qbonnard
Copy link

@qbonnard qbonnard commented Apr 5, 2021

An exception is thrown if etherscan's json response does not include the status field, e.g. on
https://api.etherscan.io/api?module=proxy&action=eth_getTransactionByHash&txhash=0x087c822fc9b75bc4894df309cc18820f3d77659f0bc115a7035de39461a456a3

This PR first checks for the presence of the status field before checking whether it's 1

Let me know if there is a better way to do this!

daoready and others added 30 commits November 30, 2018 00:34
@daoready
ABI tuple parser
b235be5
@daoready
https://github.com/Bloxy-info/web3-eth
8c4bb63
@daoready
fix event with tuple parse
c24a2f1
@astudnev
Structures of tuples
24e20f2
@astudnev
Structures of tuples
ef34f18
@astudnev
array of tuples ABI parse
8c0758c
@astudnev
fixed tuple structures parse
3988476
@astudnev
fixed tuple structures parse
4c1e8bc
@astudnev
fixed tuple structures parse
5724059
@astudnev
Constructor arguments parsed in trial cycle if can not detect automat…
2122429 
…ically
@astudnev
Fix for decoding wrong data set
e3419c5
@astudnev
fixes for TRON compatibility
85ac968
@astudnev
string[32] support
d3fbfb3
@astudnev
string[32] support
fdc9f47
@astudnev
trc token type
a22a93d
@astudnev
fix parser for dynamic types
132d967
@astudnev
fix parser for fixed types
84715b3
@astudnev
debug module added
fe63311
@astudnev
bulk simulation for geth method hash
5c8f85a
@astudnev
added properties for transaction
d7c586b
@astudnev
added properties for transaction
fb2f077
@astudnev
Celo blockchain does not have difficulty on web3
f50f170
@astudnev
suicide address field
9214ee9
@astudnev
suicide address field
1a67dbf
@astudnev
suicide address field
b98be95
@astudnev
Added timeout for debug_module tracing
7cbbed2
@astudnev
tracesByBlockNumber added
c645b5f
@astudnev
parity_module.rb module added
70144cf
@astudnev
web3.eth call used instead of direct name
9af1287
@astudnev
JSON level up to 1500
6aae6f7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@qbonnard @daoready @astudnev