Skip to content

Commit

Permalink
Merge pull request #5052 from terraform-providers/b-aws_codebuild_pro…
Browse files Browse the repository at this point in the history
…ject-env-type-panic

resource/aws_codebuild_project: Prevent panic with missing environment variable type
  • Loading branch information
bflad authored Jul 2, 2018
2 parents 56c6e6c + 92971fd commit 47a8934
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion aws/resource_aws_codebuild_project.go
Original file line number Diff line number Diff line change
Expand Up @@ -835,7 +835,12 @@ func resourceAwsCodeBuildProjectEnvironmentHash(v interface{}) int {
for _, e := range environmentVariables {
if e != nil { // Old statefiles might have nil values in them
ev := e.(map[string]interface{})
buf.WriteString(fmt.Sprintf("%s:%s:%s-", ev["name"].(string), ev["type"].(string), ev["value"].(string)))
buf.WriteString(fmt.Sprintf("%s:", ev["name"].(string)))
// type is sometimes not returned by the API
if v, ok := ev["type"]; ok {
buf.WriteString(fmt.Sprintf("%s:", v.(string)))
}
buf.WriteString(fmt.Sprintf("%s-", ev["value"].(string)))
}
}

Expand Down

0 comments on commit 47a8934

Please sign in to comment.