From 52ae22457756fd91e4f166550c16f3f9127dc346 Mon Sep 17 00:00:00 2001
From: Sebastian Czoch <sebastian@czoch.pl>
Date: Wed, 10 Jan 2018 20:32:03 +0100
Subject: [PATCH] Change default token-prefix to proper one for the Alerta
 service

---
 CHANGELOG.md                  | 6 ++++++
 etc/kapacitor/kapacitor.conf  | 3 +--
 integrations/streamer_test.go | 4 ++--
 server/server_test.go         | 4 ++--
 services/alerta/config.go     | 2 +-
 services/alerta/service.go    | 2 +-
 6 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4e1e1c256a..6ce8649ae2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+### Bugfixes
+
+- [#1165]((https://github.com/influxdata/kapacitor/issues/1165): Fix wrong default "token-prefix" value for the alerta service.
+
 ## v1.4.0 [2017-12-08]
 
 The v1.4.0 release has many new features, here is a list of some of the highlights:
diff --git a/etc/kapacitor/kapacitor.conf b/etc/kapacitor/kapacitor.conf
index 921f34c642..39db3ea097 100644
--- a/etc/kapacitor/kapacitor.conf
+++ b/etc/kapacitor/kapacitor.conf
@@ -409,8 +409,7 @@ default-retention-policy = ""
   # Default authentication token.
   token = ""
   # Default token prefix
-  # If you are on older versions of alerta you may need to change this to "Key"
-  token-prefix = "Bearer"
+  token-prefix = "Key"
   # Default environment.
   environment = ""
   # Default origin.
diff --git a/integrations/streamer_test.go b/integrations/streamer_test.go
index a9e47bd8f6..03b89ec1c1 100644
--- a/integrations/streamer_test.go
+++ b/integrations/streamer_test.go
@@ -8692,7 +8692,7 @@ stream
 	exp := []interface{}{
 		alertatest.Request{
 			URL:           "/alert",
-			Authorization: "Bearer testtoken1234567",
+			Authorization: "Key testtoken1234567",
 			PostData: alertatest.PostData{
 				Resource:    "cpu",
 				Event:       "serverA",
@@ -8706,7 +8706,7 @@ stream
 		},
 		alertatest.Request{
 			URL:           "/alert",
-			Authorization: "Bearer anothertesttoken",
+			Authorization: "Key anothertesttoken",
 			PostData: alertatest.PostData{
 				Resource:    "resource: serverA",
 				Event:       "event: TestStream_Alert",
diff --git a/server/server_test.go b/server/server_test.go
index 0724c75af7..9662660515 100644
--- a/server/server_test.go
+++ b/server/server_test.go
@@ -8703,7 +8703,7 @@ func TestServer_AlertHandlers(t *testing.T) {
 				Kind: "alerta",
 				Options: map[string]interface{}{
 					"token":        "testtoken1234567",
-					"token-prefix": "Bearer",
+					"token-prefix": "Key",
 					"origin":       "kapacitor",
 					"group":        "test",
 					"environment":  "env",
@@ -8724,7 +8724,7 @@ func TestServer_AlertHandlers(t *testing.T) {
 				got := ts.Requests()
 				exp := []alertatest.Request{{
 					URL:           "/alert",
-					Authorization: "Bearer testtoken1234567",
+					Authorization: "Key testtoken1234567",
 					PostData: alertatest.PostData{
 						Resource:    "alert",
 						Event:       "id",
diff --git a/services/alerta/config.go b/services/alerta/config.go
index 839afb855f..98c3d6871e 100644
--- a/services/alerta/config.go
+++ b/services/alerta/config.go
@@ -17,7 +17,7 @@ type Config struct {
 	// The authentication token for this notification, can be overridden per alert.
 	Token string `toml:"token" override:"token,redact"`
 	// The prefix for the Authentication field where the token is stored
-	// This defaults to Bearer but you may need to set this to "Key" for older versions of alerta
+	// This defaults to Key
 	TokenPrefix string `toml:"token-prefix" override:"token-prefix"`
 	// The environment in which to raise the alert.
 	Environment string `toml:"environment" override:"environment"`
diff --git a/services/alerta/service.go b/services/alerta/service.go
index cea6d1c1c1..e8baacebba 100644
--- a/services/alerta/service.go
+++ b/services/alerta/service.go
@@ -24,7 +24,7 @@ const (
 	defaultEvent       = "{{ .ID }}"
 	defaultGroup       = "{{ .Group }}"
 	defaultTimeout     = time.Duration(24 * time.Hour)
-	defaultTokenPrefix = "Bearer"
+	defaultTokenPrefix = "Key"
 )
 
 type Diagnostic interface {