Require APP_KEY to be defined in environment
ClosedPublic
Actions

Authored by dereckson on Feb 24 2020, 09:18.

Details

Reviewers

Commits

rDCACHET9e48de4e9b32: Require APP_KEY to be defined in environment

Summary

The Cachet documentation stresses APP_KEY should be stable and not volatile, as
the lack of key would "result in all of your encrypted/hashed data being lost".

Reference: https://docs.cachethq.io/docs/installing-cachet

Test Plan

Run container with and without APP_KEY set

Diff Detail

Repository

rDCACHET Cachet Docker image

Lint

No Lint Coverage

Unit

No Test Coverage

Branch

master

Build Status

Buildable 3490
Build 3739: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Feb 24 2020, 09:18

dereckson created this revision.

Harbormaster completed remote builds in B3490: Diff 5667.Feb 24 2020, 09:18

dereckson added inline comments.Feb 24 2020, 09:23

files/usr/local/sbin/init-container
3	/usr/local/sbin/init-container: 3: [: -Z: unexpected operator

-Z -> -z

Harbormaster completed remote builds in B3491: Diff 5668.Feb 24 2020, 09:24

Forgot $

Harbormaster completed remote builds in B3492: Diff 5669.Feb 24 2020, 09:49

dereckson accepted this revision.Feb 24 2020, 09:49

This revision is now accepted and ready to land.Feb 24 2020, 09:49

dereckson edited the test plan for this revision. (Show Details)Feb 24 2020, 09:49

Closed by commit rDCACHET9e48de4e9b32: Require APP_KEY to be defined in environment (authored by dereckson). · Explain WhyFeb 24 2020, 09:50

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rDCACHET9e48de4e9b32: Require APP_KEY to be defined in environment.

Revision Contents
Changeset List

Path

Size

files/

usr/

local/

sbin/

init-container

11 lines

Diff 5667

View Options

files/usr/local/sbin/init-container

Require APP_KEY to be defined in environmentClosedPublicActions