auth0-samples / auth0-electron-samples

[DEPRECATED] Auth0-Electron Sample
MIT License
18 stars 21 forks source link

Cannot set property 'openerId' of undefined #4

Closed MaximeFrancoeur closed 7 years ago

MaximeFrancoeur commented 7 years ago

I cannot test this project with my personal clientId and domain.

After execute npm install and npm start I click on login with Facebook or Google or any else and I receive a popup with this error Cannot set property 'openerId' of undefined

Full stacktrace : 
Uncaught Exception:
TypeError: Cannot set property 'openerId' of undefined
    at mergeBrowserWindowOptions (/Users/MaximeFr/Desktop/auth0-electron-samples-sample/node_modules/electron-prebuilt/dist/Electron.app/Contents/Resources/electron.asar/browser/guest-window-manager.js:44:35)
    at EventEmitter.<anonymous> (/Users/MaximeFr/Desktop/auth0-electron-samples-sample/node_modules/electron-prebuilt/dist/Electron.app/Contents/Resources/electron.asar/browser/guest-window-manager.js:144:13)
    at emitMany (events.js:127:13)
    at EventEmitter.emit (events.js:201:7)
    at WebContents.<anonymous> (/Users/MaximeFr/Desktop/auth0-electron-samples-sample/node_modules/electron-prebuilt/dist/Electron.app/Contents/Resources/electron.asar/browser/api/web-contents.js:219:13)
    at emitTwo (events.js:106:13)
    at WebContents.emit (events.js:191:7)
MaximeFrancoeur commented 7 years ago

Any update ?

ghost commented 7 years ago

This looks to be an issue with Electron handling opening a new browser window https://github.com/electron/electron/issues/7519

The fix doesn't seem to have hit NPM yet, so the short-term solution is to change your electron version to the latest version before the bug was introduced. In package.json, under devDependencies, set the following: "electron": "1.4.2" Then run the project: npm install npm start

MaximeFrancoeur commented 7 years ago

Thank you, this solution work fine.