230 Commits (86c06c17dc5963d51a78149fa5adc50c8cce2260)

Author	SHA1	Message	Date
Alex Garibay	76970f6258	Finish documentation and tests for request backoffs	6 years ago
zachdaniel	75c88fac85	WIP: test rolling window and request coordinator	6 years ago
zachdaniel	bd31796d40	WIP: Finish/clean up rolling window tracker	6 years ago
Alex Garibay	be7271a8d9	WIP: clean up request coordinator with functionality and docs	6 years ago
zachdaniel	9410a86605	WIP: application + request coordinator cleanup	6 years ago
zachdaniel	961a2fb03c	WIP: request coordinator + starting rolling window	6 years ago
zachdaniel	697be65e82	WIP: Rolling window event counter.	6 years ago
Alex Garibay	632981e16b	WIP: generalize rolling window	6 years ago
zachdaniel	0e3fdc3995	WIP: Add rolling window rate limiter + retry logic	6 years ago
Alex Garibay	1f021a7e1f	Provide default 2nd arg for executing contract functions on Geth (#970)	6 years ago
Konstantin Zolotarev	1674540779	Fix ping/pong from ganache ws implementation	6 years ago
Amanda Sposito	b86335cf34	Add 'transaction_index' in the fetchers to save at 'internal_transaction' ... Add 'transaction_index' in the fetchers to save at 'internal_transaction'	6 years ago
Sebastian Abondano	1cbaafd894	Adds log rotation ... Why: * To limit the total size of the logs retained while still allowing analysis of recent events. * Issue link: n/a This change addresses the need by: * Editing the logger backends to do file rotation every 50mb and keep the last 19 log files per logger backend. 50mb * 20 = ~1gb	6 years ago
Sebastian Abondano	2b8ceb4e32	Update block reward contract beneficiary balances ... Why: * "Parity’s consensus engine allows using a smart contract for block reward calculation." - see https://wiki.parity.io/Block-Reward-Contract for details. This means that we need to use `trace_block` (https://wiki.parity.io/JSONRPC-trace-module#trace_block) to fetch the block reward contract beneficiaries and update their balances. Currently this doesn't happen so there are addresses (block contract reward beneficiaries) with inaccurate balances and missing rows in the `address_coin_balances` table. * Issue link: https://github.com/poanetwork/blockscout/issues/767 This change addresses the need by: * Adding `EthereumJSONRPC.fetch_beneficiaries/2` to help us fetch the block reward contract beneficiaries of a given block range. This only works with Parity. * Editing Indexer's dev and prod config by adding `trace_block` config to `:method_to_url`. * Adding `block_reward_contract_beneficiaries` to `Indexer.AddressExtraction` for it to know how to get addresses in this scenario. * Editing `Indexer.Block.Fetcher.fetch_and_import_range/2` to get block reward contract beneficiaries, create their addresses if they don't yet exist, and update their coin balances.	6 years ago
Luke Imhoff	1100f56fa1	Test WebSocketClient with cowboy_websocket_handler ... Allows to properly simulate TCP disconnect.	6 years ago
Luke Imhoff	b1f4696569	Reconnect, rerequest, and resubscribe when websocket disconnects ... * Use struct, `%EthereumJSONRPC.WebSocket.WebSocketClient.Options{}` to hold `web_socket_options`. * `%EthereumJSONRPC.WebSocket.Registration{}` * Record `t:EthereumJSONRPC.request/0` in `%EthereumJSONRPC.WebSocket.Registration{}`, so that the request can be re-requested if the websocket disconnects and reconnects. * Drop server-side `subscription_id` from `%EthereumJSONRPC.WebSocket.Registration{}` as it can retrieved from `request` now. * `EthereumJSONRPC.WebSocket.WebSocketClient` * Track whether the web socket is connected or not, so that requests aren't sent when in the disconnected state. This prevents double-sends as `handle_call` can be called before `onconnect`, which caused that first subscribe to be sent twice. * Instead of the subscriber knowing the server-side subscription id, an opaque `t:reference/0` is used, so that the reference is stable in the subscriber's `%Subscription{}`, but the subscription ID can change on reconnect, which means replacing `subscription_id_to_subscription` in `%WebSocketClient{}` with 3 maps 1. `subscription_id_to_subscription_reference` - used translate server-side, per-connection subscription to the stable reference in `subscription_reference_to_subscription` 2. `subscription_reference_to_subscription_id` - used to convert stable reference to subscription id to unsubscribe. 3. `subscription_reference_to_subscription` - used to get full Subscription, including the subscriber_pid for when subscription messages are delivered or the subscription needs to be resubscribed using the `web_socket_options`. * `ondisconnect`, `:reconnect` immediately. * When disconnecting, tell unsubscribers they were successful. * Hold any json_rpc or subscribe requests to retry on reconnect. * Hold any subscriptions to resubscribe on reconnect. * `onconnect` * re-request the saved json_rpc and subscribe requests. * issue new requests to re-establish the subscriptions that had already succeeded in their subscribe requests. * `%EthereumJSONRPC.Subscription{}` * Replace `id`, which did match the server created ID, with a `t:reference/0`, which allows for indirection and for the server created ID to change on reconnect while the subscriber does not need to care about the reconnections. * `EthereumJSONRPC.WebSocket` * `options` is switched from a Keyword.t to a struct, `t`, so that it is easier to pattern match for the reconnect logic in `WebSocketClient`. * `web_socket_options` is no longer a generic `term` because the code did require it to be a map with a `:web_socket` key for the `t:GenServer.server/0`.	6 years ago
Luke Imhoff	32535bd73c	Simplify WebSocketClient.respond_to_registration ... All clauses set request_id_to_registration in state to new_request_id_to_registration, so it does not need to be passed. The catchall cause does the equivalent because new_request_id_to_registration was request_id_to_registration already as Map.pop popped nothing.	6 years ago
Konstantin Zolotarev	ff8053d656	fix formatting	6 years ago
Konstantin Zolotarev	3eee0a2944	Fix spacing	6 years ago
Konstantin Zolotarev	0604f7a234	Fixes for pr comments	6 years ago
Konstantin Zolotarev	f89a27adf4	Fixes for ganache integration	6 years ago
Nam Chu Hoai	d17917ce30	Allow unsecure WebSocket connections ... Some setup like ours will have the WebSocket connection in a private network to a local node and thus do not require a secure connection. This change allows to specify ws:// instead of wss:// endpoints as well as reading from the ENV var for the geth config of the indexer Co-authored-by: Luke Imhoff <luke.imhoff@dockyard.com>	6 years ago
Luke Imhoff	5d0c1fc18a	succucessfully -> successfully	6 years ago
Luke Imhoff	8da99addcb	JSONR-RPC -> JSON-RPC	6 years ago
Luke Imhoff	b93377cccd	chainId -> chain_id for elixir variable	6 years ago
Luke Imhoff	64bd973076	suscribed -> subscribed	6 years ago
Luke Imhoff	a4509ed42e	resultscan -> results can	6 years ago
Luke Imhoff	36c939f14a	Ethereums -> Ethereum	6 years ago
Luke Imhoff	fa1c9ac944	passsed -> passed	6 years ago
Luke Imhoff	076d12d905	quanity -> quantity	6 years ago
Luke Imhoff	6b53d96075	_optons -> _options	6 years ago
Luke Imhoff	7eddae586e	transfered -> transferred	6 years ago
Luke Imhoff	480300b542	Ethreums -> Ethereum	6 years ago
Luke Imhoff	6dbb511700	failes -> fails	6 years ago
Luke Imhoff	0b5729c144	correclty -> correctly	6 years ago
Luke Imhoff	e20c98949e	econded -> encoded	6 years ago
Luke Imhoff	33c12ca780	Ethererum -> Ethereum	6 years ago
Luke Imhoff	dc669423c2	peformed -> performed	6 years ago
Lokraan	92f99cac22	Ran mix format on code. ... Confirm to CircleCi tests. Was failing due to not updating gettext, javascript lint standards, and an accidental edit in `block_controller.ex` where I added `:block => :required` to `necessity_by_association`. I also updated the query in `Explorer.Chain.address_to_validation_count/1` in order to make it more functional (use built in Ecto queries instead of hand written SQL query.) Conform to code review comments. Instances of "Blocks #{gettext("Validated")}" -> gettext("Blocks Validated") The class for the validation link in the mobile menus was "nav-link" instead of "dropdown-item" so I changed it. Ran `mix gettext.extract --merge` in order to update the gettext("Blocks Validated") Conform to code review. Added `active` classes to the corresponding links on mobile navs. Removed redundant if statements when in the page that required the content. Created a function in `address_view.ex` called `validator?` that accepts the number of validations and returns true if greater than 0. Replaced `if @validation_count > 0` with `if validator?(@validation_count`. "Please add the active class to the dropdown item." Added the `active` class where the page was active on for mobile and desktop navs. Also replaced `nav-item` with `dropdown-item` on the mobile navs where it was necessary.	6 years ago
Luke Imhoff	2f7be3fb7d	OTP application descriptions ... The Application selector in Wobserver shows the descriptions, so it is useful to be set.	6 years ago
Luke Imhoff	96d5aea311	Block.SecondDegreeRelation ... The database table block_second_degree_relations contains links between the `uncle_hash` of the uncle and the `nephew_hash` of the block that said `uncle_hash` was an uncle. It also has a `uncle_fetched_at` that is `null` until the `uncle_hash` is confirmed as fetched in the database. This does not contain such a fetcher.	6 years ago
Luke Imhoff	48b302f935	Return uncles from EtherumJSONRPC.Block.elixir_to_params	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
Luke Imhoff	baa5746a25	Don't infer status for gas and gas_used ... The heuristic of gas == gas_used is an error has been proven to be incorrect with test transactions on Ethereum Classic, so remove it. When the status is not given it will be propagated up as `nil` and must be filled in by the caller using the internal transactions.	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
Luke Imhoff	c10b6c218b	Disable another test that times out on Infura	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
Luke Imhoff	ded4ec4246	Disable tests for Geth that timeout on Infura ... Infura experiencing timeouts with 50 second timeout set.	6 years ago
Luke Imhoff	b7d7ceaaa0	Temporary work-around for #673 value-only transactions ... The gas_used == gas heuristic means failures pre-Byzantium (and on ETC) is only a heuristic: it fails for value-only transactions where ETC submitters are using exactly the transaction base fee (21,000 gas) often. Because this is so common, we'll use this work-around now until we can restructure EthereumJSONRPC and Indexer to verify pre-Byzantium status using Internal Transactions.	6 years ago
Stamates	18380efe0a	fix broken tests ... Co-authored-by: Luke Imhoff <luke.imhoff@dockyard.com>	6 years ago