Setting a variable default to `null` dumps a stack trace

[ddaws]

Describe the bug

Setting the default value for a variable to null errors, dumping a stack trace. This is the pattern for defining Terraform variables as optional and seems like it should be supported.

Steps to Reproduce

variable "my_api_key" {
  type    = string
  default = null # optional
}

Here is a Waypoint file that reproduces this and the logs / stack trace:

https://gist.github.com/ddaws/d1b14d43e2daf06af918a22ebdf945c9

Expected behavior

Waypoint should support null default values to indicate a variable is optional.

Waypoint Platform Versions

Additional version and platform information to help triage the issue if
applicable:

• Waypoint CLI Version: CLI: v0.10.2 (2c9926e)
• Waypoint Server Platform and Version: Server: v0.10.2, docker
• Waypoint Plugin: vault

Additional context

My goal is to be able to source secrets from Vault, or optionally pass them at command line.

[briancain]

Hey @ddaws - yep, that looks to be a bug! Thanks for reporting this! I think we should at least fix the panic here. Otherwise for now you should be ok setting the default to empty string to avoid the panic.