diff options
author | Matthew Holt <mholt@users.noreply.github.com> | 2020-03-26 14:02:29 -0600 |
---|---|---|
committer | Matthew Holt <mholt@users.noreply.github.com> | 2020-03-26 14:02:29 -0600 |
commit | ea3688e1c057a042bbae97175f42eb4ec3029667 (patch) | |
tree | fda2b46b322872d7d24bba1bcc3d6983a597bb8d /modules | |
parent | c87f82f0ce58ec714b3e13fbe69c322a0d612c67 (diff) |
caddytls: Remove ManageSync
This seems unnecessary for now and we can always add it in later if
people have a good reason to need it.
Diffstat (limited to 'modules')
-rw-r--r-- | modules/caddytls/automation.go | 9 | ||||
-rw-r--r-- | modules/caddytls/tls.go | 13 |
2 files changed, 4 insertions, 18 deletions
diff --git a/modules/caddytls/automation.go b/modules/caddytls/automation.go index fcf6454..24a21cb 100644 --- a/modules/caddytls/automation.go +++ b/modules/caddytls/automation.go @@ -97,15 +97,6 @@ type AutomationPolicy struct { // load. OnDemand bool `json:"on_demand,omitempty"` - // If true, certificate management will be conducted - // in the foreground; this will block config reloads - // and return errors if there were problems with - // obtaining or renewing certificates. This is often - // not desirable, especially when serving sites out - // of your control. Default: false - // TODO: is this really necessary per-policy? why not a global setting... - ManageSync bool `json:"manage_sync,omitempty"` - // Issuer stores the decoded issuer parameters. This is only // used to populate an underlying certmagic.Config's Issuer // field; it is not referenced thereafter. diff --git a/modules/caddytls/tls.go b/modules/caddytls/tls.go index b2c6324..54f0e23 100644 --- a/modules/caddytls/tls.go +++ b/modules/caddytls/tls.go @@ -247,8 +247,8 @@ func (t *TLS) Cleanup() error { func (t *TLS) Manage(names []string) error { // for a large number of names, we can be more memory-efficient // by making only one certmagic.Config for all the names that - // use that config, rather than calling ManageSync/ManageAsync - // once for every name; so first, bin names by AutomationPolicy + // use that config, rather than calling ManageAsync once for + // every name; so first, bin names by AutomationPolicy policyToNames := make(map[*AutomationPolicy][]string) for _, name := range names { ap := t.getAutomationPolicyForName(name) @@ -257,14 +257,9 @@ func (t *TLS) Manage(names []string) error { // now that names are grouped by policy, we can simply make one // certmagic.Config for each (potentially large) group of names - // and call ManageSync/ManageAsync just once for the whole batch + // and call ManageAsync just once for the whole batch for ap, names := range policyToNames { - var err error - if ap.ManageSync { - err = ap.magic.ManageSync(names) - } else { - err = ap.magic.ManageAsync(t.ctx.Context, names) - } + err := ap.magic.ManageAsync(t.ctx.Context, names) if err != nil { return fmt.Errorf("automate: manage %v: %v", names, err) } |