make update-gofmt

Author: stlaz

options.go

@@ -84,7 +84,7 @@ type Options struct {
 	ValidateURL    string `flag:"validate-url" cfg:"validate_url"`
 	Scope          string `flag:"scope" cfg:"scope"`
 	ApprovalPrompt string `flag:"approval-prompt" cfg:"approval_prompt"`
-	RequestLogging bool `flag:"request-logging" cfg:"request_logging"`
+	RequestLogging bool   `flag:"request-logging" cfg:"request_logging"`
 
 	SignatureKey string   `flag:"signature-key" cfg:"signature_key" env:"OAUTH2_PROXY_SIGNATURE_KEY"`
 	UpstreamCAs  []string `flag:"upstream-ca" cfg:"upstream_ca"`
@@ -99,7 +99,7 @@ type Options struct {
 	// Logging out of the console will destroy the user's token. The logoutRedirect
 	// provides the user the option to perform single logout (SLO) through the identity
 	// provider to destroy their single sign-on session.
-	LogoutRedirectURL       string `flag:"logout-url" cfg:"logout_url"`
+	LogoutRedirectURL string `flag:"logout-url" cfg:"logout_url"`
 
 	// internal values that are set after config validation
 	redirectURL       *url.URL

options_test.go

@@ -73,9 +73,9 @@ func TestProxyURLs(t *testing.T) {
 	assert.Equal(t, nil, o.Validate(&testProvider{}))
 	t.Logf("%#v / %#v", o.Upstreams, o.proxyURLs)
 	expected := []*url.URL{
-		&url.URL{Scheme: "http", Host: "127.0.0.1:8080", Path: "/"},
+		{Scheme: "http", Host: "127.0.0.1:8080", Path: "/"},
 		// note the '/' was added
-		&url.URL{Scheme: "http", Host: "127.0.0.1:8081", Path: "/"},
+		{Scheme: "http", Host: "127.0.0.1:8081", Path: "/"},
 	}
 	assert.Equal(t, expected, o.proxyURLs)
 }
@@ -228,4 +228,4 @@ func TestValidateCookieSameSite(t *testing.T) {
 			assert.Equal(t, nil, o.Validate(&testProvider{}))
 		})
 	}
-}
+}

providers/internal_util_test.go

@@ -44,7 +44,7 @@ func NewValidateSessionStateTest() *ValidateSessionStateTest {
 			if token_param == "" {
 				missing := false
 				received_headers := r.Header
-				for k, _ := range vt_test.header {
+				for k := range vt_test.header {
 					received := received_headers.Get(k)
 					expected := vt_test.header.Get(k)
 					if received == "" || received != expected {

providers/openshift/provider.go

@@ -187,7 +187,7 @@ func parseSubjectAccessReviewsByHost(review string) (map[string][]string, error)
 	}
 
 	reviews := make(map[string][]string)
-	for k, _ := range json.MustMap() {
+	for k := range json.MustMap() {
 		data, err := json.Get(k).EncodePretty()
 		if err != nil {
 			return nil, err