Skip to content

Commit ac8886b

Browse files
appleboyclaude
andauthored
fix(docker): create home directory for codegpt user in container (#260)
- Replace file.IsDir check with direct os.MkdirAll to avoid fatal error when config directory path does not exist - Remove -H flag from adduser so the home directory is created for the codegpt user in Docker Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent d299214 commit ac8886b

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

cmd/cmd.go

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -93,14 +93,8 @@ func initConfig() {
9393
viper.SetConfigName(".codegpt")
9494
cfgFile = path.Join(configFolder, ".codegpt.yaml")
9595

96-
isDir, err := file.IsDir(configFolder)
97-
if err != nil {
98-
log.Fatalf("failed to check if config folder %s is a directory: %v", configFolder, err)
99-
}
100-
if !isDir {
101-
if err := os.MkdirAll(configFolder, os.ModePerm); err != nil {
102-
log.Fatal(err)
103-
}
96+
if err := os.MkdirAll(configFolder, os.ModePerm); err != nil {
97+
log.Fatalf("failed to create config folder %s: %v", configFolder, err)
10498
}
10599
}
106100

docker/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ LABEL org.opencontainers.image.licenses=MIT
1414

1515
RUN apk add --no-cache ca-certificates git && \
1616
rm -rf /var/cache/apk/* && \
17-
adduser -D -H -u 1000 codegpt
17+
adduser -D -u 1000 codegpt
1818

1919
COPY release/${TARGETOS}/${TARGETARCH}/codegpt /bin/
2020

0 commit comments

Comments
 (0)