blockscout

Commit Graph

Author	SHA1	Message	Date
Sebastian Abondano	6be5bc81b7	Derive status from internal transaction at index 0 Why: * Issue #757 was created after we noticed a few constraint violations for the `error` column in the `transactions` table. The constraint violations came about as we attempted to update transactions with a successful status but some value for the error field. The constraint is violated in this scenario because we expect successful transactions to not have an error. [EIP 658](https://github.com/ethereum/EIPs/pull/658), which apparently is also called [EIP 98](https://github.com/ethereum/EIPs/issues/98#issuecomment-318670322) , says that the status should be set to 1 (success) if the outermost code execution succeeded (internal transaction with index 0), or it should be set to 0 (failure) if the outermost code execution failed. We're currently deriving the status from the last internal transaction when in fact we should be deriving it from the first internal transaction. * Issue link: https://github.com/poanetwork/blockscout/issues/757 This change addresses the need by: * Editing `Explorer.Chain.Import.update_transactions/2` to set the error for a transaction equal to the error, if any, of the internal transaction with index 0. * Editing `Explorer.Chain.Import.update_transactions/2` to set a transaction's status as successful if it's internal transaction at index 0 doesn't have an error, and as failed if it does. This only applies to pre-Byzantium and Ethereum Classic, for which transaction status is not set from receipts.	6 years ago
Stamates	e000921433	Remove FakeAdapter for count estimation and just assert that an integer is returned from count estimation functions.	6 years ago
Stamates	cfac04336d	Remove pagination and return top 250 addresses by balance	6 years ago
Stamates	6ee50a6993	Estimated address count of addresses with balance > 0 using EXPLAIN rows estimate	6 years ago
Stamates	c060b86527	Wireup top_addresses page	6 years ago
Stamates	af6976317d	Use EXPLAIN in count query to speed up and get estimated row results	6 years ago
Stamates	4598eba91f	Remove transaction table joins from address_transaction_count query and count only distinct hashes	6 years ago
Tim Mecklem	d5d039a78d	Add address transaction count query back and include token transfers	6 years ago
Tim Mecklem	285fa9fa1e	Run transaction fetch queries separately per address_field This results in a much better query plan that takes advantage of some multi-column indexes coming soon to a GitHub near you.	6 years ago
Stamates	c515d765c1	Split address transactions query into separate query/fetches and combine the results due to postgres not selecting optimized query plan consistently	6 years ago
jimmay5469	a321b90878	Add full details to internal transaction tiles Co-authored-by: Stamates <stamates@hotmail.com>	6 years ago
Sebastian Abondano	8da1345d51	Add transaction#getstatus API endpoint Why: * For API users to be able to get the error for a given transaction hash. Example usage: ``` /api?module=transaction&action=getstatus&txhash={transactionHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/138 This change addresses the need by: * Adding `Explorer.Etherscan.get_transaction_error/1` to get the error for a given transaction hash. * Adding `API.RPC.TransactionController.getstatus/2` action to process requests made to the `transaction#getstatus` API endpoint. * Editing `API.RPC.TransactionView` to render transaction errors ('status') as required. * Adding `transaction#getstatus` API endpoint to the API docs page. Docs data lives in `BlockScoutWeb.Etherscan`.	6 years ago
Luke Imhoff	b92c8511f5	Show transaction error in UI	6 years ago
Luke Imhoff	102f467948	Denormalize last internal transaction error to transaction Allow UI and APIs to get the overall transaction error easily when status is `:error` without have to preload all the internal transactions.	6 years ago
Luke Imhoff	b32f263eda	Allow status to be nil until internal transactions are indexed For pre-Byzantium or Ethereum Classic (which never adopted EIP-658), the status in receipts will be `nil`, but we still want to ensure the status is eventually set, so when setting internal transactions, in addition to setting `internal_transactions_indexed_at` and `created_contract_address_hash`, the `status` will be updated. * If the `status` is already set, leave it as is * If the `status` is not set * If the last internal transaction for a transaction has an error, `status` is `:error` * If the last internal transaction for a transaction has no error, `status` is `:ok`	6 years ago
Delweng Zheng	de1a101c8c	fix(apps): adjust gas_used to Decimal	6 years ago
Amanda Sposito	3244113be4	Increase timeout at the `Repo.transaction` level at the 'Indexer.TokenBalance.Fetcher'	6 years ago
Felipe Renan	53cb6de504	Handle multiple values when reading Smart Contract functions We received the following error when a function executed by the contract returned multiple values: ``` an exception was raised: ** (FunctionClauseError) no function clause matching in Explorer.SmartContract.Reader.new_value (explorer) lib/explorer/smart_contract/reader.ex:272: Explorer.SmartContract.Reader.new_value( %{"name" => "narcoName", "type" => "string"}, ["Justin Sun", 0, 0, [8, 6, 9, 2, 2, 37], [0, 0, 0, 0], "m0310100904381635021206170608010621080706", 6, 33, [1533368345, 1533331805, 1533339540, 1533333875, 0, 0], 277, [72, 29, 59, 0, 1, 0, 2, 73, 0] ] ) (explorer) lib/explorer/smart_contract/reader.ex:268: anonymous fn/3 in Explorer.SmartContract.Reader.link_outputs_and_values/3 ``` As the returned values match with the ABI outputs, we are relying on output's index to fetch the correct value for each output. This commit also fixes the template to deal with different results, such as: `[8, 6, 9, 2, 2, 37]`, that is an array of numbers but the Phoenix view considers it as a char-list.	6 years ago
Sebastian Abondano	ed66f9f87e	Orders account#getminedblocks by block number Why: * To match Etherscan's API we want to ensure blocks returned by the `account#getminedblocks` API endpoint are ordered by descending block number. We noticed this wasn't the case when looking into a couple of flaky tests on CI. This commit should also make sure the tests don't randomly fail any more. These are the two tests: - test/block_scout_web/controllers/api/rpc/address_controller_test.exs:1674 - test/explorer/etherscan_test.exs:1014 * Issue link: n/a This change addresses the need by: * Editing `Explorer.Etherscan.list_blocks/1` to order blocks by descending block number. * Related tests were edited according to the change above.	6 years ago
Alex Garibay	0a922bb1a3	Display primary address name when available (#610 ) * Display primary address name in overview tile * Display primary address name in links Co-authored-by: Luke Imhoff <Kronic.Deth@gmail.com>	6 years ago
Gustavo Santos Ferreira	605d28cb4d	preload token in queries that need it	6 years ago
Igor Florian	d460fb8d50	Deal with :bad_gateway errors while querying contracts functions We were counting on a specific error standard from ethereum_jsonrpc for formatting the errors from the SmartContract.Reader and we ended up getting an exception because we were getting a different error. There are at least two different error standards returned from ethereum_jsonrpc: 1. ```elixir { :ok, [ %{ id: id, jsonrpc: "2.0", error: %{ code: "12345", message: "<message>" } } ] } ``` 2. ```elixir {:error, {:bad_gateway, "<url>"}} ``` We're now expecting the :bad_gateway error so we can format it properly.	6 years ago
Felipe Renan	d9ff36c00d	Fix token balances' query This query was considering the next-to-last token balance when the last one had the value 0. In order to this work properly, I'm moving the query that gets token balances from the last block to a sub-query and moving the condition to consider just values greater than 0 to the main query. This way we are sure that we always are considering the last block and we are skipping the ones that have the value equal to 0.	6 years ago
Felipe Renan	d0766c2017	Rafactor to count the holders from token balances instead of token transfers	6 years ago
Felipe Renan	b856eed594	Build query to fetch the holders	6 years ago
Sebastian Abondano	a372acaa1d	Add contract#getabi API endpoint Why: * For API users to be able to get the ABI for a given contract address. Example usage: ``` /api?module=contract&action=getabi&address={addressHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/138 This change addresses the need by: * Editing router to support `contract#getabi` API endpoint. * Creating `API.RPC.ContractController.getabi/2` action to process requests to `contract#getabi` endpoint. * Creating `API.RPC.ContractView` to render `contract#getabi` responses. * Adding documentation data for the new `contract#getabi` API endpoint. Documentation data lives in `BlockScoutWeb.Etherscan`	6 years ago
Lucas Narciso	1942248670	Add a "Tokens" tab on the Address page - Add the route and the controller; - Add the overview and the token list html; - Sort tokens by type and then number of transfers; - Support pagination.	6 years ago
Lucas Narciso	46998b5afb	Create query to count transfers This query needs a token and an address, and will return how many times that address has interacted with that token. Also, fix some minor indentation issues and typos on chain_test.	6 years ago
Sebastian Abondano	a07998547f	API account#txlist filterby=from support Why: * For API users to be able to get a list of transactions in which a given address is the 'sender' (address is the 'from' in transaction). Example usage: ``` /api?module=account&action&txlist&address={someAddress}&filterby=from ``` * Issue link: https://github.com/poanetwork/blockscout/issues/635 This change addresses the need by: * Editing `where_address_match/3` in `Explorer.Etherscan` to support `filter_by: "from"` option. * Editing `put_filter_by/2` in `API.RPC.AddressController` to support 'from' as an allowed value of the `filterby` parameter. * Editing API docs page per changes mentioned above.	6 years ago
Stamates	4e05488136	Send internal_transaction ids on Pub-Sub and Repo fetch with preloads of internal_transaction in Notifier Co-authored-by: Timothy Mecklem <timothy@mecklem.com>	6 years ago
Stamates	3f571aa4bb	Live updates for internal_transactions on address page Co-authored-by: Tim Mecklem <timothy@mecklem.com>	6 years ago
Sebastian Abondano	152b2246ab	Add account#tokenbalance API endpoint Why: * For API users to be able to fetch the token balance for given contract and address hash. Example usage: ``` /api?module=account&action=tokenbalance&contractaddress={contractAddressHash}&address={addressHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/571 This change addresses the need by: * Creating `Explorer.Etherscan.get_token_balance/2` to fetch the latest token balance for a given contract address and address hash. * Creating `API.RPC.AddressController.tokenbalance/2` action to process requests to `account#tokenbalance` API endpoint. * Editing `API.RPC.AddressView` to render token balances as needed. * Adding documentation data for the new API endpoint mentioned above. Documentation data lives in `BlockScoutWeb.Etherscan`.	6 years ago
Igor Florian	7077a41576	Change Address fetched_balance's columns to fetched_coin_balace	6 years ago
Igor Florian	acf14d6cf3	Rename Balance references to Address.CoinBalance	6 years ago
Igor Florian	56f578717f	Rename table and indexes	6 years ago
Stamates	ae47e8f6dc	Token transfers shown for all transactions tiles (preload not working in chain queries)	6 years ago
Sebastian Abondano	24e5da232f	Add filterby flag to account#txlist Why: * For API users to get a list of transactions in which the given address is the 'to' in the transaction. Example usage: ``` /api?module=account&action&txlist&address={someAddress}&filterby=to ``` * Issue link: https://github.com/poanetwork/blockscout/issues/138 This change addresses the need by: * Editing `API.RPC.AddressController`'s `txlist/2` action to support a `filterby` flag in which the only allowed value for now is `to`. This is an optional parameter. When given this filters the returned transactions to only those in which the given address matches the `to` field. When not given, the address provided could match the to, from, or contract address for the transaction. * Editing `Explorer.Etherscan`'s `list_transactions/3` to support the new `filterby` flag mentioned above. * Editing API docs data in `BlockScoutWeb.Etherscan` to include information about the new `filterby` flag added in this commit.	6 years ago
Sebastian Abondano	0b42ed19f4	Add account#getminedblocks API endpoint Why: * For API users to be able to get a list of blocks mined by a given address. Example usage: ``` /api?module=account&action=getminedblocks&address={addressHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/138 This change addresses the need by: * Adding `list_blocks/2` to `Explorer.Etherscan` to be get blocks as required by the new API endpoint added in this commit. * Adding `getminedblocks/2` action to `API.RPC.AddressController` to handle `account#getminedblocks` API requests. * Editing API.RPC.AddressView to render blocks as needed. * Adding docs for `account#getminedblocks` API endpoint. API docs live in `BlockScoutWeb.Etherscan`	6 years ago
Felipe Renan	74be090451	Remove unnecessary code in Token.BalanceReader Once we are fetching the token balances from the database, this code doesn't make sense anymore.	6 years ago
Felipe Renan	0c05600126	Add query to fetch the last token balances	6 years ago
Felipe Renan	d7bfcd1e87	Change Address.TokenBalance value to be decimal It was being created as WEI, but the correct type is decimal such as we are doing with Token and Token Transfers.	6 years ago
Felipe Renan	ad50a21c1e	Fix dialyzer	6 years ago
Igor Florian	53e7866016	Fetch TokenBalances value in the Catchup Indexer	6 years ago
Igor Florian	c2ff17756c	Get TokensBalance in the Blockchain We're now able to get the balances from a Address given it's address_hash, block_number and token_contract_address_hash.	6 years ago
Luke Imhoff	80a9170ebb	Port on_conflict from Balance to TokenBalance	6 years ago
Luke Imhoff	7eeb3226e2	Correct uniqueness of token balances Token balances, unlike balances, use 3 fields for uniqueness because 1 address can have a separate value for EACH token_contract_address_hash at EACH block_number.	6 years ago
Sebastian Abondano	1867d16417	Add account#tokentx API endpoint Why: * For API users to be able to get a list of token transfer events by address. A contract address can also be given, in addition to an address. Optional params include: 'startblock', 'endblock', 'sort', 'page', and 'offset'. Example usage: ``` /api?module=account&action=tokentx&address={addressHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/571 This change addresses the need by: * Adding `Explorer.Etherscan.list_token_transfers/3` to get token transfer data required from db. * Adding `tokentx/2` action to `API.RPC.AddressController` to handle `?module=account&action=tokentx&...` API requests. * Editing `API.RPC.AddressView` to render token transfer events as needed. * Deleting duplicate line in `Explorer.Chain.TokenTransfer`. This looks like a typo. * Other than adding tests for the new functions mentioned above, `optional_params/1` within `API.RPC.AddressController` was promoted to a public function and unit tests were added for it. * Adding docs for `account#tokentx` API endpoint. Documentation data lives in `BlockScoutWeb.Etherscan`.	6 years ago
Luke Imhoff	c6458e7272	Include transaction_hashes when update_transactions excepts	6 years ago
Alex Garibay	cade255e15	Add support for named addresses (#590 )	6 years ago
Igor Florian	b05196df0f	Create records for TokenBalances in the database The TokenBalances will be built in the BlockFetcher and will be stored with value and value_fetch_at as nil. When the TokenBalanceFetcher is plugued this info will be fetched and updated.	6 years ago

... 18 19 20 21 22 ...

1589 Commits (dd03109511d77b36b820f873022ea351beadabc9)