fix: transaction not found on blockscout #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hoanguyenkh
reviewed
Oct 27, 2022
| handler.logger.Errorf("error parsing tx response from blockscout: %v", err) | ||
| httpapi.InternalServerError(ctx) | ||
| return | ||
| if strings.Contains(fmt.Sprint(err), "transaction not found") { |
There was a problem hiding this comment.
need move "transaction not found" to constant value
hoanguyenkh
added a commit
that referenced
this pull request
Nov 23, 2022
* init project * update git ignore * update index * update config * chore: update gitignore * refactor: update chain indexing * chore: update config * fix: sync block * bump chain-indexing * update config * remove nft api * bump chain-indexing * bump chain-indexing * bump indexing add concurrency get txs * bump indexing fix get tx error * bump indexing fix get tx error * remove chainstats * add http retry * bump version chain-indexing * bump version chain-indexing * bump version chain-indexing * bump version chain-indexing * bump version chain-indexing * bump version chain-indexing * bump version chain-indexing * fix validator view * add debug validator * add debug validator * move chain indexing to astra indexing (#1) * move chain indexing to astra indexing * add metrics block height * fix get validators * add record param * update go-http-metrics change handler detail request * bump version go-http metrics * off http debug * add cache in block pagination * Update cache (#3) * refactor cache block * add cache transaction * update cache size to 10MB * refactor cache size and pagination * update cache size * update cache * update log cache * refactor cache * add metrics cache * change cache metrics name * change cache metrics name * Feat get detail evm tx (#2) * feat: implemet get detail evm txs * refactor: merge FindEvmByHash into FindByHash api * refactor: * add blockscout url_handler * handle query param ?type=evm * Merge branch 'dev' into feat--get-detail-evm-tx * refactor: * impl blockscout httpclient * remove blockscout url handler * fix: missing cache param * add get Accounts and refactor BlockscoutApp * fix msg create clawback vesting * fix miss MsgEthereumTx in account transaction * add convert from eth address to astra address * add convert from eth address to astra address * add log debug * fix check hexString * fix parser from * refactor: add more info to TransactionEvm * refactor: - change blockscout url - add CreatedContractAddressHash to TransactionEvm * fix: transaction not found on blockscout (#4) * fix: transaction not found on blockscout * refactor: remove duplicate code * feat: (#5) merge blockscout and chainindexing search api * Add evm type (#6) * add evmType * fix account transaction * fix get signature data * feat: improve search api (#8) * feat: improve search api * refactor: remove unused code * refactor: add logger to blockscout httpclient * refactor: refactor and add comment for search * chore: fix typo * chore: remove unused code * refactor search api * chore: add TokenId field to detail_tx_response * chore: add TokenType field * feat: add Msg exec (#9) * chore: refactor convention * feat: add handler tx msgExec * Fix transaction failed and not found on evm rpc (#10) * feat: parse evmTxHash and insert to database * update migration script * chore: remove comment * add find a transaction by evm hash * fix: missing method * handle get detail tx by evm tx hash * chore: add log * fix: cannot parse tx message * refactor: using typedEvent to get evmTxHash * fix: cannot search failed evm transaction * add Dockerfile * add tikici * Change config to test CI Co-authored-by: hoanguyenkh <hoa.nguyen2@tiki.vn> Co-authored-by: Khanh Hoa <49144992+hoanguyenkh@users.noreply.github.com> Co-authored-by: trung-nt3 <59964019+trung-nt3@users.noreply.github.com>
trungnt1811
deleted the
fix--transactions-not-found-on-blockscout-but-exists-in-astra-indexing
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.