Home Original page

test: stdin is not always a net.Socket by Fishrock123 · Pull Request #5935 · nodejs/node

@Fishrock123 added test

Issues and PRs related to the tests.

process

Issues and PRs related to the process subsystem.

labels

Mar 28, 2016

@Fishrock123 

`<`-ing a file into stdin actually results in a `fs.ReadStream`, rather
than a `tty.ReadStream`, and as such does not inherit from net.Socket,
unlike the other possible stdin options.

Refs: nodejs#5916
PR-URL: nodejs#5935
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

thefourtheye added a commit to thefourtheye/io.js that referenced this pull request

Mar 31, 2016

thefourtheye added a commit that referenced this pull request

Mar 31, 2016 
Refer: #5935
PR-URL: #5980
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Phillip Johnsen <johphi@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Apr 5, 2016 
`<`-ing a file into stdin actually results in a `fs.ReadStream`, rather
than a `tty.ReadStream`, and as such does not inherit from net.Socket,
unlike the other possible stdin options.

Refs: #5916
PR-URL: #5935
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Apr 5, 2016 
Refer: #5935
PR-URL: #5980
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Phillip Johnsen <johphi@gmail.com>

This was referenced

Apr 5, 2016

MylesBorins pushed a commit that referenced this pull request

Apr 11, 2016 
`<`-ing a file into stdin actually results in a `fs.ReadStream`, rather
than a `tty.ReadStream`, and as such does not inherit from net.Socket,
unlike the other possible stdin options.

Refs: #5916
PR-URL: #5935
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Apr 11, 2016 
Refer: #5935
PR-URL: #5980
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Phillip Johnsen <johphi@gmail.com>