Transactions are not checked for nLocktime and inputs are not checked for nSequence

[sirdeggen]

arc/validator/default/default_validator.go

Line 158 in 2643ccb

/* lots of our valid test transactions have this sequence number, is this not allowed?

This should check to ensure inputs are all final only if the nLocktime is being enforced. The TODO suggests that this check was raising issues during development.

I think that it should be fine to invert this. Only pass if nSequence are all final. Otherwise we have to check nLocktime, but we don't know the height we're at in terms of block without checking blockTx which complicates things.

I would suggest we add status context rather than erroring out. If the nSequence numbers are lower than max - just noting that it might be the case that this transaction does not get mined until the nLocktime value passes.

Discussion:

[sirdeggen]

Ongoing discussion within BSVA regarding how Teranode will handle locktime and sequence. This will inform the direction of this issue.