Maybe get_raw_transaction should return Result<Option<Transaction>, Error>?

[Kixunil]

It may be more natural to return None instead of Err if the transaction wasn't found similarly to how HashMap returns None instead of Err. This is loosely related to romanz/electrs#446 (comment) - the case when transaction is not found may not be exactly erroneous and may need special handling more often than other errors.