agecem/bottin
5
0
Fork 0
Code Issues 7 Pull requests Projects Releases Packages Wiki Activity

Defer DB.Close() sur chaque DataClient #22

Merged
vlbeaudoin merged 1 commit from fix/defer-dataclient-close into main 2023-06-08 19:48:57 -04:00
Conversation 0 Commits 1 Files changed 2 +3 -2
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 3870ef42dd - Show all commits

3
cmd/api.go
View file

@ -59,14 +59,13 @@ var apiCmd = &cobra.Command{
if err != nil { if err != nil {
log.Fatalf("Could not establish database connection.\n Error: %s\n", err) log.Fatalf("Could not establish database connection.\n Error: %s\n", err)
} }
defer client.DB.Close()
err = client.DB.Ping() err = client.DB.Ping()
if err != nil { if err != nil {
log.Fatalf("Database was supposed to be ready but Ping() failed.\n Error: %s\n", err) log.Fatalf("Database was supposed to be ready but Ping() failed.\n Error: %s\n", err)
} }
client.DB.Close()
e.Logger.Fatal(e.Start(fmt.Sprintf(":%d", apiPort))) e.Logger.Fatal(e.Start(fmt.Sprintf(":%d", apiPort)))
}, },
} }

2
handlers/insert.go
View file

@ -16,6 +16,7 @@ func PostMembres(c echo.Context) error {
"error": err.Error(), "error": err.Error(),
}) })
} }
defer client.DB.Close()
var membres []models.Membre var membres []models.Membre
@ -56,6 +57,7 @@ func PostProgrammes(c echo.Context) error {
"error": err.Error(), "error": err.Error(),
}) })
} }
defer client.DB.Close()
var programmes []models.Programme var programmes []models.Programme