timeout is not working with providers

[hammersharkfish]

Web3 provider timeout is getting ignored .
In my dapp I have set timeout to 5000 . web3.eth.getBlockNumber() doesn't throw even when public node takes more than 5s to respond .

Minimal example

var {Web3} = require('web3');
var web3 = new Web3(new Web3.providers.HttpProvider("https://evm.confluxrpc.com", {timeout: 0 })); 
async function getNewBlock (){ 
    newBlock = await web3.eth.getBlockNumber();
    return newBlock
    
}
getNewBlock().then((nb)=>{ 
    console.log(" SHOULDN'T BE PRINTING THE BLOCK BUT IT IS: ",nb," block number")
})

Expected behavior

innerError: { code: -32000, message: 'failed after 0 retries: timeout' },

OUTPUT

SHOULDN'T BE PRINTING THE BLOCK BUT IT IS: 72889142n block number

Environment

Windows
Node.js v19.0.0.
web3 8.19.2

[Muhammad-Altabba]

Thanks @hammersharkfish for opening this issue.
What is the version of web3 you are using? Seems there is a typo because there is no version 8.19.2.

[hammersharkfish]

My bad web3@4.0.1

[Muhammad-Altabba]

Thanks @hammersharkfish,

It seems this is a missed feature in version 4.x that was available in version 1.x.

Feel free to implement it if you like to.

And in the meantime, till this feature is added, you can use Promise.race with something like:

const blockNumber = await Promise.race([
    getNewBlock(),
    new Promise(function(resolve, reject) {
        setTimeout(function() {
            reject("Timeout");
        }, 5000);
    })
]);

[hammersharkfish]

Please assign the issue to me .

[shubhamshd]

Hi @hammersharkfish @Muhammad-Altabba is this issue still open ??
If yes, would you allow me to give it a try??

[hammersharkfish]

@shubhamshd I have written the code for the timeout already. I have run into installing dependencies using wsl so can't test it . #6228 .

[shubhamshd]

ok cool thanks @hammersharkfish!!!

[jdevcs]

@hammersharkfish can you share link to PR? if you have any issue, one of our team member can look that.