Change default token-prefix to proper one for the Alerta service

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:

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.

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",

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",

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"`

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 {