Upgrade to use new version of released passport-openidconnect.

auth0 · Mar 28, 2018 · 26a58aa · 26a58aa
1 parent 03ee130
commit 26a58aa
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 22 deletions.
diff --git a/lib/passport-auth0-openidconnect/strategy.js b/lib/passport-auth0-openidconnect/strategy.js
@@ -32,6 +32,7 @@ function Strategy(options, verify) {
  });
 
  options = xtend({}, options, {
+ issuer: options.domain,
  authorizationURL: 'https://' + options.domain + '/authorize',
  tokenURL: 'https://' + options.domain + '/oauth/token',
  userInfoURL: 'https://' + options.domain + '/userinfo'
@@ -40,7 +41,7 @@ function Strategy(options, verify) {
  OpenIDConnectStrategy.call(this, options, verify);
 
  this.name = 'auth0-oidc';
- 
+
 }
 
 /**
@@ -51,4 +52,4 @@ util.inherits(Strategy, OpenIDConnectStrategy);
 /**
  * Expose `Strategy`.
  */
-module.exports = Strategy;
+module.exports = Strategy;
diff --git a/package.json b/package.json
@@ -17,7 +17,7 @@
  "should": "^7.1.1"
  },
  "dependencies": {
- "passport-openidconnect": "https://github.com/siacomuzzi/passport-openidconnect/tarball/state_store",
+ "passport-openidconnect": "0.0.2",
  "pkginfo": "^0.3.1",
  "request": "^2.65.0",
  "xtend": "^4.0.0"

diff --git a/test/strategy.test.js b/test/strategy.test.js
@@ -22,7 +22,7 @@ describe('constructor with bad params', function () {
  });
 
  it('should fail if no clientSecret is passed as option',function(){
- (function(){ 
+ (function(){
  new Auth0OidcStrategy(
  {domain:'foo',clientID:'bar'},
  function(accessToken, idToken, profile, done){}
@@ -43,22 +43,23 @@ describe('constructor with bad params', function () {
 describe('constructor with right params and pre-configured endpoints', function () {
  var strategy, configuration;
 
- before(function () {
+ before(function(done) {
  strategy = new Auth0OidcStrategy({
- domain: 'jj.auth0.com',
- clientID: 'testid',
- clientSecret: 'testsecret',
- callbackURL: '/callback',
- authorizationURL: 'https://jj.auth0.com/authorize',
- tokenURL: 'https://jj.auth0.com/oauth/token',
- userInfoURL: 'https://jj.auth0.com/userinfo'
- },
- function(accessToken, idToken, profile, done) {}
- );
-
- strategy._configurers.should.have.length(1);
- strategy._configurers[0](null, function (err, config) {
- configuration = config;
+ domain: 'jj.auth0.com',
+ clientID: 'testid',
+ clientSecret: 'testsecret',
+ callbackURL: '/callback',
+ authorizationURL: 'https://jj.auth0.com/authorize',
+ tokenURL: 'https://jj.auth0.com/oauth/token',
+ userInfoURL: 'https://jj.auth0.com/userinfo'
+ },
+ function(accessToken, idToken, profile, done) {});
+
+ strategy._setup(null, function(err, params) {
+ if (err) { return done(err); }
+ configuration = params;
+
+ done();
  });
  });
 
@@ -69,11 +70,11 @@ describe('constructor with right params and pre-configured endpoints', function
  it('oidc authorizationURL should be properly set', function () {
  configuration.authorizationURL.should.eql('https://jj.auth0.com/authorize');
  });
- 
+
  it('oidc tokenURL should be properly set', function () {
  configuration.tokenURL.should.eql('https://jj.auth0.com/oauth/token');
  });
- 
+
  it('oidc userInfoURL should be properly set', function () {
  configuration.userInfoURL.should.eql('https://jj.auth0.com/userinfo');
  });
@@ -94,4 +95,4 @@ describe('constructor with right params and pre-configured endpoints', function
  strategy._skipUserProfile.should.be.false();
  });
 
-});
+});