Commit a22b4ed6 authored by Mark Tyneway's avatar Mark Tyneway Committed by GitHub

Merge pull request #2141 from ethereum-optimism/fix/dtl-context-api-error

dtl: handle errors in context api
parents 21806b62 8f72064d
---
'@eth-optimism/data-transport-layer': patch
---
Handle case where the remote block isn't found for `GET /eth/context/latest` and `GET /eth/context/blocknumber/:number`
......@@ -340,6 +340,9 @@ export class L1TransportServer extends BaseService<L1TransportServerOptions> {
const blockNumber = Math.max(0, tip - this.options.confirmations)
const block = await this.state.l1RpcProvider.getBlock(blockNumber)
if (block === null) {
throw new Error(`Cannot GET /eth/context/latest at ${blockNumber}`)
}
return {
blockNumber: block.number,
......@@ -366,6 +369,10 @@ export class L1TransportServer extends BaseService<L1TransportServerOptions> {
}
const block = await this.state.l1RpcProvider.getBlock(number)
if (block === null) {
throw new Error(`Cannot GET /eth/context/blocknumber/${number}`)
}
return {
blockNumber: block.number,
timestamp: block.timestamp,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment