fix: allow only one backend connection at a time

[0x416e746f6e]

fixes #153

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
rollup-boost	⬜️ Ignored (Inspect)			Apr 9, 2025 0:24am

[0xForerunner]

This doesn't seem like the right solution to me? We have situations where we want to have multiple concurrent requests.

[0x416e746f6e]

We have situations where we want to have multiple concurrent requests.

what are those @0xForerunner?

[0xForerunner]

Right here for example

[0xForerunner]

Causes issues with concurrency.

[0x416e746f6e]

Right here for example

I was more looking to hear about use-cases when concurrency is needed, rather than places where it's being used because it can be used.

op-node typically opens exactly 2 connections with it's EL authrpc:

• no. 1 is used for FCU, getPayload, newPayload, and getBlockByHash calls
• no. 2 is used for getBlockByNumber

there is no concurrency there. why suddenly we need it in rollup-boost?

[0xOsiris]

Right here for example

I was more looking to hear about use-cases when concurrency is needed, rather than places where it's being used because it can be used.

op-node typically opens exactly 2 connections with it's EL authrpc:

• no. 1 is used for FCU, getPayload, newPayload, and getBlockByHash calls
• no. 2 is used for getBlockByNumber

there is no concurrency there. why suddenly we need it in rollup-boost?

There are certainly cases where we are going to have an open connection, and we cannot drop the subsequent connection. Not all FCU's are guaranteed to come in at 2s intervals. For example when op-node is backfilling deposit blocks to catch up to the L1 block time. FCU/Get Payload requests are going to be coming in on millisecond time horizon's, but not dropping this connections is critically important.

In any case we never want to drop a new connection just because we currently have an open one even if we shouldn't be holding on to the connection in the first place. This would break liveness