Skip to content

Commit 89325da

Browse files
committed
Fix: fix throws and rejects to be error objects and consistent
2 parents 78392b4 + d0b95da commit 89325da

File tree

6 files changed

+6
-6
lines changed

6 files changed

+6
-6
lines changed

lib/algorithms/concat.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ function concatDeriveFn(name) {
2020
var keyLen = props.length,
2121
hashLen = CONSTANTS.HASHLENGTH[name];
2222
if (!keyLen) {
23-
return Promise.reject("invalid key length");
23+
return Promise.reject(new Error("invalid key length"));
2424
}
2525

2626
// setup otherInfo

lib/algorithms/hmac.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ function hmacVerifyFN(name) {
151151
}
152152
promise = promise.then(function(result) {
153153
if (!result) {
154-
return Promise.reject("verifaction failed");
154+
return Promise.reject(new Error("verifaction failed"));
155155
}
156156

157157
return {

lib/jwe/encrypt.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ function JWEEncrypter(cfg, fields, recipients) {
148148
Object.defineProperty(this, "final", {
149149
value: function(data, encoding) {
150150
if (finalized) {
151-
throw new Error("already final");
151+
return Promise.reject(new Error("already final"));
152152
}
153153

154154
// last-minute data

lib/jwk/keystore.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -593,7 +593,7 @@ JWKStore.asKeyStore = function(ks) {
593593
} else if ("keys" in ks) {
594594
keys = ks.keys;
595595
} else {
596-
return Promise.reject("invalid keystore");
596+
return Promise.reject(new Error("invalid keystore"));
597597
}
598598

599599
keys = keys.map(function(k) {

lib/jws/sign.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ var JWSSigner = function(cfg, signatories) {
9898
Object.defineProperty(this, "final", {
9999
value: function(data, encoding) {
100100
if (finalized) {
101-
throw new Error("already final");
101+
return Promise.reject(new Error("already final"));
102102
}
103103

104104
// last-minute data

lib/jws/verify.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ var JWSVerifier = function(ks) {
9999
content,
100100
sig.signature);
101101
} else {
102-
algPromise = Promise.reject("key does not match");
102+
algPromise = Promise.reject(new Error("key does not match"));
103103
}
104104
algPromise = algPromise.then(function(result) {
105105
var payload = result.data.toString("ascii");

0 commit comments

Comments
 (0)