response.download does not return /proc/* files

[mleblebici]

response.download() returns empty for file under /proc directory like /proc/meminfo, /proc/cpuinfo and others. /proc is a pseudo filesystem, but these files' content can be readable with fs.readFile. I think it should also be readable with response.download and response.sendFile.

[dougwilson]

Hi @mleblebici yes, I agree they should be able to work. Unfortunately I just use Windows mainly and don't really have experience there on what would be wrong. Express is just using fs.createReadStream under the hood to read them. Would you be willing to make a pull request with a fix?

[mleblebici]

Hi @dougwilson, it seems the problem stems from send module. Normally fs.createReadStream is able to read contents of /proc/* files, but when using send module, it introduces a problem somewhere. I used following code and it was not able to read the contents.

var http = require('http')
var send = require('send')

var server = http.createServer(function onRequest(req, res) {
	send(req, '/proc/version').pipe(res)
})

server.listen(1234)

[dougwilson]

Hi @mleblebici thanks for investigating! Can you open a bug over there? Ideally if you can open a pull request with a fix, as I don't think I can personally fix it due to not having the /proc/ stuff on my system to use for testing what is wrong.

[mleblebici]

Hi @dougwilson , I also investigated the problem with send library. It seems fs.stat returns size of 0 for /proc/* files. So, I added a getSize function to correctly get size of /proc/* files. It fixed the problem. But, as I am not a developer (especially not a JS developer), you should probably check to make sure it does not cause any performance issue or create any additional bug.
Issue Link: pillarjs/send#212
PR Link: pillarjs/send#213

[dougwilson]

Thanks! Have you checked with the Node.js project that fs.stat returning a size of 0 for those files is not simply a bug in their stat function? We'll want some kind of confirmation before landing a change like that.

[krzysdz]

Virtual files such as /proc/meminfo don't have a known size and stat syscall returns size 0 for them.

[dougwilson]

Thanks @krzysdz ! Well, that sucks, as normal files that are zero size also return zero :(