Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adds working dockerfile #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jmbmxer
Copy link

@jmbmxer jmbmxer commented Sep 23, 2016

Created simple Dockerfile to run go-flashpaper in a container. To build run docker build . -t test/flashpaper and run using docker run -d -p 8443:8443 test/flashpaper. The service will then be available locally at https://127.0.0.1:8443 if you are running Docker for Mac. A self-signed certificate is used for TLS.

FROM golang:1.7.1-onbuild
MAINTAINER Jimmy Mesta "[email protected]"

RUN wget https://dl.eff.org/certbot-auto -P /usr/local/sbin && \
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing you were downloading certbot so you could get a real cert, but it doesn't appear to be used anywhere in the RUN stanzas.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed Let's Encrypt support and using a self-signed cert only. We found that this app works well behind a LB / proxy and terminating TLS there with a valid certificate. I'm open to other suggestions though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants