Add timout for fetching peer id

[NikolasHaimerl]

Add timeout for getIndexProviderPeerId call

Description

We should add a timeout for the getIndexProviderPeerId call in backend/lib/resolve-payload-cids.js, similarly to what we had in the original implementation in spark-checker.

Current Implementation

return await getIndexProviderPeerId(
  `fo${minerId}`,
  smartContract,
  { rpcFn: makeRpcRequest }
)

Background

It seems we didn't configure any timeout in the original implementation in spark-deal-observer. This timeout is necessary to ensure the application doesn't hang indefinitely when making external calls.

Reference

Related to implementation in
#143
CheckerNetwork/spark-checker#129
CheckerNetwork/spark-spot-check#7

Proposed Solution

Add a timeout parameter to the getIndexProviderPeerId call similar to the original implementation in spark-checker.

Changes need to be done in

• index-provider-peer-id
• spark-checker
• spark-deal-observer --> feat: add timeout to get peer id #150
• spark-spot-check

[juliangruber]

Blocks CheckerNetwork/spark-checker#129
Blocks #143

[juliangruber]

@NikolasHaimerl I changed this issue type from Tech debt to Feature, as it's a new feature for this library (it wasn't supporting timeouts before) and something we need to address now, not eventually

[bajtos]

IMO:

• This is a Feature in spark-deal-observer
• It's a Bug in index-provider-peer-id because it was supposed to allow the spark-checker to preserve the existing timeout behaviour.

Please prioritise shipping Curio DDO support first.

I agree we should implement support for timeouts soon. Since @NikolasHaimerl has already started the work, I am adding this issue to the current milestone.