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
fix(playback): consistently check for destroyed
attribute
#2152
Conversation
Node, as of 14.1, started migrating the Client Request terminology from `aborted` to `destroyed`. In order to supported our current Node version support, 10.x+, we need to check both flags. Nock was already doing this in the router, but not during playback.
d3762c4
to
b8629dc
Compare
* | ||
* @param {ClientRequest} req | ||
* @returns {boolean} | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
love all the context in the comment 👏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same! Thank you, @mastermatt.
🎉 This PR is included in version 13.0.9 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Node, as of 14.1, started migrating the Client Request terminology from
aborted
todestroyed
. In order to supported our current Node versionsupport, 10.x+, we need to check both flags. Nock was already doing this
in the router, but not during playback.
Identified from #2146