fixup! chore(coderd): add tests for big oidc tokens

johnstcn · johnstcn · commit e4fba39025a6 · 2024-03-05T11:28:41.000Z
diff --git a/coderd/userauth.go b/coderd/userauth.go
@@ -6,15 +6,13 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"io"
 	"net/http"
 	"net/mail"
 	"regexp"
 	"sort"
 	"strconv"
 	"strings"
 	"sync"
-	"sync/atomic"
 	"time"
 
 	"github.com/coreos/go-oidc/v3/oidc"
@@ -1191,26 +1189,6 @@ func blankFields(claims map[string]interface{}) []string {
 	return fields
 }
 
-// fieldSizes returns the sorted sizes of fields in the claims map
-func fieldSizes(claims map[string]interface{}) []int {
-	fs := claimFields(claims)
-	lens := make([]int, len(fs))
-	for i, f := range fs {
-		v, ok := claims[f]
-		if !ok {
-			lens[i] = -1
-			continue
-		}
-		vs, ok := v.(string)
-		if !ok {
-			lens[i] = -1
-			continue
-		}
-		lens[i] = len(vs)
-	}
-	return lens
-}
-
 // mergeClaims merges the claims from a and b and returns the merged set.
 // claims from b take precedence over claims from a.
 func mergeClaims(a, b map[string]interface{}) map[string]interface{} {
@@ -1894,20 +1872,3 @@ func parseStringSliceClaim(claim interface{}) ([]string, error) {
 	// Not sure what the user gave us.
 	return nil, xerrors.Errorf("invalid claim type. Expected an array of strings, got: %T", claim)
 }
-
-type countWriter struct {
-	w            io.Writer
-	bytesWritten atomic.Int64
-}
-
-func (cw *countWriter) Write(p []byte) (int, error) {
-	n, err := cw.w.Write(p)
-	if n > 0 {
-		cw.bytesWritten.Add(int64(n))
-	}
-	return n, err
-}
-
-func (cw *countWriter) Length() int {
-	return int(cw.bytesWritten.Load())
-}
