node-oom-heapdump 2.2.0 → 3.0.0-beta

package/.travis.yml

@@ -8,4 +8,6 @@ node_js:
   - "12"
   - "13"
   - "14"
+  - "15"
+  - "16"
 script: npm run dummy

package/CHANGELOG.md

@@ -1,3 +1,14 @@
+09-02-2022 Paul Rütter
+- 3.0.0
+- Updated dependencies to resolve https://github.com/blueconic/node-oom-heapdump/issues/19
+- Removed `gc-stats` as it was not used a lot anymore (breaking change) and contained security issues.
+- Removed creating heapdumps on out of memory, as this is superseeded by native node functionality.
+- Remove unused dependencies and native code.
+
+21-12-2020 Paul Rütter
+- 2.2.0
+- Updated ini
+
 12-10-2020 Paul Rütter
 - 2.1.0
 - Added Node 14 support

package/README.md

@@ -1,34 +1,29 @@
 [![TravisCI Build Status](https://travis-ci.org/blueconic/node-oom-heapdump.svg?branch=master)](https://travis-ci.org/blueconic/node-oom-heapdump)
 
 # node-oom-heapdump
-Node module which will create a V8 heap snapshot right before an "Out of Memory" error occurs.
+Node module that can create heapdumps and CPU profiles on request like 'v8-profiler', but does this off-process so it doesn't interfere with execution of the main process.
 
-It can also create heapdumps and CPU profiles on request like 'v8-profiler', but does this off-process so it doesn't interfere with execution of the main process.
-
-Tested on Node.js 7.x, 8.x, 9.x, 10.x, 11.x, 12.x, 13.x and 14.x.
+Tested on Node.js 7.x, 8.x, 9.x, 10.x, 11.x, 12.x, 13.x, 14.x and 16.x.
 No support for Node.js < 7.0 at the moment (although this can be fixed if needed).  
 
 Also comes with prebuilt binaries (hosted on Github releases), thanks to Stuart Miller (https://github.com/spmiller).
 
-# Why?
-When running nodejs processes in a low memory environment, every out of memory that occurs is interesting.
-To figure out why a process went out of memory, a heap snapshot (e.g. heapdump) can help a lot.
-This module creates a heap snapshot right before an out of memory error occurs (by leveraging 'SetOOMErrorHandler' of the V8 engine).
-It shows what the heap was filled with right before the out of memory error occured and can be opened with Chrome DevTools (Memory tab).
+## Node.js 14.18.x and higher
+https://github.com/nodejs/node/pull/33010 landed in Node.js 14.18.0, which makes this module no longer needed for heapdumps on out of memory.
+One can use the `--heapsnapshot-near-heap-limit` Node.js CLI option as an native alternative.
+See https://nodejs.org/dist/latest-v14.x/docs/api/cli.html#cli_heapsnapshot_near_heap_limit_max_count.
 
-There are several modules around which can create heapdumps (v8-profiler, node-heapdump), but these run in the same process as the one going out of memory. Often, creating heapdump won't work when the node process is already struggling.
-This module creates the heap snapshot from a separate process, which solves this issue.
-Also, these modules are not able to create a heapdump when an out of memory occurs.
+For Node versions older than 14, use the `2.2.0` release, where this functionality is still present (not maintained anymore).
+For Node versions 14 and up, use `--heapsnapshot-near-heap-limit` for out-of-memory heapdumps. This functionality has been removed from `3.0.0`.
 
-# What?
-Based on the work of 'trevnorris' (https://github.com/trevnorris/node-ofe/), this module uses 'isolate.SetOOMErrorHandler' (https://v8docs.nodesource.com/node-8.9/d5/dda/classv8_1_1_isolate.html#a08fd4087f39c33b4ac1c20ad953ce4e3) of the V8 engine, and then creates a heapdump when an actual Out of Memory occurs. To make this happen, a native C++ add-on is used. 
-Node-gyp is needed to compile this add-on.
+One can still use the API of this module to create CPU profiles and heapdumps on request
 
-When creating a heapdump of CPU profile on request, the DevTools protocol is used to create these files (no native add-on).
-The --inspect node.js flag is needed to make this work (which is validated on startup).
+# Why?
+There are several modules around which can create heapdumps (v8-profiler, node-heapdump), although this module creates the heap snapshot from a separate process which performs better and doesn't affect the main event loop.
 
-# Example
-Just run "npm test" to see it in action. It creates a heapdump named "my_heapdump.heapsnapshot" in the 'tests' directory of this module.
+# What?
+When creating a heapdump of CPU profile on request, the DevTools protocol is used to create these files (no native add-on needed).
+The --inspect node.js flag is needed to make this work (which is validated on startup).
 
 # Usage
 
@@ -57,20 +52,10 @@ These might impact performance though.
 On Node.js 12.x the latter two flags seem to cause some stability issues (see https://github.com/nodejs/node/issues/27552#issuecomment-542695931). So, if you encounter issues on Node.js 12.x in combination with those flags, please refrain from using these.
 
 # Options
-* heapdumpOnOOM - boolean whether to create a heapdump when an out of memory occurs. Default true.
-* OOMImplementation - Either "NATIVE_HOOK" or "GC_MONITORING". 
-"NATIVE_HOOK" relies on the native v8 hook and makes sure that the heapdump is actually created when the OoM occurs. It's more impacted by the OoMKiller of Unix systems though, when being run in memory restricted environments like Docker. 
-"GC_MONITORING" is the old implementation, which relies on GC monitoring. It's less impacted by OoMKiller, because the 'threshold' parameter determines when to create the heapdump. Use this option when you run into the OoMKiller of the Unix OS. Default is "NATIVE_HOOK".
-* path - the path where the heapdump ends up when an out of memory error occurs. '.heapsnapshot' is automatically appended. Defaults to this modules' directory.
-* addTimestamp - add a timestamp to the out of memory heapdump filename, to make it unique. Default is false.
 * port - optionally, the alternative DevTools protocol port. Defaults to 9229. Should map on the port given to the --inspect arg.
 
-The following options are only relevant when OOMImplementation="GC_MONITORING".
-* limit - optionally, specify a limit to how many heapdumps will be created when being above the threshold. Default is 3.
-* threshold - integer between 0 and 100 (%) which determines when to make the heapdump. When the used heapSize exceeds the threshold, a heapdump is made. This value can be tuned depending on your configuration; if memory usage is very volatile, a lower value might make more sense. Default is 70.
-
 # API
-Besides creating heapdumps when an out of memory error occurs, there also is an API for creating heapdumps and CPU profiles on request. See below for the currently available API.
+The  API for creating heapdumps and CPU profiles on request. See below for the currently available API.
 
 Notice that you cannot create a heapdump while a CPU profile is being generated and vice versa; an Error will be thrown if this is the case.
 

package/build/node_oom_heapdump_native.vcxproj

@@ -36,6 +36,7 @@
     <ExecutablePath>$(ExecutablePath);$(MSBuildProjectDirectory)\..\bin\;$(MSBuildProjectDirectory)\..\bin\</ExecutablePath>
     <IgnoreImportLibrary>true</IgnoreImportLibrary>
     <IntDir>$(Configuration)\obj\$(ProjectName)\</IntDir>
+    <LinkIncremental Condition="'$(Configuration)|$(Platform)'=='Release|x64'">false</LinkIncremental>
     <LinkIncremental Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">true</LinkIncremental>
     <OutDir>$(SolutionDir)$(Configuration)\</OutDir>
     <TargetExt Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">.node</TargetExt>
@@ -47,10 +48,10 @@
   </PropertyGroup>
   <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">
     <ClCompile>
-      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+      <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
       <BasicRuntimeChecks>EnableFastChecks</BasicRuntimeChecks>
       <BufferSecurityCheck>true</BufferSecurityCheck>
-      <CompileAsWinRT>false</CompileAsWinRT>
       <DebugInformationFormat>OldStyle</DebugInformationFormat>
       <DisableSpecificWarnings>4351;4355;4800;4251;4275;4244;4267;%(DisableSpecificWarnings)</DisableSpecificWarnings>
       <ExceptionHandling>false</ExceptionHandling>
@@ -59,31 +60,38 @@
       <OmitFramePointers>false</OmitFramePointers>
       <Optimization>Disabled</Optimization>
       <PrecompiledHeader>NotUsing</PrecompiledHeader>
-      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;DEBUG;_DEBUG;V8_ENABLE_CHECKS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
+      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;_GLIBCXX_USE_CXX11_ABI=1;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;DEBUG;_DEBUG;V8_ENABLE_CHECKS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <RuntimeLibrary>MultiThreadedDebug</RuntimeLibrary>
       <StringPooling>true</StringPooling>
       <SuppressStartupBanner>true</SuppressStartupBanner>
       <TreatWarningAsError>false</TreatWarningAsError>
       <WarningLevel>Level3</WarningLevel>
+      <WholeProgramOptimization>true</WholeProgramOptimization>
     </ClCompile>
+    <Lib>
+      <AdditionalOptions>/LTCG:INCREMENTAL %(AdditionalOptions)</AdditionalOptions>
+    </Lib>
     <Link>
-      <AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;DelayImp.lib;&quot;C:\\Users\\paul.BLUECONIC\\AppData\\Local\\node-gyp\\Cache\\12.20.0\\x64\\node.lib&quot;</AdditionalDependencies>
+      <AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;DelayImp.lib;&quot;C:\\Users\\paul.BLUECONIC\\AppData\\Local\\node-gyp\\Cache\\16.14.0\\x64\\node.lib&quot;</AdditionalDependencies>
+      <AdditionalOptions>/LTCG:INCREMENTAL %(AdditionalOptions)</AdditionalOptions>
+      <EnableCOMDATFolding>true</EnableCOMDATFolding>
       <GenerateDebugInformation>true</GenerateDebugInformation>
+      <OptimizeReferences>true</OptimizeReferences>
       <OutputFile>$(OutDir)$(ProjectName).node</OutputFile>
       <SuppressStartupBanner>true</SuppressStartupBanner>
       <TargetExt>.node</TargetExt>
       <TargetMachine>MachineX64</TargetMachine>
     </Link>
     <ResourceCompile>
-      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
-      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;DEBUG;_DEBUG;V8_ENABLE_CHECKS;%(PreprocessorDefinitions);%(PreprocessorDefinitions)</PreprocessorDefinitions>
+      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;_GLIBCXX_USE_CXX11_ABI=1;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;DEBUG;_DEBUG;V8_ENABLE_CHECKS;%(PreprocessorDefinitions);%(PreprocessorDefinitions)</PreprocessorDefinitions>
     </ResourceCompile>
   </ItemDefinitionGroup>
   <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'">
     <ClCompile>
-      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+      <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
       <BufferSecurityCheck>true</BufferSecurityCheck>
-      <CompileAsWinRT>false</CompileAsWinRT>
       <DebugInformationFormat>OldStyle</DebugInformationFormat>
       <DisableSpecificWarnings>4351;4355;4800;4251;4275;4244;4267;%(DisableSpecificWarnings)</DisableSpecificWarnings>
       <ExceptionHandling>false</ExceptionHandling>
@@ -95,32 +103,41 @@
       <OmitFramePointers>true</OmitFramePointers>
       <Optimization>Full</Optimization>
       <PrecompiledHeader>NotUsing</PrecompiledHeader>
-      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;%(PreprocessorDefinitions)</PreprocessorDefinitions>
+      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;_GLIBCXX_USE_CXX11_ABI=1;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <RuntimeLibrary>MultiThreaded</RuntimeLibrary>
       <RuntimeTypeInfo>false</RuntimeTypeInfo>
       <StringPooling>true</StringPooling>
       <SuppressStartupBanner>true</SuppressStartupBanner>
       <TreatWarningAsError>false</TreatWarningAsError>
       <WarningLevel>Level3</WarningLevel>
+      <WholeProgramOptimization>true</WholeProgramOptimization>
     </ClCompile>
+    <Lib>
+      <AdditionalOptions>/LTCG:INCREMENTAL %(AdditionalOptions)</AdditionalOptions>
+    </Lib>
     <Link>
-      <AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;DelayImp.lib;&quot;C:\\Users\\paul.BLUECONIC\\AppData\\Local\\node-gyp\\Cache\\12.20.0\\x64\\node.lib&quot;</AdditionalDependencies>
+      <AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;DelayImp.lib;&quot;C:\\Users\\paul.BLUECONIC\\AppData\\Local\\node-gyp\\Cache\\16.14.0\\x64\\node.lib&quot;</AdditionalDependencies>
+      <AdditionalOptions>/LTCG:INCREMENTAL %(AdditionalOptions)</AdditionalOptions>
+      <EnableCOMDATFolding>true</EnableCOMDATFolding>
       <GenerateDebugInformation>true</GenerateDebugInformation>
+      <OptimizeReferences>true</OptimizeReferences>
       <OutputFile>$(OutDir)$(ProjectName).node</OutputFile>
       <SuppressStartupBanner>true</SuppressStartupBanner>
       <TargetExt>.node</TargetExt>
       <TargetMachine>MachineX64</TargetMachine>
     </Link>
     <ResourceCompile>
-      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\12.20.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
-      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;%(PreprocessorDefinitions);%(PreprocessorDefinitions)</PreprocessorDefinitions>
+      <AdditionalIncludeDirectories>C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\include\node;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\src;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\config;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\openssl\openssl\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\uv\include;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\zlib;C:\Users\paul.BLUECONIC\AppData\Local\node-gyp\Cache\16.14.0\deps\v8\include;..\node_modules\nan;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+      <PreprocessorDefinitions>NODE_GYP_MODULE_NAME=node_oom_heapdump_native;USING_UV_SHARED=1;USING_V8_SHARED=1;V8_DEPRECATION_WARNINGS=1;V8_DEPRECATION_WARNINGS;V8_IMMINENT_DEPRECATION_WARNINGS;_GLIBCXX_USE_CXX11_ABI=1;WIN32;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_HAS_EXCEPTIONS=0;OPENSSL_NO_PINSHARED;OPENSSL_THREADS;BUILDING_NODE_EXTENSION;%(PreprocessorDefinitions);%(PreprocessorDefinitions)</PreprocessorDefinitions>
     </ResourceCompile>
   </ItemDefinitionGroup>
   <ItemGroup>
     <None Include="..\binding.gyp"/>
   </ItemGroup>
   <ItemGroup>
-    <ClCompile Include="..\lib\node_oom_heapdump_native.cc"/>
+    <ClCompile Include="..\lib\node_oom_heapdump_native.cc">
+      <ObjectFileName>$(IntDir)\lib\node_oom_heapdump_native.obj</ObjectFileName>
+    </ClCompile>
   </ItemGroup>
   <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets"/>
   <Import Project="$(VCTargetsPath)\BuildCustomizations\masm.targets"/>

package/index.js

@@ -85,8 +85,7 @@ function parseOptions(options) {
     options.heapdumpOnOOM = true;
   }
   if (options.OOMImplementation === undefined){
-    // default is the "new" implementation
-    // the other option is "GC_MONITORING", which is the old implementation (which is less impacted by OoMKiller)
+    // default is the implementation, the old one was discarded because it wasn't used
     options.OOMImplementation = "NATIVE_HOOK";
   }
   if (options.port === undefined) {

package/lib/index.js

@@ -7,47 +7,6 @@ class NodeOomHeapDumpImpl {
     this._opts = options;
     this._files = [];
     this._busy = false;
-    this._count = 0;
-    this._limitReached = false;
-
-    if (this._opts.heapdumpOnOOM) {
-      if (options.OOMImplementation === "NATIVE_HOOK") {
-        require('bindings')('node_oom_heapdump_native.node').call(this._getHeapSnapshotPath(this._opts.path), this._opts.addTimestamp);
-      } else if (options.OOMImplementation === "GC_MONITORING") {
-        this._monitorHeap();
-      }
-    }
-  }
-
-  _monitorHeap() {
-    // see https://www.npmjs.com/package/gc-stats	
-    let gcStats = new require('gc-stats')();
-    gcStats.on('stats', (stats) => {
-      // gctype 2 is a Full GC (Mark/Sweep/Compact)	
-      if (stats.gctype === 2 && !this._busy) {
-        let memoryUsagePercentage = Math.round((stats.after.usedHeapSize / stats.after.heapSizeLimit) * 100);
-        if (memoryUsagePercentage > this._opts.threshold) {
-          if (this._count < this._opts.limit) {
-            // this is a full GC and the used heap size is using more than x% of the assigned heap space limit	
-            console.warn('OoM is imminent: Full GC (Mark/Sweep/Compact) complete and still more than %s% (%s%) of the heap is used. Creating heapdump now. GC stats: ', this._opts.threshold, Math.round(memoryUsagePercentage), JSON.stringify(stats));
-
-            this.createHeapSnapshot(this._opts.path, "OoM");
-
-            // block execution of other code for a while (5 second) to enable snapshot to be created	
-            const time = Date.now();
-            let diff = 0;
-            do {
-              diff = Date.now() - time;
-            }
-            while (diff < 5000);
-          } else if (!this._limitReached) {
-            this._limitReached = true;
-            console.warn("OoM heapdump limit reached (%s); no more heapdumps will be created.", this._opts.limit);
-            return;
-          }
-        }
-      }
-    });
   }
 
 

package/package.json

@@ -1,25 +1,18 @@
 {
   "name": "node-oom-heapdump",
-  "version": "2.2.0",
-  "description": "Create a V8 heap snapshot when an \"Out of Memory\" error occurs, or create a heap snapshot or CPU profile on request.",
+  "version": "3.0.0-beta",
+  "description": "Create a heap snapshot or CPU profile on request, off-process through the DevTools protocol",
   "main": "index.js",
   "scripts": {
-    "rebuild": "node-pre-gyp install --build-from-source",
-    "install": "node-pre-gyp install --fallback-to-build",
-    "test": "node --max_old_space_size=80 --inspect ./tests/oom_app.js",
-    "dummy": "node -e \"process.exit(0)\""
+    "dummy": "node -e \"process.exit(0)\"",
+    "api-publish-beta": "npm publish --tag beta"
   },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/blueconic/node-oom-heapdump.git"
   },
-  "binary": {
-    "module_name": "node_oom_heapdump_native",
-    "module_path": "./build/Release",
-    "host": "https://github.com/blueconic/node-oom-heapdump/releases/download/{version}"
-  },
   "engines": {
-    "node": ">=7.0.0"
+    "node": ">=14.0.0"
   },
   "keywords": [
     "nodejs",
@@ -40,12 +33,8 @@
     "eslint-config-google": "^0.14.0"
   },
   "dependencies": {
-    "bindings": "^1.5.0",
-    "chrome-remote-interface": "^0.28.2",
-    "gc-stats": "^1.4.0",
-    "nan": "^2.14.2",
-    "node-pre-gyp": "^0.17.0",
+    "chrome-remote-interface": "^0.31.2",
     "require-main-filename": "^2.0.0",
-    "ws": "^7.4.1"
+    "ws": "^8.5.0"
   }
 }

package/.github/workflows/publish-native-assets-to-github-releases.yml

@@ -1,37 +0,0 @@
-# This workflow will prebuild native binaries for supported NodeJS versions, and add them to the Github release that triggered the workflow
-
-name: Add native binaries to release
-
-on:
-  release:
-    types: [created]
-
-jobs:
-  augment-release:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        node-version: [7.x, 8.x, 9.x, 10.x, 11.x, 12.x, 14.x]
-        os: [ubuntu-latest, macos-latest, windows-latest]
-    steps:
-      - uses: actions/checkout@v2
-      - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v1
-        with:
-          node-version: ${{ matrix.node-version }}
-      - name: patch node gyp on windows to support Visual Studio 2019
-        if: matrix.os == 'windows-latest'
-        shell: powershell
-        run: |
-          npm install --global node-gyp@5.1.1
-          npm prefix -g | % {npm config set node_gyp "$_\node_modules\node-gyp\bin\node-gyp.js"}
-      - name: build using node-pre-gyp
-        run: |
-          npm install --build-from-source
-          ./node_modules/.bin/node-pre-gyp package
-      - name: Upload native binaries for Node ${{ matrix.node-version }} for ${{ matrix.os }}
-        uses: csexton/release-asset-action@v2
-        with:
-          pattern: "build/stage/*.tar.gz"
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-          release-url: ${{ github.event.release.upload_url }}

package/binding.gyp

@@ -1,10 +0,0 @@
-{
-	"targets" : [{
-		"target_name" : "node_oom_heapdump_native",
-		"sources" : [ "lib/node_oom_heapdump_native.cc" ],
-		"include_dirs": [
-			'<!(node -e "require(\'nan\')")'
-		],
-		"win_delay_load_hook" : "false"
-	}]
-}

package/lib/node_oom_heapdump_native.cc

@@ -1,99 +0,0 @@
-#include <nan.h>
-#include <v8-profiler.h>
-#include <stdlib.h>
-#if defined(_WIN32)
-  #include <time.h>
-  #define snprintf _snprintf
-#else
-  #include <sys/time.h>
-#endif
-
-using namespace v8;
-
-char filename[256];
-bool addTimestamp;
-
-class FileOutputStream: public OutputStream {
-  public:
-    FileOutputStream(FILE* stream): stream_(stream) { }
-    virtual int GetChunkSize() {
-      return 65536;
-    }
-    virtual void EndOfStream() { }
-    virtual WriteResult WriteAsciiChunk(char* data, int size) {
-      const size_t len = static_cast<size_t>(size);
-      size_t off = 0;
-      while (off < len && !feof(stream_) && !ferror(stream_))
-        off += fwrite(data + off, 1, len - off, stream_);
-      return off == len ? kContinue : kAbort;
-    }
-
-  private:
-    FILE* stream_;
-};
-
-void OnOOMError(const char *location, bool is_heap_oom) {
-  if (addTimestamp) {
-    // Add timestamp to filename
-    time_t rawtime;
-    struct tm* timeinfo;
-    time(&rawtime);
-    timeinfo = localtime(&rawtime);
-
-    char * pch;
-    pch = strstr (filename,".heapsnapshot");
-    strncpy (pch,"",1);
-    strcat (filename, "-%Y%m%dT%H%M%S.heapsnapshot");
-    
-    char newFilename[256];
-    strftime(newFilename, sizeof(filename), filename, timeinfo);
-    strcpy(filename, newFilename);
-  }
-
-  fprintf(stderr, "Generating Heapdump to '%s' now...\n", filename);
-  FILE* fp = fopen(filename, "w");
-  if (fp == NULL) abort();
-
-  // Create heapdump, depending on which Node.js version this can differ
-  // for now, just support Node.js 7 and higher
-  const HeapSnapshot* snap = v8::Isolate::GetCurrent()->GetHeapProfiler()->TakeHeapSnapshot();
-  
-  FileOutputStream stream(fp);
-  snap->Serialize(&stream, HeapSnapshot::kJSON);
-  fclose(fp);
-
-  fprintf(stderr, "Done! Exiting process now.\n");
-  exit(1);
-}
-
-void ParseArgumentsAndSetErrorHandler(const FunctionCallbackInfo<Value>& args) {
-  Isolate* isolate = args.GetIsolate();
-  isolate->SetOOMErrorHandler(OnOOMError);
-  
-  // parse JS arguments
-  // 1: filename
-  // 2: addTimestamp boolean
-  #if NODE_VERSION_AT_LEAST(13, 0, 0)
-  Local<Context> context = isolate->GetCurrentContext();
-  String::Utf8Value fArg(isolate, args[0]->ToString(context).ToLocalChecked());
-  #elif NODE_VERSION_AT_LEAST(12, 0, 0)
-  String::Utf8Value fArg(isolate, args[0]->ToString(isolate));
-  #elif NODE_VERSION_AT_LEAST(9, 0, 0)
-  String::Utf8Value fArg(isolate, args[0]->ToString());
-  #else
-  String::Utf8Value fArg(args[0]->ToString());
-  #endif
-  strncpy(filename, (const char*)(*fArg), sizeof(filename) - 1);
-
-  #if NODE_VERSION_AT_LEAST(12, 0, 0)
-  addTimestamp = args[1]->BooleanValue(isolate);
-  #else
-  addTimestamp = args[1]->BooleanValue();
-  #endif
-}
-
-void init(Local<Object> exports) {
-  NODE_SET_METHOD(exports, "call", ParseArgumentsAndSetErrorHandler);
-}
-
-NODE_MODULE(NODE_OOM_HEAPDUMP_NATIVE, init)

package/tests/index.js

@@ -1,27 +0,0 @@
-let cp = require("child_process");
-let fs = require("fs");
-let path = require("path");
-
-describe('Heapdumps', function () {
-  it('should be created in x seconds', function (done) {
-    this.timeout(250000);
-
-    let child = cp.fork(path.resolve(__dirname, './oom_app.js'), null, {
-      cmd: path.dirname(require.main.filename),
-      stdio: 'inherit',
-      execArgv: ["--max_old_space_size=40", "--optimize_for_size", "--always_compact", "--inspect=9229"]
-    });
-
-    setTimeout(function () {
-      child.kill();
-      fs.lstat(path.resolve(__dirname, "../abc.heapsnapshot"), (err, stats) => {
-        if (!err && stats.isFile()) {
-          done();
-        } else {
-          done(err);
-        }
-        clearTimeout(handle);
-      })
-    }, 20000);
-  });
-});

package/tests/long_running_process.js

@@ -1,27 +0,0 @@
-let oomLib = require("../index.js")({
-    heapdumpOnOOM: false
-});
-
-var i = 0;
-var path = "";
-
-var handle = setInterval(function () {
-    i++;
-
-    oomLib.createHeapSnapshot(require("path").resolve("../", "myName")).then((p) => {
-        path = p;
-    }).catch((err) => {
-        console.error(err);
-    });
-
-    if (i === 3) {
-        oomLib.deleteHeapSnapshot(path).then(() => {
-            //
-        }).catch((err) => {
-            console.error("err", err);
-        });
-        setTimeout(function () {
-            process.exit(0);
-        }, 100);
-    }
-}, 2000);

package/tests/long_running_process_cpu.js

@@ -1,35 +0,0 @@
-let oomLib = require("../index.js")({
-    heapdumpOnOOM: false
-});
-
-var i = 0;
-var path = "";
-
-oomLib.createCpuProfile(require("path").resolve("myCPU.cpuprofile"), 3000).then((p) => {
-    console.error("CPU profile", p);
-
-    //oomLib.deleteAllCpuProfiles();
-}).catch((err) => {
-    console.error(err);
-});
-
-var handle = setInterval(function () {
-    i++;
-
-    oomLib.createHeapSnapshot(require("path").resolve("../", "myName")).then((p) => {
-        path = p;
-    }).catch((err) => {
-        console.error(err);
-    });
-
-    if (i === 5) {
-        /* oomLib.deleteHeapSnapshot(path).then(() => {
-             //
-         }).catch((err) => {
-             console.error("err", err);
-         });*/
-        setTimeout(function () {
-            process.exit(0);
-        }, 100);
-    }
-}, 1000);

package/tests/oom_app.js

@@ -1,22 +0,0 @@
-let path = require('path');
-
-let oom = require("../index.js")({
-    path: path.resolve(__dirname, 'my_heapdump'),
-    heapdumpOnOOM: true,
-    //OOMImplementation: "GC_MONITORING", // use the old implementation
-    addTimestamp: false
-});
-
-// It is important to use named constructors (like the one below), otherwise
-// the heap snapshots will not produce useful outputs for you.
-function LeakingClass1() {
-}
-
-var leaks = [];
-var handle = setInterval(function () {
-    for (var i = 0; i < 100000; i++) {
-        leaks.push(new LeakingClass1);
-    }
-
-    console.error('Leaks: %d', leaks.length);
-}, 100);