From 3e8526bc51e0e8f298a75bb94d900829c5775de9 Mon Sep 17 00:00:00 2001
From: Kyle Carberry <kyle@carberry.com>
Date: Fri, 23 Sep 2022 16:27:12 +0000
Subject: [PATCH] fix: Remove unused scopes from parameter computation

---
 coderd/parameter/compute.go      | 2 --
 coderd/parameter/compute_test.go | 2 --
 coderd/provisionerdaemons.go     | 6 ------
 coderd/templateversions.go       | 3 ---
 4 files changed, 13 deletions(-)

diff --git a/coderd/parameter/compute.go b/coderd/parameter/compute.go
index 4531c4aa74d8e..c291e730f0f9f 100644
--- a/coderd/parameter/compute.go
+++ b/coderd/parameter/compute.go
@@ -15,8 +15,6 @@ import (
 // ComputeScope targets identifiers to pull parameters from.
 type ComputeScope struct {
 	TemplateImportJobID       uuid.UUID
-	OrganizationID            uuid.UUID
-	UserID                    uuid.UUID
 	TemplateID                uuid.NullUUID
 	WorkspaceID               uuid.NullUUID
 	AdditionalParameterValues []database.ParameterValue
diff --git a/coderd/parameter/compute_test.go b/coderd/parameter/compute_test.go
index d84608a3c3e9c..f68008b3f48b8 100644
--- a/coderd/parameter/compute_test.go
+++ b/coderd/parameter/compute_test.go
@@ -18,7 +18,6 @@ func TestCompute(t *testing.T) {
 	generateScope := func() parameter.ComputeScope {
 		return parameter.ComputeScope{
 			TemplateImportJobID: uuid.New(),
-			OrganizationID:      uuid.New(),
 			TemplateID: uuid.NullUUID{
 				UUID:  uuid.New(),
 				Valid: true,
@@ -27,7 +26,6 @@ func TestCompute(t *testing.T) {
 				UUID:  uuid.New(),
 				Valid: true,
 			},
-			UserID: uuid.New(),
 		}
 	}
 	type parameterOptions struct {
diff --git a/coderd/provisionerdaemons.go b/coderd/provisionerdaemons.go
index 58eb2315e16c2..55920bfa0d32e 100644
--- a/coderd/provisionerdaemons.go
+++ b/coderd/provisionerdaemons.go
@@ -226,12 +226,10 @@ func (server *provisionerdServer) AcquireJob(ctx context.Context, _ *proto.Empty
 		// Compute parameters for the workspace to consume.
 		parameters, err := parameter.Compute(ctx, server.Database, parameter.ComputeScope{
 			TemplateImportJobID: templateVersion.JobID,
-			OrganizationID:      job.OrganizationID,
 			TemplateID: uuid.NullUUID{
 				UUID:  template.ID,
 				Valid: true,
 			},
-			UserID: user.ID,
 			WorkspaceID: uuid.NullUUID{
 				UUID:  workspace.ID,
 				Valid: true,
@@ -283,9 +281,7 @@ func (server *provisionerdServer) AcquireJob(ctx context.Context, _ *proto.Empty
 		// Compute parameters for the dry-run to consume.
 		parameters, err := parameter.Compute(ctx, server.Database, parameter.ComputeScope{
 			TemplateImportJobID:       templateVersion.JobID,
-			OrganizationID:            job.OrganizationID,
 			TemplateID:                templateVersion.TemplateID,
-			UserID:                    user.ID,
 			WorkspaceID:               uuid.NullUUID{},
 			AdditionalParameterValues: input.ParameterValues,
 		}, nil)
@@ -476,8 +472,6 @@ func (server *provisionerdServer) UpdateJob(ctx context.Context, request *proto.
 		parameters, err := parameter.Compute(ctx, server.Database, parameter.ComputeScope{
 			TemplateImportJobID: job.ID,
 			TemplateID:          templateID,
-			OrganizationID:      job.OrganizationID,
-			UserID:              job.InitiatorID,
 		}, nil)
 		if err != nil {
 			return nil, xerrors.Errorf("compute parameters: %w", err)
diff --git a/coderd/templateversions.go b/coderd/templateversions.go
index 03945a6f9b57f..54820b49eacf7 100644
--- a/coderd/templateversions.go
+++ b/coderd/templateversions.go
@@ -148,7 +148,6 @@ func (api *API) templateVersionSchema(rw http.ResponseWriter, r *http.Request) {
 
 func (api *API) templateVersionParameters(rw http.ResponseWriter, r *http.Request) {
 	ctx := r.Context()
-	apiKey := httpmw.APIKey(r)
 	templateVersion := httpmw.TemplateVersionParam(r)
 	if !api.Authorize(r, rbac.ActionRead, templateVersion) {
 		httpapi.ResourceNotFound(rw)
@@ -171,8 +170,6 @@ func (api *API) templateVersionParameters(rw http.ResponseWriter, r *http.Reques
 	}
 	values, err := parameter.Compute(ctx, api.Database, parameter.ComputeScope{
 		TemplateImportJobID: job.ID,
-		OrganizationID:      job.OrganizationID,
-		UserID:              apiKey.UserID,
 	}, &parameter.ComputeOptions{
 		// We *never* want to send the client secret parameter values.
 		HideRedisplayValues: true,

<!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.0 Transitional//EN' 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd'>
<html xmlns='http://www.w3.org/1999/xhtml'>
<head>
<title>pFad - Phonifier reborn</title>
<meta http-equiv='Content-Type' content='text/html; charset=utf-8' />
</head>
<body>
<h1>Pfad - The Proxy pFad of &#169; 2024 Garber Painting. All rights reserved.</h1>


<!-- Disclaimer -->
<p>Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.</p>
<br>
<p>Alternative Proxies:</p><p><a href="http://rainy.clevelandohioweatherforecast.com/php-proxy/index.php?q=https://patch-diff.githubusercontent.com/raw/coder/coder/pull/4171.patch" target="_blank">Alternative Proxy</a></p><p><a href="http://rainy.clevelandohioweatherforecast.com/pFad/index.php?u=https://patch-diff.githubusercontent.com/raw/coder/coder/pull/4171.patch" target="_blank">pFad Proxy</a></p><p><a href="http://rainy.clevelandohioweatherforecast.com/pFad/v3index.php?u=https://patch-diff.githubusercontent.com/raw/coder/coder/pull/4171.patch" target="_blank">pFad v3 Proxy</a></p><p><a href="http://rainy.clevelandohioweatherforecast.com/pFad/v4index.php?u=https://patch-diff.githubusercontent.com/raw/coder/coder/pull/4171.patch" target="_blank">pFad v4 Proxy</a></p></body>
</html>