-
Notifications
You must be signed in to change notification settings - Fork 30k
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
V8's native Promise misbehaves when resolved with a thenable #2747
Comments
/cc @littledan who I know was looking for promise issues that are causing pain in the wild. |
@Fishrock123 Are we promoting V8 promises? |
https://nodejs.org/en/blog/release/v4.0.0/
|
What, should we pretend they don't exist? I wouldn't say we are promoting them though. I'd definitely not recommend using native promises, at least for now. Actually let's just override the global variable for 5.x+ |
Thanks for the bug report and drawing my attention to it. This looks pretty significant. Sorry it hasn't been addressed so far. |
Closing as the originally reported issue (https://code.google.com/p/v8/issues/detail?id=4162) appears to have been fixed. |
Thanks again for the bug report; please let me know if you have any more issues with Promises. |
I see the v4 post is promoting V8 promises, so just wanted to make sure this issue is on your radar for LTS considerations.
https://code.google.com/p/v8/issues/detail?id=4162
More discussion and test case in zloirock/core-js#78
The text was updated successfully, but these errors were encountered: