Various fixes for local user export setup

ref DEV-2363
authgear · Dec 12, 2024 · 8d451ee · 8d451ee
2 parents 4a3f6c4 + ded4903
commit 8d451ee
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -184,7 +184,7 @@ This project uses asdf, and there is a .tool-versions file at the project root.
 
 ```sh
 docker compose up -d minio
-docker compose exec -it bash
+docker compose exec -it minio bash
 
 # Inside the container
 mc alias set local http://localhost:9000 "$MINIO_ROOT_USER" "$MINIO_ROOT_PASSWORD"

diff --git a/pkg/lib/userexport/service.go b/pkg/lib/userexport/service.go
@@ -184,6 +184,12 @@ func (s *UserExportService) convertDBUserToRecord(user *user.UserForExport) (rec
 }
 
 func (s *UserExportService) ExportRecords(ctx context.Context, request *Request, task *redisqueue.Task) (outputFilename string, err error) {
+	defer func() {
+		if err != nil {
+			s.Logger.WithError(err).Error("export failed")
+		}
+	}()
+
 	resultFile, err := os.CreateTemp("", fmt.Sprintf("export-%s.tmp", task.ID))
 	if err != nil {
 		return