Remove legacy notify

kylecarbs · kylecarbs · commit 18022ba60b51 · 2023-06-28T00:55:26.000Z
diff --git a/cli/cliui/gitauth_test.go b/cli/cliui/gitauth_test.go
@@ -2,7 +2,6 @@ package cliui_test
 
 import (
 	"context"
-	"net/url"
 	"sync/atomic"
 	"testing"
 	"time"
@@ -33,7 +32,7 @@ func TestGitAuth(t *testing.T) {
 						ID:              "github",
 						Type:            codersdk.GitProviderGitHub,
 						Authenticated:   fetched.Load(),
-						AuthenticateURL: "https://example.com/gitauth/github?redirect=" + url.QueryEscape("/gitauth?notify"),
+						AuthenticateURL: "https://example.com/gitauth/github",
 					}}, nil
 				},
 				FetchInterval: time.Millisecond,
diff --git a/coderd/templateversions.go b/coderd/templateversions.go
@@ -320,14 +320,6 @@ func (api *API) templateVersionGitAuth(rw http.ResponseWriter, r *http.Request)
 			})
 			return
 		}
-		query := redirectURL.Query()
-		// The frontend uses a BroadcastChannel to notify listening pages for
-		// Git auth updates if the "notify" query parameter is set.
-		//
-		// It's important we do this in the backend, because the same endpoint
-		// is used for CLI authentication.
-		query.Add("redirect", "/gitauth?notify")
-		redirectURL.RawQuery = query.Encode()
 
 		provider := codersdk.TemplateVersionGitAuth{
 			ID:              config.ID,
