Merge pull request #25 from mozilla/download_from_url
Changed naming for some pages, no longer stores files by name on server
This commit is contained in:
commit
1163f2b65b
16
app.js
16
app.js
@ -26,12 +26,17 @@ app.get('/assets/download/:id', function(req, res) {
|
||||
let id = req.params.id;
|
||||
client.hget(id, "filename", function(err, reply) { // maybe some expiration logic too
|
||||
if (!reply) {
|
||||
res.send('error');
|
||||
res.sendStatus(404);
|
||||
} else {
|
||||
res.setHeader('Content-Disposition', 'attachment; filename=' + reply);
|
||||
res.setHeader('Content-Type', 'application/octet-stream');
|
||||
|
||||
res.download(__dirname + '/static/' + reply);
|
||||
|
||||
res.download(__dirname + '/static/' + id, reply, function(err) {
|
||||
if (!err) {
|
||||
client.del(id);
|
||||
fs.unlinkSync(__dirname + '/static/' + id);
|
||||
}
|
||||
});
|
||||
}
|
||||
})
|
||||
|
||||
@ -46,13 +51,14 @@ app.route('/upload/:id')
|
||||
console.log("Uploading: " + filename);
|
||||
|
||||
//Path where image will be uploaded
|
||||
fstream = fs.createWriteStream(__dirname + '/static/' + filename);
|
||||
fstream = fs.createWriteStream(__dirname + '/static/' + req.params.id);
|
||||
file.pipe(fstream);
|
||||
fstream.on('close', function () {
|
||||
let id = req.params.id;
|
||||
client.hset(id, "filename", filename, redis.print);
|
||||
client.hset(id, "expiration", 0, redis.print);
|
||||
console.log("Upload Finished of " + filename);
|
||||
client.expire(id, 86400000);
|
||||
console.log("Upload Finished of " + filename);
|
||||
res.send(id);
|
||||
});
|
||||
});
|
||||
|
@ -1,7 +1,7 @@
|
||||
<!DOCTYPE html>
|
||||
<html>
|
||||
<head>
|
||||
<title>Page Title</title>
|
||||
<title>Download your file</title>
|
||||
<script type="text/javascript" src="/file.js"></script>
|
||||
</head>
|
||||
<body>
|
||||
|
@ -44,11 +44,13 @@ function download() {
|
||||
var downloadUrl = URL.createObjectURL(blob);
|
||||
var a = document.createElement("a");
|
||||
a.href = downloadUrl;
|
||||
a.download = xhr.getResponseHeader('Content-Disposition').match(/filename="(.+)"/)[1];;
|
||||
a.download = xhr.getResponseHeader('Content-Disposition').match(/filename="(.+)"/)[1];
|
||||
console.log(xhr.getResponseHeader('Content-Disposition'));
|
||||
document.body.appendChild(a);
|
||||
a.click();
|
||||
})
|
||||
.catch(function(err){
|
||||
alert('This link is either invalid or has expired.');
|
||||
console.error(err);
|
||||
});
|
||||
})
|
||||
|
@ -1,7 +1,7 @@
|
||||
<!DOCTYPE html>
|
||||
<html>
|
||||
<head>
|
||||
<title>Page Title</title>
|
||||
<title>Firefox Fileshare</title>
|
||||
<script src="file.js"></script>
|
||||
|
||||
</head>
|
||||
|
@ -1 +1 @@
|
||||
This is where files will go.
|
||||
This is where downloaded files are stored.
|
||||
|
Loading…
Reference in New Issue
Block a user