chore(host): Remove TODOs (#438)

Removes the TODO comments in the host server; The catch-all error handling here fits our usecase.
op-rs · Aug 21, 2024 · a61ab47 · a61ab47
1 parent ceef403
commit a61ab47
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/bin/host/src/server.rs b/bin/host/src/server.rs
@@ -85,8 +85,7 @@ where
             P: PreimageOracleServer,
         {
             loop {
-                // TODO: More granular error handling. Some errors here are expected, such as the
-                // client closing the pipe, while others are not and should throw.
+                // Break the loop on any error. An error in this path indicates a closed pipe.
                 if server.next_preimage_request(fetcher).await.is_err() {
                     break;
                 }
@@ -110,8 +109,7 @@ where
             H: HintReaderServer,
         {
             loop {
-                // TODO: More granular error handling. Some errors here are expected, such as the
-                // client closing the pipe, while others are not and should throw.
+                // Break the loop on any error. An error in this path indicates a closed pipe.
                 if server.next_hint(router).await.is_err() {
                     break;
                 }