Merge pull request #20 from Comcast/status-forbidden

Changed StatusUnauthorized to StatusForbidden

bascule/basculehttp/constructor.go

@@ -58,7 +58,7 @@ func (c *constructor) decorate(next http.Handler) http.Handler {
 		if err != nil {
 			logger.Log(level.Key(), level.ErrorValue(), bascule.ErrorKey, err.Error(), "key", key,
 				"auth", authorization[i+1:])
-			WriteResponse(response, http.StatusUnauthorized, err)
+			WriteResponse(response, http.StatusForbidden, err)
 			return
 		}
 

bascule/basculehttp/constructor_test.go

@@ -65,7 +65,7 @@ func TestConstructor(t *testing.T) {
 			constructor:        c,
 			requestHeaderKey:   testHeader,
 			requestHeaderValue: "Basic AFJDK",
-			expectedStatusCode: http.StatusUnauthorized,
+			expectedStatusCode: http.StatusForbidden,
 		},
 	}
 	for _, tc := range tests {

bascule/basculehttp/enforcer.go

@@ -64,7 +64,7 @@ func (e *enforcer) decorate(next http.Handler) http.Handler {
 					}
 				}
 				logger.Log(append(emperror.Context(err), level.Key(), level.ErrorValue(), bascule.ErrorKey, errs)...)
-				WriteResponse(response, http.StatusUnauthorized, err)
+				WriteResponse(response, http.StatusForbidden, err)
 				return
 			}
 		}

bascule/basculehttp/enforcer_test.go

@@ -64,7 +64,7 @@ func TestEnforcer(t *testing.T) {
 				Authorization: "jwt",
 				Token:         bascule.NewToken("", "", bascule.Attributes{}),
 			},
-			expectedStatusCode: http.StatusUnauthorized,
+			expectedStatusCode: http.StatusForbidden,
 		},
 	}
 	for _, tc := range tests {