Commit2438749

committed

Merge branch 'validate-aws-credentials' into 'master'

fix: check AWS credentials and notify about missing environment variablesSee merge request postgres-ai/database-lab!130

2 parents2352410 +beb806d commit2438749Copy full SHA for 2438749

File tree

-0

lines changed

-0

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,14 @@ func newRDSDumper(rdsCfg RDSConfig) (rdsDumper, error) {`
`68`	`68`	`returnnil,errors.Wrap(err,"failed to start AWS session")`
`69`	`69`	`}`
`70`	`70`
	`71`	`+credentials,err:=awsSession.Config.Credentials.Get()`
	`72`	`+iferr!=nil\|\|!credentials.HasKeys() {`
	`73`	`+log.Dbg(err)`
	`74`	`+`
	`75`	+returnnil,errors.New(`failed to check AWS credentials.
	`76`	+Set up valid environment variables AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY`)
	`77`	`+}`
	`78`	`+`
`71`	`79`	`return&rdsDumper{`
`72`	`80`	`rdsCfg:rdsCfg,`
`73`	`81`	`iamSvc:iam.New(awsSession,aws.NewConfig()),`

Comments

(0)