CODEWITHSUNDEEP
javascriptnode.js
lordoftherings
lordoftherings

Reputation: 11

How to make these links to download files when clicked in node.js?

I'm new to Node and i'm really stuck.

I'm trying to make my web server to look up files in current directory and to display them in browser as links so i could download them.

However the list of files gets just updated with every request and displayed over and over. I can't use any frameworks or external components and i have been stuck on this for two days. I really have done a lot of research and tried a lot of things and still can't get it working.

I'm gonna add my last attempt's code below and if anyone could help me with even a little bit of information, it would be most appreciated. Thanks!

 var http = require("http");
 var fs = require("fs");
 var currentServerDir = ".";
 var port = 8080;
 var content = "";
 var server = http.createServer(handlerRequest).listen(8080);

 function handlerRequest(request, response) {
     fs.readdir(currentServerDir, function getFiles(error, items) {
         items.forEach(function getItems(item) {
             content += "<br><a href= " + "\" " + item + "\" " + ">" + item + "</a><br>";
         });
     });
     response.writeHead(200, {
         'Content-Type': 'text/html'
     });
     response.write(content);
     response.end();
 }

EDIT:

I followed Node.js Generate html

and borrowed some code from there. Now i can click on a file, but instead of downloading or viewing it just says "undefined".

    var http = require('http');
    var content
    function getFiles()
    {
    fs.readdir(currentServerDir, function getFiles(error, items)
    {
    items.forEach(function getItems(item)
    {
    content+= "<br><a href= " + "\" " + item + "\" " + ">" +item +        "</a><br>";
    });
    });
    }

    http.createServer(function (req, res) {
    var html = buildHtml(req);

    res.writeHead(200, {
   'Content-Type': 'text/html',
   'Content-Length': html.length,
   'Expires': new Date().toUTCString()
   });

       res.end(html);

   }).listen(8080);

   function buildHtml(req) {
   var header = '';
   var body = content;

    return '<!DOCTYPE html>'
    + '<html><header>' + header + '</header><body>' + body + '</body>    </html>';
    };

Upvotes: 1

Views: 2307

Answers (3)

Sonaryr
Sonaryr

Reputation: 1122

There are a few issues in your code.

  1. You never call getFiles() resulting that content will never be filled
  2. Should you call getFiles() content will still be empty in the end because you use fs.readdir(). This is an async function and will not be able to fill content before it used to build your html page.
  3. You handle every request to your server the same, so you will not be able download any files because you will always just display your page.

You can fix the first 2 easy by using the gist AlexS posted as a comment on your question. The third one will require a bit more setup, but can be made easy if you use Express.

Upvotes: 1

Lauromine
Lauromine

Reputation: 1493

If you dont care about browser support you can use the download attribute in the a tag. It looks like this :

<a href="test.html" download="nameOfFileAfterDL">DL me!</a>

Supported by Firefox 14+ and Chrome 20+

But if you want a general solution then, you can zip your files before sending them, as far as I know .zip are always downloaded when "clicking on them".

You can use 'node-zip or archiver to zip your files.

Upvotes: 0

Limezest
Limezest

Reputation: 140

You could add the download tag to your HTML link :

items.forEach(function getItems(item)
{
   content+= "<br><a href= " + "\" " + item + "\" " + " download>" +item +        "</a><br>";
});

MDN reference

Upvotes: 0

Related Questions