Merge pull request #164 from NearSocial/release-2.5.4

Release 2.5.4

Author: Caleb Jacob

CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## 2.5.4
+
+- Added optional `commitModalBypass` feature config. When the `<CommitButton />` component is used inside of a widget with a matching `src` prop, the `CommitModal` will be bypassed and `onCommit()` will be called instantly when the button is clicked. If for some reason the requested transaction is invalid, the `CommitModal` will still appear to show an error message to the user. View example below to see configuration options.
+- Added optional `enableWidgetSrcWithCodeOverride` boolean feature flag. This is helpful to enable when developing in a local environment when using a `redirectMap` in combination with the new `commitModalBypass` feature. With this enabled, any widget that is overridden via `redirectMap` can still reference its `src` prop to respect your `commitModalBypass` config.
+
+```js
+initNear({
+  features: {
+    commitModalBypass: {
+      authorIds: ["mob.near", "root.near"], // Bypass for all widgets published by these accounts
+      sources: [
+        "cool.near/widget/CoolComponent",
+        "awesome.near/widget/AwesomeComponent",
+      ], // Bypass for specific components
+    },
+    enableWidgetSrcWithCodeOverride: isLocalEnvironment,
+  },
+});
+```
+- Add string type check to the `href` property on the `a` tag. Reported by BrunoModificato from OtterSec.
+
 ## 2.5.3
 
 - FIX: Remove `cachedPropery` from `elliptic.utils`. Reported by BrunoModificato from OtterSec. 

dist/index.js

@@ -1,6 +1,6 @@
 {
   "name": "near-social-vm",
-  "version": "2.5.3",
+  "version": "2.5.4",
   "description": "Near Social VM",
   "main": "dist/index.js",
   "files": [

package.json

@@ -162,8 +162,26 @@ export const CommitModal = (props) => {
     }
   }
 
-  const show =
+  const bypassConfig = {
+    authorIds: near?.features?.commitModalBypass?.authorIds ?? [],
+    sources: near?.features?.commitModalBypass?.sources ?? [],
+  };
+  const [widgetSrcAccountId] = (widgetSrc ?? "").split("/");
+  const matchesModalBypassConfig =
+    (!!widgetSrcAccountId &&
+      bypassConfig.authorIds.indexOf(widgetSrcAccountId) > -1) ||
+    (!!widgetSrc && bypassConfig.sources.indexOf(widgetSrc.split("@")[0]) > -1);
+  const shouldBypassModal = !cantCommit && matchesModalBypassConfig;
+
+  const isReadyToCommit =
     !!commit && showIntent && !asyncCommitStarted && writePermission !== null;
+  const show = isReadyToCommit && !shouldBypassModal;
+
+  useEffect(() => {
+    if (isReadyToCommit && shouldBypassModal && !commitInProgress) {
+      onCommit();
+    }
+  }, [isReadyToCommit, shouldBypassModal, commitInProgress]);
 
   return (
     <Modal size="xl" centered scrollable show={show} onHide={onCancel}>

src/lib/components/Commit.js

@@ -102,7 +102,7 @@ export const Widget = React.forwardRef((props, forwardedRef) => {
       setSrc(src);
     } else if (srcOrCode?.code) {
       setCode(srcOrCode.code);
-      setSrc(null);
+      setSrc(near?.features?.enableWidgetSrcWithCodeOverride ? propsSrc : null);
     }
   }, [near, srcOrCode, nonce]);
 

src/lib/components/Widget.js

@@ -611,9 +611,10 @@ class VmStack {
     } else if (basicElement === "a") {
       Object.entries(attributes).forEach(([name, value]) => {
         if (name.toLowerCase() === "href") {
-          attributes[name] = isValidAttribute("a", "href", value)
-            ? value
-            : "about:blank";
+          attributes[name] =
+            isString(value) && isValidAttribute("a", "href", value)
+              ? value
+              : "about:blank";
         }
       });
     } else if (element === "Widget") {