-
Notifications
You must be signed in to change notification settings - Fork 29.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't kill node.js / REPL #11342
Comments
Easy way to reproduce? Is this with the built-in promises implementation or a custom one? |
Oh, I assumed it had to do with the previous node.js warning similar to "soon every promise not handled will result in a bluescreen" Let me check... You mean it could be a hidden SEGFAULT? |
require('child_process').exec("chromedriver")
const driver = require('selenium-webdriver')
let doc = new driver.Builder() .usingServer('http://localhost:9515') .withCapabilities({chromeOptions: {binary: '/usr/local/bin/Electron'} }) .forBrowser('electron') .build()
doc.get("http://xyz.com")
doc.findElement(driver.By.name('abc')) forgive my ignorance: is there a kind of error which makes the REPL quit by design? |
When I said 'easy', I was hoping for something that doesn't involve all of selenium... Can you reproduce with just a bare REPL, no third-party dependencies? |
I was hoping it had to do with this warning:
looks like this is a separate issue. however since the warning indicates a horrible plan to make node.js terminate ever so often, is there any (other) place to petition against this planned behavior? |
See #8217 and the issues linked therein. Can this issue be closed then? |
thx |
Version: v7.5.0
Platform: Darwin iMac.local 16.4.0 Darwin Kernel Version 16.4.0: Thu Dec 22 22:53:21 PST 2016; root:xnu-3789.41.3~3/RELEASE_X86_64 x86_64
Somehow a broken promise kills the whole REPL:
This must never occur!
The text was updated successfully, but these errors were encountered: