Blockheight expiry strategy is accidentally conservative

[steveluscher]

mango-client-v3/src/client.ts

Lines 540 to 543 in 51c939c

	const timeoutBlockHeight = signedAtBlock
	? signedAtBlock.lastValidBlockHeight +
	MAXIMUM_NUMBER_OF_BLOCKS_FOR_TRANSACTION
	: 0;

Hey! I stumbled upon this line of code. I don't think this does what it's meant to do. lastValidBlockHeight is actually ‘the block height at which this recent blockhash will no longer be valid,’ so timeoutBlockHeight is just signedAtBlock.lastValidBlockHeight here.