Skip to content

Commit 09d98c1

Browse files
committed
tech : remove async
1 parent e44415c commit 09d98c1

2 files changed

Lines changed: 20 additions & 30 deletions

File tree

index.js

Lines changed: 17 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
var url = require('url')
77
var URL = url.URL
8-
var async = require('async')
98
var request = require('request-promise')
109
var clientRoles = require('./rest/clientRoles')
1110
var clients = require('./rest/clients')
@@ -710,47 +709,36 @@ function token (options) {
710709

711710
return new Promise(function (resolve, reject) {
712711
request(requestOptions)
713-
.then(function (data) {
714-
var verifyClaims = {
715-
access_claims: function (done) {
712+
.then(async function (data) {
713+
const verifyClaims = [
714+
new Promise((resolve, reject) => {
716715
AccessToken.verify(data.access_token, {
717716
key: self.jwks.keys[0],
718717
issuer: self.issuer
719-
}, function (err, claims) {
720-
if (err) { return done(err) }
721-
done(null, claims)
718+
}, (err, claims) => {
719+
if (err) { reject(err) }
720+
resolve(claims)
722721
})
723-
}
724-
}
725-
// when requesting a token using client credentials no ID information is
726-
// returned
722+
})
723+
]
727724
if (formRequestData.grant_type !== 'client_credentials') {
728-
verifyClaims.id_claims = function (done) {
725+
verifyClaims.push(new Promise((resolve, reject) => {
729726
IDToken.verify(data.id_token, {
730727
iss: self.issuer,
731728
aud: self.client_id,
732729
key: self.jwks.keys[0]
733730
}, function (err, token) {
734-
if (err) { return done(err) }
735-
done(null, token.payload)
731+
if (err) { reject(err) }
732+
resolve(token.payload)
736733
})
737-
}
734+
}))
738735
}
739-
// verify tokens
740-
async.parallel(verifyClaims, function (err, result) {
741-
if (err) {
742-
return reject(err)
743-
}
744-
745-
data.id_claims = result.id_claims
746-
data.access_claims = result.access_claims
747-
748-
resolve(data)
749-
})
750-
})
751-
.catch(function (err) {
752-
reject(err)
736+
const [accessClaims, idClaims] = await Promise.all(verifyClaims)
737+
data.id_claims = idClaims
738+
data.access_claims = accessClaims
739+
resolve(data)
753740
})
741+
.catch(err => reject(err))
754742
})
755743
}
756744

test/anvilClientSpec.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -959,7 +959,9 @@ describe('Anvil Connect Client', function () {
959959
promise = anvil.token({
960960
grant_type: 'client_credentials',
961961
scope: 'realm'
962-
}).then(success).catch(failure)
962+
}).then(data => {
963+
return success(data)
964+
}).catch(failure)
963965
return promise
964966
})
965967
afterEach(function () {

0 commit comments

Comments
 (0)