@bonniernews/local-esi - npm Package Compare versions

Comparing version 0.7.0 to 0.7.1

lib/ESIListener.js

@@ -248,3 +248,3 @@ "use strict";
 
-        return writeToResult(handleProcessingInstructions(text), next);
+        return writeToResult(handleProcessingInstructions.bind(null, text), next); //handleProcessingInstructions may cause an (expected) error and we're not sure writeToResult will actually write so we pass a function that it can call if it should write
       } catch (err) {
@@ -311,2 +311,6 @@ return next(err);
 
+    if (text.match(/\$substr\((.*?)\)/ig)) {
+      throw new Error("$substr invoked without start index parameter");
+    }
+
     text = removeReservedCharacters(text);
@@ -392,2 +396,5 @@
   function writeToResult(chunk, next) {
+    if (typeof chunk === "function" && (context.inReplacement || shouldWrite())) {
+      chunk = chunk();
+    }
     if (context.inReplacement) {
@@ -394,0 +401,0 @@ context.replacement += chunk;

package.json

 {
   "name": "@bonniernews/local-esi",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "Local Edge Side Includes parser",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

22961

increased by1.86%

Lines of code

648

increased by0.93%

No dependency changes