Commit62734c3

committed

fix: Force bash for non-standard flags to exec

This addresses a regression introduced by#224. POSIX sh does notdefine arguments to the exec built-in, resulting in an error whenusing 'exec -a' to set the process name (argv[0]).This change executes /bin/bash instead of sh, and also changesto use 'exec -l', which automatically handles the hyphen prefixrequired to trigger login shell behavior.This also explicitly runs /bin/sh, aligning the behavior of"coder sh" and the frontend Terminal application.

1 parentb8067c0 commit62734c3Copy full SHA for 62734c3

File tree

1 file changed

-4

lines changed

internal/cmd
- shell.go

1 file changed

-4

lines changed

`‎internal/cmd/shell.go`

Lines changed: 9 additions & 4 deletions

Original file line number	Diff line number	Diff line change
@@ -82,15 +82,20 @@ coder sh front-end-dev cat ~/config.json`,
`82`	`82`
`83`	`83`	`funcshell(cmd*cobra.Command,cmdArgs []string)error {`
`84`	`84`	`ctx:=cmd.Context()`
`85`		`-command:="sh"`
`86`		`-args:=[]string{"-c"}`
	`85`	`+varcommandstring`
	`86`	`+varargs []string`
`87`	`87`	`iflen(cmdArgs)>1 {`
	`88`	`+command="/bin/sh"`
	`89`	`+args= []string{"-c"}`
`88`	`90`	`args=append(args,strings.Join(cmdArgs[1:]," "))`
`89`	`91`	`}else {`
`90`	`92`	`// Bring user into shell if no command is specified.`
`91`	`93`	`shell:="$(getent passwd $(id -u) \| cut -d: -f 7)"`
`92`		`-name:="-$(basename "+shell+")"`
`93`		`-args=append(args,fmt.Sprintf("exec -a %q %q",name,shell))`
	`94`	`+`
	`95`	`+// force bash for the '-l' flag to the exec built-in`
	`96`	`+command="/bin/bash"`
	`97`	`+args= []string{"-c"}`
	`98`	`+args=append(args,fmt.Sprintf("exec -l %q",shell))`
`94`	`99`	`}`
`95`	`100`
`96`	`101`	`envName:=cmdArgs[0]`

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!

Commit62734c3

File tree

1 file changed

1 file changed

`‎internal/cmd/shell.go`

0 commit comments