Skip to content
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

New version that binds promises on then() instead of resolve() #141

Open
rochdev opened this issue Dec 13, 2018 · 0 comments
Open

New version that binds promises on then() instead of resolve() #141

rochdev opened this issue Dec 13, 2018 · 0 comments

Comments

@rochdev
Copy link

rochdev commented Dec 13, 2018

This is a follow up from last week's OTSC meeting. Basically, most if not all other implementations (zone.js, domains, cls-hooked, etc) bind promises on then(). This makes interoperability with this module very difficult for libraries that must support context propagation in multiple Node versions.

@watson Should there be some kind of v2 branch where we could merge our branch currently living in https://github.com/DataDog/async-listener/tree/datadog ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant