fix: Always explicitly catch exceptions. (#132)

sharkedj · seebees · commit bf888714acc5 · 2019-07-08T12:20:03.000-07:00
Older js engines require this. See https://stackoverflow.com/a/21624550
diff --git a/modules/material-management/src/cryptographic_material.ts b/modules/material-management/src/cryptographic_material.ts
@@ -30,7 +30,7 @@ try {
    */
   const { timingSafeEqual: nodeTimingSafeEqual } = require('crypto')
   timingSafeEqual = nodeTimingSafeEqual || portableTimingSafeEqual
-} catch {
+} catch (e) {
   timingSafeEqual = portableTimingSafeEqual
 }
 /* https://codahale.com/a-lesson-in-timing-attacks/ */

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ try {`
`30`	`30`	`*/`
`31`	`31`	`const { timingSafeEqual: nodeTimingSafeEqual } = require('crypto')`
`32`	`32`	`timingSafeEqual = nodeTimingSafeEqual \|\| portableTimingSafeEqual`
`33`		`-} catch {`
	`33`	`+} catch (e) {`
`34`	`34`	`timingSafeEqual = portableTimingSafeEqual`
`35`	`35`	`}`
`36`	`36`	`/* https://codahale.com/a-lesson-in-timing-attacks/ */`