Skip to content

Commit 25c259d

Browse files
author
Martin
authored
Merge pull request #1576 from snyk/fix/do_not_panic_on_aws_init
fix: do not panic on aws provider init
2 parents 0cc71d1 + af980fc commit 25c259d

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

enumeration/remote/aws/provider.go

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,14 @@ func NewAWSTerraformProvider(version string, progress enumeration.ProgressCounte
6363
if err != nil {
6464
return nil, err
6565
}
66-
p.session = session.Must(session.NewSessionWithOptions(session.Options{
66+
67+
p.session, err = session.NewSessionWithOptions(session.Options{
6768
SharedConfigState: session.SharedConfigEnable,
68-
}))
69+
})
70+
if err != nil {
71+
return nil, err
72+
}
73+
6974
tfProvider, err := terraform.NewTerraformProvider(installer, terraform.TerraformProviderConfig{
7075
Name: p.name,
7176
DefaultAlias: *p.session.Config.Region,

0 commit comments

Comments
 (0)