Display a better error on unauthorized

Not being logged in is a common user error, and the message we display can be more specific to an end user. Provide a friendly message with the server message in parethesis (in case this is a more complex server error).

Display a better error on unauthorized
dccf114f · Clayton Coleman · a6d9b091 · dccf114f
Commit dccf114f authored Mar 15, 2016 by Clayton Coleman
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 2 deletions

helpers.go pkg/kubectl/cmd/util/helpers.go +7 -2

No files found.
--- a/pkg/kubectl/cmd/util/helpers.go
+++ b/pkg/kubectl/cmd/util/helpers.go
@@ -171,10 +171,15 @@ func StandardErrorMessage(err error) (string, bool) {
 	if debugErr, ok := err.(debugError); ok {
 		glog.V(4).Infof(debugErr.DebugError())
 	}
-	_, isStatus := err.(errors.APIStatus)
+	status, isStatus := err.(errors.APIStatus)
 	switch {
 	case isStatus:
-		return fmt.Sprintf("Error from server: %s", err.Error()), true
+		switch s := status.Status(); {
+		case s.Reason == "Unauthorized":
+			return fmt.Sprintf("error: You must be logged in to the server (%s)", s.Message), true
+		default:
+			return fmt.Sprintf("Error from server: %s", err.Error()), true
+		}
 	case errors.IsUnexpectedObjectError(err):
 		return fmt.Sprintf("Server returned an unexpected response: %s", err.Error()), true
 	}