From 4f697cda95d38bee0c525f6feeeba621a58bfa8b Mon Sep 17 00:00:00 2001
From: Kyle Carberry <kyle@carberry.com>
Date: Thu, 17 Nov 2022 05:04:28 +0000
Subject: [PATCH] fix: Parse boolean in slice for gitauth

This was causing a panic for the new `no_refesh` option!
---
 cli/deployment/config.go      | 4 +++-
 cli/deployment/config_test.go | 2 ++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/cli/deployment/config.go b/cli/deployment/config.go
index ad4b6efea26c0..127b14df6f5b9 100644
--- a/cli/deployment/config.go
+++ b/cli/deployment/config.go
@@ -529,9 +529,11 @@ func readSliceFromViper[T any](vip *viper.Viper, key string, value any) []T {
 				newType := reflect.Indirect(reflect.New(elementType))
 				instance = &newType
 			}
-			switch instance.Field(i).Type().String() {
+			switch v := instance.Field(i).Type().String(); v {
 			case "[]string":
 				value = vip.GetStringSlice(configKey)
+			case "bool":
+				value = vip.GetBool(configKey)
 			default:
 			}
 			instance.Field(i).Set(reflect.ValueOf(value))
diff --git a/cli/deployment/config_test.go b/cli/deployment/config_test.go
index 84be72bcd4754..515851439ee84 100644
--- a/cli/deployment/config_test.go
+++ b/cli/deployment/config_test.go
@@ -167,6 +167,7 @@ func TestConfig(t *testing.T) {
 			"CODER_GITAUTH_0_TOKEN_URL":     "https://token.com",
 			"CODER_GITAUTH_0_REGEX":         "github.com",
 			"CODER_GITAUTH_0_SCOPES":        "read write",
+			"CODER_GITAUTH_0_NO_REFRESH":    "true",
 
 			"CODER_GITAUTH_1_ID":            "another",
 			"CODER_GITAUTH_1_TYPE":          "gitlab",
@@ -187,6 +188,7 @@ func TestConfig(t *testing.T) {
 				TokenURL:     "https://token.com",
 				Regex:        "github.com",
 				Scopes:       []string{"read", "write"},
+				NoRefresh:    true,
 			}, {
 				ID:           "another",
 				Type:         "gitlab",