From 516b74f43a3e4bb670825adff9d53053465f1759 Mon Sep 17 00:00:00 2001 From: Mariano Cano Date: Mon, 23 Aug 2021 15:33:16 -0700 Subject: [PATCH] Add comment about unused code. --- pki/pki.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/pki/pki.go b/pki/pki.go index 7ba56ad3..1b6c83e7 100644 --- a/pki/pki.go +++ b/pki/pki.go @@ -735,6 +735,13 @@ func (p *PKI) GenerateConfig(opt ...ConfigOption) (*authconfig.Config, error) { if !config.AuthorityConfig.EnableAdmin { config.AuthorityConfig.Provisioners = provisioners } else { + // At this moment this code path is never used because `step ca + // init` will always set enableAdmin to false for a standalone + // deployment. Once we move `step beta` commands out of the beta we + // should probably default to this route. + // + // Note that we might want to be able to define the database as a + // flag in `step ca init` so we can write to the proper place. db, err := db.New(config.DB) if err != nil { return nil, err