Skip to content

Commit

Permalink
initial commit
Browse files Browse the repository at this point in the history
  • Loading branch information
Julius von Froreich committed Jul 26, 2018
0 parents commit de59293
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 0 deletions.
22 changes: 22 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# Node LTS
FROM node:8.11.3-slim

RUN apt-get update && \
apt-get install -y --no-install-recommends \
git \
python \
make \
gcc \
g++ \
libc6-dev

RUN git clone https://github.com/Financial-Times/polyfill-service.git /polyfill

WORKDIR /polyfill

RUN npm run install

EXPOSE 8801

This comment has been minimized.

Copy link
@FlorianWendelborn

FlorianWendelborn Jul 31, 2018

Member

@fvj I don’t think it’s intentional that this is exposing 8801, but using PORT=80.


CMD ["node" ,"--optimize_for_size" ,"--max_old_space_size=460" ,"--gc_interval=100" ,"packages/polyfill-service/bin/polyfill-service"]

2 changes: 2 additions & 0 deletions environment
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
PORT=80

This comment has been minimized.

Copy link
@FlorianWendelborn

FlorianWendelborn Jul 31, 2018

Member

@fvj These could be added as ENV PORT 80 to the Dockerfile.

NODE_ENV=dev

This comment has been minimized.

Copy link
@FlorianWendelborn

FlorianWendelborn Jul 31, 2018

Member

@fvj Is it intentional that you’re using a dev environment?

0 comments on commit de59293

Please sign in to comment.