Merge pull request negz#26 from negz/err

Provide more context when authentication errors occur
viafoura · Mar 12, 2018 · af9266a · af9266a
2 parents fa5a882 + 0023286
commit af9266a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/frontend/src/kuberos.vue b/frontend/src/kuberos.vue
@@ -1,7 +1,7 @@
 <template>
   <div id="kuberos">
     <b-alert v-if="error" show variant="danger">
-      <p>Could not connect to authentication API: {{error.response.status}} {{error.response.statusText}}</p>
+      <p>Authentication failed: {{error.response.status}} {{error.response.statusText}}: {{error.response.data}}</p>
     </b-alert>
     <b-container v-else fluid>
       <b-row><br /></b-row>

diff --git a/kuberos.go b/kuberos.go
@@ -54,7 +54,7 @@ var (
 	ErrInvalidKubeCfgEndpoint = errors.New("invalid redirect endpoint")
 
 	// ErrInvalidState indicates the provided state param was not as expected.
-	ErrInvalidState = errors.New("invalid state parameter")
+	ErrInvalidState = errors.New("invalid state parameter: user agent or IP address changed between requests")
 
 	// ErrMissingCode indicates a response without an OAuth 2.0 authorization
 	// code