fix: corrected options.validate to default true (#1022)

* fix: corrected options.validate to default true

* chore: added test

* chore: corrected options.validate check

spec/validation.spec.js

@@ -348,6 +348,24 @@ describe("Request validation middleware", () => {
     expect(response.statusCode).toEqual(200);
   });
 
+  it("should validate if options passed but not validate flag", () => {
+    const newUrl =
+      fullUrl.pathname + fullUrl.search + "&somethingUnexpected=true";
+    const request = httpMocks.createRequest(
+      Object.assign({}, defaultRequest, {
+        originalUrl: newUrl,
+      })
+    );
+
+    const middleware = webhook(token, {});
+
+    middleware(request, response, () => {
+      expect(true).toBeFalsy();
+    });
+
+    expect(response.statusCode).toEqual(403);
+  });
+
   it("should accept manual host+proto", (done) => {
     const request = httpMocks.createRequest(
       Object.assign({}, defaultRequest, {

src/webhooks/webhooks.ts

@@ -355,11 +355,8 @@ export function webhook(
     }
   }
 
-  if (!options) {
-    options = {
-      validate: true,
-    };
-  }
+  if (!options) options = {};
+  if (options.validate == undefined) options.validate = true;
 
   // Process arguments
   var tokenString;