From 9b902428ea828d606b2d1a75b21350b8cfa1a99d Mon Sep 17 00:00:00 2001
From: Lars Lehtonen <lars.lehtonen@gmail.com>
Date: Mon, 15 Mar 2021 17:08:25 -0700
Subject: [PATCH] server: prune unused constants and functions (#477)

This removes unused `const` variables and functions from `server` and its subpackages.
---
 server/datastore/mysql/mysql.go         |  4 ----
 server/kolide/users_test.go             |  2 --
 server/service/service_sessions_test.go |  2 --
 server/sso/authorization_response.go    |  7 -------
 server/sso/types.go                     |  3 ---
 server/websocket/websocket.go           | 10 ----------
 6 files changed, 28 deletions(-)

diff --git a/server/datastore/mysql/mysql.go b/server/datastore/mysql/mysql.go
index 0b7b0fb1f0..d9297aa536 100644
--- a/server/datastore/mysql/mysql.go
+++ b/server/datastore/mysql/mysql.go
@@ -275,10 +275,6 @@ func (d *Datastore) Close() error {
 	return d.db.Close()
 }
 
-func (d *Datastore) log(msg string) {
-	d.logger.Log("comp", d.Name(), "msg", msg)
-}
-
 func sanitizeColumn(col string) string {
 	return columnCharsRegexp.ReplaceAllString(col, "")
 }
diff --git a/server/kolide/users_test.go b/server/kolide/users_test.go
index 5a431f9792..cec43a6d3b 100644
--- a/server/kolide/users_test.go
+++ b/server/kolide/users_test.go
@@ -18,8 +18,6 @@ func TestValidatePassword(t *testing.T) {
 		{"jason", "bar0baz!?", "jason@kolide.co", true, false},
 	}
 
-	const bcryptCost = 6
-
 	for _, tt := range passwordTests {
 		user := newTestUser(t, tt.Username, tt.Password, tt.Email)
 
diff --git a/server/service/service_sessions_test.go b/server/service/service_sessions_test.go
index cb8ed32f59..3bb5959631 100644
--- a/server/service/service_sessions_test.go
+++ b/server/service/service_sessions_test.go
@@ -13,8 +13,6 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-const bcryptCost = 6
-
 func TestAuthenticate(t *testing.T) {
 	ds, err := inmem.New(config.TestConfig())
 	require.Nil(t, err)
diff --git a/server/sso/authorization_response.go b/server/sso/authorization_response.go
index eb6636f826..88ff921034 100644
--- a/server/sso/authorization_response.go
+++ b/server/sso/authorization_response.go
@@ -106,13 +106,6 @@ func (r resp) rawResponse() string {
 	return r.rawResp
 }
 
-func (r resp) authResponse() (*Response, error) {
-	if r.response != nil {
-		return r.response, nil
-	}
-	return nil, errors.New("missing SAML response")
-}
-
 // DecodeAuthResponse extracts SAML assertions from IDP response
 func DecodeAuthResponse(samlResponse string) (kolide.Auth, error) {
 	var authInfo resp
diff --git a/server/sso/types.go b/server/sso/types.go
index 37b929b7f2..707430c885 100644
--- a/server/sso/types.go
+++ b/server/sso/types.go
@@ -21,7 +21,6 @@ type AuthnRequest struct {
 	NameIDPolicy                *NameIDPolicy          `xml:"NameIDPolicy,omitempty"`
 	RequestedAuthnContext       *RequestedAuthnContext `xml:"RequestedAuthnContext,omitempty"`
 	Signature                   *Signature             `xml:"Signature,omitempty"`
-	originalString              string
 }
 
 // Response is submitted to the service provider (Kolide) from the IDP via a callback.
@@ -43,8 +42,6 @@ type Response struct {
 	Signature Signature `xml:"Signature"`
 	Issuer    Issuer    `xml:"Issuer"`
 	Status    Status    `xml:"Status"`
-
-	originalString string
 }
 
 type Issuer struct {
diff --git a/server/websocket/websocket.go b/server/websocket/websocket.go
index 28487114b2..46073d5541 100644
--- a/server/websocket/websocket.go
+++ b/server/websocket/websocket.go
@@ -4,7 +4,6 @@ package websocket
 
 import (
 	"encoding/json"
-	"time"
 
 	"github.com/igm/sockjs-go/v3/sockjs"
 
@@ -13,20 +12,11 @@ import (
 )
 
 const (
-	// maxMessageSize is used to set a read limit on the websocket and
-	// prevent clients from flooding us with data.
-	maxMessageSize int64 = 8096
-
 	// authType is the type string used for auth messages.
 	authType string = "auth"
 
 	// errType is the type string used for error messages.
 	errType string = "error"
-
-	// defaultTimeout is the default timeout that should be used for
-	// sending and receiving over the websocket. It is used unless
-	// Conn.Timeout is set explicitly after Upgrade is called.
-	defaultTimeout time.Duration = 3 * time.Second
 )
 
 // JSONMessage is a wrapper struct for messages that will be sent across the wire