Backport PR #1334 on branch 0.4.x (Do not clean up resources after execution) (#1335)

* Backport PR #1334: Do not clean up resources after execution

* Only clean kernel client

* Fix CI

* OpenSSL issue workaround

---------

Co-authored-by: martinRenou <[email protected]>

Author: meeseeksmachine

.github/workflows/main.yml

@@ -32,7 +32,7 @@ jobs:
 
     - name: Create the conda environment
       shell: bash -l {0}
-      run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs yarn=1 flake8 ipywidgets matplotlib xeus-cling openssl=1.1.1l "traitlets>=5.0.3,<6"
+      run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs yarn=1 flake8 ipywidgets matplotlib xeus-cling "traitlets>=5.0.3,<6"
 
     - name: Install dependencies
       shell: bash -l {0}
@@ -82,7 +82,7 @@ jobs:
 
     - name: Create the conda environment
       shell: bash -l {0}
-      run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs yarn=1 flake8 ipywidgets matplotlib xeus-cling openssl=1.1.1l "traitlets>=5.0.3,<6"
+      run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs yarn=1 flake8 ipywidgets matplotlib xeus-cling "traitlets>=5.0.3,<6"
 
     - name: Install dependencies
       shell: bash -l {0}

packages/jupyterlab-preview/package.json

@@ -72,6 +72,7 @@
   },
   "jupyterlab": {
     "extension": true,
+    "webpackConfig": "webpack.lab.config.js",
     "schemaDir": "schema",
     "outputDir": "../../voila/labextension",
     "discovery": {

packages/jupyterlab-preview/webpack.lab.config.js

@@ -0,0 +1,6 @@
+const crypto = require('crypto');
+
+// Workaround for loaders using "md4" by default, which is not supported in FIPS-compliant OpenSSL
+const cryptoOrigCreateHash = crypto.createHash;
+crypto.createHash = algorithm =>
+  cryptoOrigCreateHash(algorithm === 'md4' ? 'sha256' : algorithm);

packages/voila/package.json

@@ -45,9 +45,9 @@
   },
   "style": "style/index.css",
   "scripts": {
-    "build": "npm run build:lib && webpack",
+    "build": "jlpm run build:lib && webpack",
     "build:lib": "tsc",
-    "build:prod": "npm run build:lib && webpack --production",
+    "build:prod": "jlpm run build:lib && webpack --production",
     "clean": "jlpm run clean:lib",
     "clean:lib": "rimraf lib tsconfig.tsbuildinfo",
     "test": "echo \"Error: no test specified\" && exit 1",

packages/voila/webpack.config.js

@@ -1,5 +1,12 @@
 var path = require('path');
 
+const crypto = require('crypto');
+
+// Workaround for loaders using "md4" by default, which is not supported in FIPS-compliant OpenSSL
+const cryptoOrigCreateHash = crypto.createHash;
+crypto.createHash = algorithm =>
+  cryptoOrigCreateHash(algorithm === 'md4' ? 'sha256' : algorithm);
+
 var rules = [
   { test: /\.css$/, use: ['style-loader', 'css-loader'] },
   // required to load font-awesome

voila/notebook_renderer.py

@@ -306,7 +306,6 @@ async def _jinja_cell_generator(self, nb, kernel_id):
         await self._cleanup_resources()
 
     async def _cleanup_resources(self):
-        await ensure_async(self.executor.km.cleanup_resources())
         await ensure_async(self.executor.kc.stop_channels())
 
     async def load_notebook(self, path):