Fix crash on invalid state and lower state size to fit as key

plugin.json

@@ -2,7 +2,7 @@
     "id": "github",
     "name": "GitHub",
     "description": "GitHub plugin for Mattermost.",
-    "version": "0.7.0",
+    "version": "0.7.1",
     "server": {
       "executables": {
           "linux-amd64": "server/dist/plugin-linux-amd64",

server/api.go

@@ -80,7 +80,7 @@ func (p *Plugin) connectUserToGitHub(w http.ResponseWriter, r *http.Request) {
 
 	conf := p.getOAuthConfig()
 
-	state := fmt.Sprintf("%v_%v", model.NewId(), userID)
+	state := fmt.Sprintf("%v_%v", model.NewId()[0:15], userID)
 
 	p.API.KVSet(state, []byte(state))
 
@@ -106,7 +106,6 @@ func (p *Plugin) completeConnectUserToGitHub(w http.ResponseWriter, r *http.Requ
 		http.Error(w, "missing stored state", http.StatusBadRequest)
 		return
 	} else if string(storedState) != state {
-		fmt.Println(err.Error())
 		http.Error(w, "invalid state", http.StatusBadRequest)
 		return
 	}

server/manifest.go

@@ -5,5 +5,5 @@ var manifest = struct {
 	Version string
 }{
 	Id:      "github",
-	Version: "0.7.0",
+	Version: "0.7.1",
 }

webapp/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github",
-  "version": "0.6.2",
+  "version": "0.7.1",
   "description": "Github plugin for Mattermost",
   "main": "src/index.js",
   "scripts": {

webapp/src/manifest.js

@@ -1,2 +1,2 @@
 export const id = 'github';
-export const version = '0.7.0';
+export const version = '0.7.1';