Merge pull request #704 from geek/master

Add WebAssembly to allowed globals
hapijs · May 9, 2017 · 44d7e5f · 44d7e5f
2 parents c9678a4 + ffb783a
commit 44d7e5f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/lib/leaks.js b/lib/leaks.js
@@ -96,6 +96,10 @@ exports.detect = function (customGlobals) {
         whitelist.Reflect = true;
     }
 
+    if (global.WebAssembly) {
+        whitelist.WebAssembly = true;
+    }
+
     if (global.DTRACE_HTTP_SERVER_RESPONSE) {
         whitelist.DTRACE_HTTP_SERVER_RESPONSE = true;
         whitelist.DTRACE_HTTP_SERVER_REQUEST = true;

diff --git a/test/leaks.js b/test/leaks.js
@@ -142,6 +142,16 @@ describe('Leaks', () => {
         done();
     });
 
+    it('ignores WebAssembly global', (done) => {
+
+        testedKeys.push('WebAssembly');
+        global.WebAssembly = global.WebAssembly || 1;
+
+        const leaks = Lab.leaks.detect();
+        expect(leaks.length).to.equal(0);
+        done();
+    });
+
     it('ignores Harmony globals', (done) => {
 
         const harmonyGlobals = internals.harmonyGlobals;