Commit35341eb

committed

fix: add dbauthz.AsSystemRestricted in updatecheck, when creating first user, and when registering InMemoryProvisionerd

1 parenta351e29 commit35341ebCopy full SHA for 35341eb

File tree

-4

lines changed

-4

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -791,7 +791,7 @@ func (api *API) CreateInMemoryProvisionerDaemon(ctx context.Context, debounce ti`
`791`	`791`	`}()`
`792`	`792`
`793`	`793`	`name:=namesgenerator.GetRandomName(1)`
`794`		`-daemon,err:=api.Database.InsertProvisionerDaemon(ctx, database.InsertProvisionerDaemonParams{`
	`794`	`+daemon,err:=api.Database.InsertProvisionerDaemon(dbauthz.AsSystemRestricted(ctx), database.InsertProvisionerDaemonParams{`
`795`	`795`	`ID:uuid.New(),`
`796`	`796`	`CreatedAt:database.Now(),`
`797`	`797`	`Name:name,`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@ import (`
`20`	`20`	`"cdr.dev/slog"`
`21`	`21`
`22`	`22`	`"github.com/coder/coder/coderd/database"`
	`23`	`+"github.com/coder/coder/coderd/database/dbauthz"`
`23`	`24`	`)`
`24`	`25`
`25`	`26`	`const (`
`@@ -224,7 +225,7 @@ func (c *Checker) notifyIfNewer(prev, next Result) {`
`224`	`225`	`}`
`225`	`226`
`226`	`227`	`func (c*Checker)lastUpdateCheck(ctx context.Context) (rResult,errerror) {`
`227`		`-s,err:=c.db.GetLastUpdateCheck(ctx)`
	`228`	`+s,err:=c.db.GetLastUpdateCheck(dbauthz.AsSystemRestricted(ctx))`
`228`	`229`	`iferr!=nil {`
`229`	`230`	`returnr,err`
`230`	`231`	`}`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ import (`
`37`	`37`	`// @Router /users/first [get]`
`38`	`38`	`func (apiAPI)firstUser(rw http.ResponseWriter,rhttp.Request) {`
`39`	`39`	`ctx:=r.Context()`
`40`		`-userCount,err:=api.Database.GetUserCount(ctx)`
	`40`	`+userCount,err:=api.Database.GetUserCount(dbauthz.AsSystemRestricted(ctx))`
`41`	`41`	`iferr!=nil {`
`42`	`42`	`httpapi.Write(ctx,rw,http.StatusInternalServerError, codersdk.Response{`
`43`	`43`	`Message:"Internal error fetching user count.",`
`@@ -78,7 +78,7 @@ func (api API) postFirstUser(rw http.ResponseWriter, r http.Request) {`
`78`	`78`	`}`
`79`	`79`
`80`	`80`	`// This should only function for the first user.`
`81`		`-userCount,err:=api.Database.GetUserCount(ctx)`
	`81`	`+userCount,err:=api.Database.GetUserCount(dbauthz.AsSystemRestricted(ctx))`
`82`	`82`	`iferr!=nil {`
`83`	`83`	`httpapi.Write(ctx,rw,http.StatusInternalServerError, codersdk.Response{`
`84`	`84`	`Message:"Internal error fetching user count.",`

Comments

(0)