summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--context.go9
1 files changed, 9 insertions, 0 deletions
diff --git a/context.go b/context.go
index 2af38ff..9661aca 100644
--- a/context.go
+++ b/context.go
@@ -100,6 +100,15 @@ func (ctx Context) LoadModule(name string, rawMsg json.RawMessage) (interface{},
}
}
+ if val == nil {
+ // returned module values are almost always type-asserted
+ // before being used, so a nil value would panic; and there
+ // is no good reason to explicitly declare null modules in
+ // a config; it might be because the user is trying to
+ // achieve a result they aren't expecting, which is a smell
+ return nil, fmt.Errorf("module value cannot be null")
+ }
+
if prov, ok := val.(Provisioner); ok {
err := prov.Provision(ctx)
if err != nil {