-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
Cannot read property 'Symbol(type)' of undefined #24546
Comments
3 tasks
leeight
added a commit
to leeight/node
that referenced
this issue
Nov 22, 2018
`finishSessionDestroy` already done session cleaning up. Fix: nodejs#24546
Trott
pushed a commit
to Trott/io.js
that referenced
this issue
Nov 25, 2018
`finishSessionDestroy` session cleanup when already done. PR-URL: nodejs#24547 Fixes: nodejs#24546 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ouyang Yadong <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Fixed in 895fe2f |
targos
pushed a commit
that referenced
this issue
Nov 25, 2018
`finishSessionDestroy` session cleanup when already done. PR-URL: #24547 Fixes: #24546 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ouyang Yadong <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Thanks! |
rvagg
pushed a commit
that referenced
this issue
Nov 28, 2018
`finishSessionDestroy` session cleanup when already done. PR-URL: #24547 Fixes: #24546 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ouyang Yadong <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
refack
pushed a commit
to refack/node
that referenced
this issue
Jan 14, 2019
`finishSessionDestroy` session cleanup when already done. PR-URL: nodejs#24547 Fixes: nodejs#24546 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ouyang Yadong <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
BethGriggs
pushed a commit
that referenced
this issue
Feb 11, 2019
`finishSessionDestroy` session cleanup when already done. PR-URL: #24547 Fixes: #24546 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ouyang Yadong <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
rvagg
pushed a commit
that referenced
this issue
Feb 28, 2019
`finishSessionDestroy` session cleanup when already done. PR-URL: #24547 Fixes: #24546 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ouyang Yadong <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Sample program:
Exception:
Cause is
destroy()
happening (and setting[kSession]
toundefined
) beforeafterShutdown
then calls[kMaybeDestroy]()
.The example shows a simple repro but if timing is wrong then it can also 'just happen' (session is destroyed while stream is shutting down.
The text was updated successfully, but these errors were encountered: