NotificationsYou must be signed in to change notification settings
Fork1.1k
Star11.6k

Commitc12bba4

authored

fix(enterprise/cli): preserve actual error when getting provisioner key details (#20813)

Fixes#20781Previously, when `GetProvisionerKey()` failed, the actual error wasswallowed:```error: unable to get provisioner key details```Now the actual error is preserved using error wrapping, so users can seethe real cause (e.g., 404 Not Found, connection refused, invalid key,etc.):```error: unable to get provisioner key details: GET https://...: 404 Not Found```This makes it much easier to diagnose configuration issues.---*Generated by [mux](https://cmux.io)*

1 parent158243d commitc12bba4Copy full SHA for c12bba4

File tree

1 file changed

-1

lines changed

enterprise/cli
- provisionerdaemonstart.go

1 file changed

-1

lines changed

`‎enterprise/cli/provisionerdaemonstart.go‎`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ func (r RootCmd) provisionerDaemonStart() serpent.Command {`
`107`	`107`	`ifprovisionerKey!="" {`
`108`	`108`	`pkDetails,err:=client.GetProvisionerKey(ctx,provisionerKey)`
`109`	`109`	`iferr!=nil {`
`110`		`-returnxerrors.New("unable to get provisioner key details")`
	`110`	`+returnxerrors.Errorf("unable to get provisioner key details: %w",err)`
`111`	`111`	`}`
`112`	`112`
`113`	`113`	`fork,v:=rangepkDetails.Tags {`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitc12bba4

File tree

1 file changed

1 file changed

`‎enterprise/cli/provisionerdaemonstart.go‎`

0 commit comments