re-enable router metrics test

openshift · Dec 15, 2017 · f6b0568 · f6b0568
1 parent b7120a8
commit f6b0568
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 11 deletions.
diff --git a/pkg/router/metrics/metrics.go b/pkg/router/metrics/metrics.go
@@ -98,14 +98,14 @@ func (l Listener) authorizeHandler(protected http.Handler) http.Handler {
 		}
 		scopedRecord.User = user
 		authorized, reason, err := l.Authorizer.Authorize(scopedRecord)
-		if authorized != authorizer.DecisionAllow || err != nil {
-			if !ok || errors.IsUnauthorized(err) {
-				glog.V(5).Infof("Unable to authorize: %v", err)
-				http.Error(w, fmt.Sprintf("Forbidden: %s", reason), http.StatusForbidden)
-			} else {
-				glog.V(3).Infof("Unable to authorize: %v", err)
-				http.Error(w, "Unable to authorize the user due to an error", http.StatusInternalServerError)
-			}
+		if err != nil {
+			glog.V(3).Infof("Unable to authorize: %v", err)
+			http.Error(w, "Unable to authorize the user due to an error", http.StatusInternalServerError)
+			return
+		}
+		if authorized != authorizer.DecisionAllow {
+			glog.V(5).Infof("Unable to authorize: %v", err)
+			http.Error(w, fmt.Sprintf("Forbidden: %s", reason), http.StatusForbidden)
 			return
 		}
 		protected.ServeHTTP(w, req)

diff --git a/test/extended/util/test.go b/test/extended/util/test.go
@@ -260,9 +260,6 @@ var (
 		// these broke in the rebase, but everything else is working
 		`should function for intra-pod communication`,
 		`should function for node-pod communication`,
-		// consistent 500 errors
-		"should expose prometheus metrics for a route",
-		"should expose the profiling endpoints",
 
 		`\[Skipped\]`,
 		`\[Slow\]`,