keypairs.js

@@ -240,7 +240,7 @@ Keypairs.signJws = function (opts) {
       if ('EC' === opts.jwk.kty) {
         // ECDSA JWT signatures differ from "normal" ECDSA signatures
         // https://tools.ietf.org/html/rfc7518#section-3.4
-        binsig = ecdsaAsn1SigToJoseSig(binsig);
+        binsig = convertIfEcdsa(binsig);
       }
 
       var sig = binsig.toString('base64')
@@ -257,7 +257,7 @@ Keypairs.signJws = function (opts) {
       };
     }
 
-    function ecdsaAsn1SigToJoseSig(binsig) {
+    function convertIfEcdsa(binsig) {
       // should have asn1 sequence header of 0x30
       if (0x30 !== binsig[0]) { throw new Error("Impossible EC SHA head marker"); }
       var index = 2; // first ecdsa "R" header byte
@@ -346,14 +346,10 @@ Enc.bufToUrlBase64 = function (buf) {
 // For 'rsa-compat' module only
 // PLEASE do not use these sync methods, they are deprecated
 Keypairs._importSync = function (opts) {
-  try {
+  if ('RSA' === opts.jwk.kty) {
+    return Rasha.importSync(opts);
+  } else {
     return Eckles.importSync(opts);
-  } catch(e) {
-    try {
-      return Rasha.importSync(opts);
-    } catch(e) {
-      console.error("options.pem does not appear to be a valid RSA or ECDSA public or private key");
-    }
   }
 };
 // PLEASE do not use these, they are deprecated

package.json

@@ -1,6 +1,6 @@
 {
   "name": "keypairs",
-  "version": "1.2.14",
+  "version": "1.2.13",
   "description": "Lightweight RSA/ECDSA keypair generation and JWK <-> PEM using node's native RSA and ECDSA support",
   "main": "keypairs.js",
   "files": [