Merge pull request #14 from moul/fix-13

Fix panic when running on older GoTTY server (#13)
moul · Oct 10, 2015 · 7ff8568 · 7ff8568
2 parents 76be79f + 97da70d
commit 7ff8568
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -79,6 +79,7 @@ Mon Aug 24 18:54:38 CEST 2015
 
 ### master (unreleased)
 
+* Fix panic when running on older GoTTY server ([#13](https://github.com/moul/gotty-client/issues/13))
 * Add 'homebrew support' ([#1](https://github.com/moul/gotty-client/issues/1))
 * Add Changelog ([#5](https://github.com/moul/gotty-client/issues/5))
 * Add GOXC configuration to build binaries for multiple architectures ([#2](https://github.com/moul/gotty-client/issues/2))

diff --git a/gotty-client.go b/gotty-client.go
@@ -111,7 +111,12 @@ func (c *Client) GetAuthToken() (string, error) {
 	}
 
 	re := regexp.MustCompile("var gotty_auth_token = '(.*)'")
-	return re.FindStringSubmatch(string(body))[1], nil
+	output := re.FindStringSubmatch(string(body))
+	if len(output) == 0 {
+		return "", fmt.Errorf("Cannot fetch GoTTY auth-token, please upgrade your GoTTY server.")
+	}
+
+	return output[1], nil
 }
 
 // Connect tries to dial a websocket server