feat(docker): switch to node:dubnium-stretch-slim for a faster and mo…

…re modern docker build

Dockerfile

@@ -1,23 +1,17 @@
-#
-# Dockerfile for ytdl-webserver
-#
-FROM ubuntu:16.04
+FROM node:dubnium-stretch-slim
 
 WORKDIR /home/app
 
-RUN apt-get update \
-    && apt-get install -y curl ffmpeg \
-    && curl -sL https://deb.nodesource.com/setup_10.x | bash - \
-    && apt-get install -y nodejs \
-    && npm install -g npm@latest \
+RUN apt update \
+    && apt install -y curl ffmpeg \
     && rm -rf /var/lib/apt/lists/*
 
 # This is on a separate line because youtube-dl needs to be frequently updated
-RUN apt-get update \
-    && apt-get install -y youtube-dl \
+RUN apt update \
+    && apt install -y youtube-dl \
     && rm -rf /var/lib/apt/lists/*
 
-# only install node_modules if the package.json changes
+# Only install node_modules if the package.json changes
 COPY package.json package-lock.json ./
 RUN npm ci
 

src/server/server.js

@@ -33,7 +33,7 @@ const provision = async () => {
   server.route({
     method: 'POST',
     path: '/download',
-    handler: (request, reply) => {
+    handler: (request) => {
       const url = request.payload.url
       const options = {
         path: path.join(__dirname, '../../public/temp'),
@@ -53,9 +53,9 @@ const provision = async () => {
   server.route({
     method: 'GET',
     path: '/request/{video}',
-    handler: (request, reply) => {
+    handler: (request, h) => {
       const videoName = encodeURIComponent(request.params.video)
-      reply.file(path.join('temp', decodeURIComponent(videoName)))
+      return h.file(path.join('temp', decodeURIComponent(videoName)))
     }
   })