fix: ensure id is string and jsonrpc is 2.0

open-rpc · Dec 5, 2019 · 5d6841d · 5d6841d
1 parent b375417
commit 5d6841d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/containers/Inspector.tsx b/src/containers/Inspector.tsx
@@ -76,11 +76,21 @@ const Inspector: React.FC<IProps> = (props) => {
         jsonrpc: "2.0",
         method: props.openrpcMethodObject.name,
         params: json.params,
-        id,
+        id: id.toString(),
       });
     }
   // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
+  useEffect(() => {
+    if (json) {
+      setJson({
+        ...json,
+        jsonrpc: "2.0",
+        id: id.toString(),
+      });
+    }
+  // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [id]);
 
   useEffect(() => {
     if (props.url) {
@@ -94,7 +104,7 @@ const Inspector: React.FC<IProps> = (props) => {
       incrementId();
       try {
         const result = await client.request(json.method, json.params);
-        setResults({ jsonrpc: "2.0", result });
+        setResults({ jsonrpc: "2.0", result, id });
       } catch (e) {
         setError(e);
       }

diff --git a/src/containers/JSONRPCRequest.tsx b/src/containers/JSONRPCRequest.tsx
@@ -3,7 +3,7 @@ import useDarkMode from "use-dark-mode";
 import { monaco, ControlledEditor, Monaco } from "@monaco-editor/react";
 import { MethodObject } from "@open-rpc/meta-schema";
 import useWindowSize from "@rehooks/window-size";
-import { addDiagnostics } from "@etclabscore/monaco-add-json-schema-diagnostics"
+import { addDiagnostics } from "@etclabscore/monaco-add-json-schema-diagnostics";
 
 interface IProps {
   onChange?: (newValue: any) => void;