Merge pull request #14746 from jaken551/master

14722 - removing unnecessary arn check that breaks assume role featur…

Merge pull request #14746 from jaken551/master
14722 - removing unnecessary arn check that breaks assume role featur…
56edba5a · Torkel Ödegaard · GitHub · a9077e1a · 64163f01 · 56edba5a
Commit 56edba5a authored Jan 07, 2019 by Torkel Ödegaard Committed by GitHub Jan 07, 2019
Show whitespace changes
Inline Side-by-side

Showing with 1 additions and 2 deletions

pkg/tsdb/cloudwatch/credentials.go
+1 -2

No files found.
--- a/pkg/tsdb/cloudwatch/credentials.go
+++ b/pkg/tsdb/cloudwatch/credentials.go
@@ -3,7 +3,6 @@ package cloudwatch
 import (
 	"fmt"
 	"os"
-	"strings"
 	"sync"
 	"time"

@@ -43,7 +42,7 @@ func GetCredentials(dsInfo *DatasourceInfo) (*credentials.Credentials, error) {
 	secretAccessKey := ""
 	sessionToken := ""
 	var expiration *time.Time = nil
-	if dsInfo.AuthType == "arn" && strings.Index(dsInfo.AssumeRoleArn, "arn:aws:iam:") == 0 {
+	if dsInfo.AuthType == "arn" {
 		params := &sts.AssumeRoleInput{
 			RoleArn:         aws.String(dsInfo.AssumeRoleArn),
 			RoleSessionName: aws.String("GrafanaSession"),