Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
nodejs
/
node
Public
Uh oh!
There was an error while loading.
Please reload this page
.
Notifications
You must be signed in to change notification settings
Fork
37k
Star
122k
Code
Issues
632
Pull requests
564
Actions
Projects
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Projects
Security and quality
Insights
net/streams: don't return the stream object from onStreamRead
- #34375
#34375
Closed
robey
wants to merge 10 commits into
nodejs:master
nodejs/node:master
from
robey:onread-fix
Copy head branch name to clipboard
Conversation
Commits
10
(10)
Checks
Files changed
Closed
net/streams: don't return the stream object from onStreamRead
#34375
robey
wants to merge 10 commits into
nodejs:master
nodejs/node:master
from
robey:onread-fix
Copy head branch name to clipboard
Commits
Commits on Jul 24, 2020
streams: don't return the stream object from onStreamRead
Show description for 0865a22
robey
committed
0865a22
View commit details
Copy full SHA for 0865a22
Browse repository at this point
streams: fix 2nd instance
robey
committed
851667e
View commit details
Copy full SHA for 851667e
Browse repository at this point
add a link to the bug, by request
robey
committed
f646c23
View commit details
Copy full SHA for f646c23
Browse repository at this point
add a test that crashes without the fix.
robey
committed
eea77a5
View commit details
Copy full SHA for eea77a5
Browse repository at this point
fix linter error
robey
committed
98daa59
View commit details
Copy full SHA for 98daa59
Browse repository at this point
add extra check in stream_base.cc to be extra sure.
robey
committed
1440eae
View commit details
Copy full SHA for 1440eae
Browse repository at this point
undo "nice to have" which breaks some tests (sorry)
robey
committed
965c0b4
View commit details
Copy full SHA for 965c0b4
Browse repository at this point
give up on asserting that everyone sees the same error
robey
committed
27bc342
View commit details
Copy full SHA for 27bc342
Browse repository at this point
Commits on Jul 30, 2020
make a guess about why new tests are failing
robey
committed
4663458
View commit details
Copy full SHA for 4663458
Browse repository at this point
Commits on Jul 31, 2020
try removing the test completely
robey
committed
251a8c9
View commit details
Copy full SHA for 251a8c9
Browse repository at this point
You can’t perform that action at this time.