Skip to content

Commit

Permalink
Bases on another image to fix issue.
Browse files Browse the repository at this point in the history
Due to bug [#66](igrigorik/node-spdyproxy#66)
we need to use another base image.

Image size: 210 MB -> 370 MB
  • Loading branch information
wernight committed Oct 4, 2016
1 parent c390a5a commit 56186fb
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
9 changes: 8 additions & 1 deletion Dockerfile
@@ -1,4 +1,11 @@
FROM node:6-slim
FROM debian:jessie

RUN set -x \
# Install requirements.
&& apt-get update \
&& apt-get install -y curl npm \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/*

RUN set -x \
# Install SPDY Proxy.
Expand Down
6 changes: 3 additions & 3 deletions entrypoint.sh
Expand Up @@ -12,10 +12,10 @@ if [ -z "$*" ]
then
if [ -z $SPDY_USERNAME ] || [ -z $SPDY_PASSWORD ]
then
exec spdyproxy --verbose --key /ssl/tls.key --cert /ssl/tls.crt
exec nodejs /usr/local/bin/spdyproxy --verbose --key /ssl/tls.key --cert /ssl/tls.crt
else
exec spdyproxy --verbose --key /ssl/tls.key --cert /ssl/tls.crt --user $SPDY_USERNAME --pass $SPDY_PASSWORD
exec nodejs /usr/local/bin/spdyproxy --verbose --key /ssl/tls.key --cert /ssl/tls.crt --user $SPDY_USERNAME --pass $SPDY_PASSWORD
fi
else
exec spdyproxy "$@"
exec nodejs /usr/local/bin/spdyproxy "$@"
fi

0 comments on commit 56186fb

Please sign in to comment.