Merge pull request #37 from fermyon/outbound-http-no-perms

Change where the HttpRequestDenied is returned
fermyon · Aug 27, 2024 · 6ec9e9d · 6ec9e9d
2 parents f00b59c + a5cb0e9
commit 6ec9e9d
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/components/outbound-wasi-http-v0.2.0/src/lib.rs b/components/outbound-wasi-http-v0.2.0/src/lib.rs
@@ -51,14 +51,29 @@ impl bindings::Guest for Component {
         let response = match outgoing_handler::handle(outgoing_request, None) {
             Ok(r) => r,
             Err(e) => {
-                return_response(outparam, 500, e.to_string().as_bytes());
+                return_response(
+                    outparam,
+                    500,
+                    format!("outgoing-handler/handle: {e}").as_bytes(),
+                );
                 return;
             }
         };
 
         let response = loop {
             if let Some(response) = response.get() {
-                break response.unwrap().unwrap();
+                let response = response.unwrap();
+                match response {
+                    Ok(r) => break r,
+                    Err(e) => {
+                        return_response(
+                            outparam,
+                            500,
+                            format!("incoming-response#get: {e}").as_bytes(),
+                        );
+                        return;
+                    }
+                };
             } else {
                 response.subscribe().block()
             }

diff --git a/tests/outbound-wasi-http-v0.2.0-no-perms/test.json5 b/tests/outbound-wasi-http-v0.2.0-no-perms/test.json5
@@ -28,7 +28,7 @@
                         "optional": true
                     }
                 ],
-                "body": "ErrorCode::HttpRequestDenied"
+                "body": "incoming-response#get: ErrorCode::HttpRequestDenied"
             },
         }
     ],