From 0061978db805ff3aa84011d45022604c13d21678 Mon Sep 17 00:00:00 2001 From: Nicola Murino Date: Fri, 14 Feb 2020 08:29:31 +0100 Subject: [PATCH] docker: better clarify when ca-certificates package is needed --- docker/sftpgo/alpine/Dockerfile | 6 ++++-- docker/sftpgo/debian/Dockerfile | 4 +++- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/docker/sftpgo/alpine/Dockerfile b/docker/sftpgo/alpine/Dockerfile index 1d798c11..6deffcab 100644 --- a/docker/sftpgo/alpine/Dockerfile +++ b/docker/sftpgo/alpine/Dockerfile @@ -12,8 +12,10 @@ FROM alpine:latest RUN apk add --no-cache ca-certificates su-exec \ && mkdir -p /data /etc/sftpgo /srv/sftpgo/config /srv/sftpgo/web /srv/sftpgo/backups -# git and rsync are optional, uncomment the next line to add support for them if needed -#RUN apk add --no-cache git rsync +# ca-certificates is needed for Cloud Storage Support and to expose the REST API over HTTPS. +# If you install git then ca-certificates will be automatically installed as dependency. +# git, rsync and ca-certificates are optional, uncomment the next line to add support for them if needed. +#RUN apk add --no-cache git rsync ca-certificates COPY --from=builder /go/bin/sftpgo /bin/ COPY --from=builder /go/src/github.com/drakkan/sftpgo/sftpgo.json /etc/sftpgo/sftpgo.json diff --git a/docker/sftpgo/debian/Dockerfile b/docker/sftpgo/debian/Dockerfile index 2ce37f1e..0e2a8795 100644 --- a/docker/sftpgo/debian/Dockerfile +++ b/docker/sftpgo/debian/Dockerfile @@ -10,7 +10,9 @@ RUN go build -i -ldflags "-s -w -X github.com/drakkan/sftpgo/utils.commit=`git d # now define the run environment FROM debian:latest -# git, rsync and ca-certificates are optional, uncomment the next line to add support for them if needed. (ca-certificate if you're going to use S3 binding feature) +# ca-certificates is needed for Cloud Storage Support and to expose the REST API over HTTPS. +# If you install git then ca-certificates will be automatically installed as dependency. +# git, rsync and ca-certificates are optional, uncomment the next line to add support for them if needed. #RUN apt-get update && apt-get install -y git rsync ca-certificates ARG BASE_DIR=/app