diff --git a/CHANGELOG.md b/CHANGELOG.md index 305f69f..68a7ff9 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -45,3 +45,9 @@ **Fix** - Bump dependencies versions (no breaking changes) [\#37](https://github.com/auth0/auth0-guardian.js/pull/37) ([dafortune](https://github.com/dafortune)). + +# [v1.2.1](https://github.com/auth0/auth0-guardian.js/tree/v1.1.1) (2017-06-13) +[Full Changelog](https://github.com/auth0/auth0-guardian.js/compare/v1.2.1...v1.2.0) + +**Fix** +- Fix enrollment:complete event in polling mode [\#37](https://github.com/auth0/auth0-guardian.js/pull/38) ([dafortune](https://github.com/dafortune)). diff --git a/lib/utils/polling_client.js b/lib/utils/polling_client.js index bcca0b5..8338f09 100644 --- a/lib/utils/polling_client.js +++ b/lib/utils/polling_client.js @@ -184,7 +184,8 @@ pollingClient.prototype.emitEvents = function emitEvents(err, oldState, newState } if (oldState && oldState.state === 'pending' && - !oldState.enrollment && newState.enrollment) { + (!oldState.enrollment || oldState.enrollment.available_methods.length === 0) && + (newState.enrollment && newState.enrollment.available_methods.length > 0)) { self.hub.emit('enrollment:confirmed', object.toCamelKeys({ device_account: newState.enrollment, tx_id: newState.id diff --git a/package.json b/package.json index 0da3ba1..f3f47c5 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "auth0-guardian-js", - "version": "1.2.0", + "version": "1.2.1", "description": "Interface", "main": "index.js", "directories": { diff --git a/test/utils/polling_client.test.js b/test/utils/polling_client.test.js index 3900967..e3d1457 100644 --- a/test/utils/polling_client.test.js +++ b/test/utils/polling_client.test.js @@ -227,7 +227,8 @@ describe('utils/polling_client', function () { state: 'pending', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: ['sms'] } }, events: [ @@ -235,6 +236,7 @@ describe('utils/polling_client', function () { name: 'enrollment:confirmed', payload: { deviceAccount: { + availableMethods: ['sms'], id: 'dev_123', test: 'test' }, @@ -257,7 +259,8 @@ describe('utils/polling_client', function () { state: 'accepted', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: ['push'] }, token: '123.123.1234' }, @@ -267,7 +270,8 @@ describe('utils/polling_client', function () { payload: { deviceAccount: { id: 'dev_123', - test: 'test' + test: 'test', + availableMethods: ['push'] }, txId: 'tx_1234' } @@ -296,7 +300,8 @@ describe('utils/polling_client', function () { state: 'rejected', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: ['totp'] } }, events: [ @@ -305,7 +310,8 @@ describe('utils/polling_client', function () { payload: { deviceAccount: { id: 'dev_123', - test: 'test' + test: 'test', + availableMethods: ['totp'] }, txId: 'tx_1234' } @@ -328,7 +334,8 @@ describe('utils/polling_client', function () { state: 'pending', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: [] } }, transitionState: { @@ -336,7 +343,8 @@ describe('utils/polling_client', function () { state: 'rejected', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: [] } }, events: [ @@ -358,7 +366,8 @@ describe('utils/polling_client', function () { state: 'pending', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: [] } }, transitionState: { @@ -366,7 +375,8 @@ describe('utils/polling_client', function () { state: 'accepted', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: [] }, token: '123.123.1234' }, @@ -395,7 +405,8 @@ describe('utils/polling_client', function () { state: 'accepted', enrollment: { id: 'dev_123', - test: 'test' + test: 'test', + available_methods: ['totp'] }, token: '123.123.1234' }, @@ -405,7 +416,8 @@ describe('utils/polling_client', function () { payload: { deviceAccount: { id: 'dev_123', - test: 'test' + test: 'test', + availableMethods: ['totp'] }, txId: 'tx_1234' }