obelisk

commit b4ba13a19e7f5a9f57b537d0139f365a76b78147
parent ce54e9ae156406cfa92a4589e390ce428586eec2
Author: parazyd <parazyd@dyne.org>
Date:   Thu, 15 Apr 2021 22:21:25 +0200

Clean up some TODOs.

Diffstat:
M
obelisk/protocol.py
 | 
7
++-----

1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/obelisk/protocol.py b/obelisk/protocol.py
@@ -298,7 +298,7 @@ class ElectrumProtocol(asyncio.Protocol):  # pylint: disable=R0904,R0902
         return {"result": resp}
 
         # The assumption is to fetch more headers if necessary.
-        # TODO: Review, help needed
+        # TODO: Review everything below, help needed
         return JsonRPCError.invalidrequest()
         if cp_height > 0 and cp_height - start_height > count:
             for i in range(cp_height - start_height):
@@ -310,14 +310,11 @@ class ElectrumProtocol(asyncio.Protocol):  # pylint: disable=R0904,R0902
                 headers.extend(data)
 
         if cp_height > 0:
-            # TODO: Review
-            # TODO: Is index is 0 or last elem?
             hdr_lst = [headers[i:i + 80] for i in range(0, len(headers), 80)]
             branch, root = merkle_branch_and_root(hdr_lst, 0)
             resp["branch"] = [safe_hexlify(i) for i in branch]
             resp["root"] = safe_hexlify(root)
 
-
     async def blockchain_estimatefee(self, writer, query):  # pylint: disable=W0613
         """Method: blockchain.estimatefee
         Return the estimated transaction fee per kilobyte for a transaction
@@ -424,7 +421,7 @@ class ElectrumProtocol(asyncio.Protocol):  # pylint: disable=R0904,R0902
         Return the unconfirmed transactions of a script hash.
         """
         # TODO: Implement
-        return
+        return JsonRPCError.invalidrequest()
 
     async def blockchain_scripthash_listunspent(self, writer, query):  # pylint: disable=W0613
         """Method: blockchain.scripthash.listunspent