Add DB and docker #2
Conversation
cmd/user/main.go
Outdated
|
|
||
| query, args, err := builderInsert.ToSql() | ||
| if err != nil { | ||
| log.Fatalf("failed to build query: %v", err) |
There was a problem hiding this comment.
ронять прод не очень хорошее дело))
There was a problem hiding this comment.
Заменила Fatalf на просто печать сообщения и возврат ошибки без завершения работы приложения.
cmd/user/main.go
Outdated
| // Hashing the password. | ||
| hashedPassword, err := bcrypt.GenerateFromPassword([]byte(req.GetPassword()), bcryptCost) | ||
| if err != nil { | ||
| panic(err) |
There was a problem hiding this comment.
сомнительно конечно панику кидать, лучше ошибку вернуть)
There was a problem hiding this comment.
Заменила panic на просто печать сообщения и возврат ошибки без завершения работы приложения.
cmd/user/main.go
Outdated
| if err != nil { | ||
| log.Fatalf("failed to generate ID: %v", err) | ||
| log.Fatalf("failed to create user: %v", err) |
There was a problem hiding this comment.
Заменила Fatalf на просто печать сообщения и возврат ошибки без завершения работы приложения.
cmd/user/main.go
Outdated
|
|
||
| query, args, err := builderSelect.ToSql() | ||
| if err != nil { | ||
| log.Fatalf("failed to build query: %v", err) |
There was a problem hiding this comment.
Заменила Fatalf на просто печать сообщения и возврат ошибки без завершения работы приложения.
cmd/user/main.go
Outdated
| if err == pgx.ErrNoRows { | ||
| return nil, err | ||
| } | ||
| log.Fatalf("failed to select user: %v", err) |
There was a problem hiding this comment.
Заменила Fatalf на просто печать сообщения и возврат ошибки без завершения работы приложения.
cmd/user/main.go
Outdated
|
|
||
| query, args, err := builderUpdate.ToSql() | ||
| if err != nil { | ||
| log.Fatalf("failed to build query: %v", err) |
There was a problem hiding this comment.
Заменила Fatalf на просто печать сообщения и возврат ошибки без завершения работы приложения.
cmd/user/main.go
Outdated
|
|
||
| s.pool.QueryRow(ctx, query, args...) |
There was a problem hiding this comment.
Заменила на Exec с последующей обработкой ошибки.
cmd/user/main.go
Outdated
| log.Fatalf("failed to build query: %v", err) | ||
| } | ||
|
|
||
| s.pool.QueryRow(ctx, query, args...) |
There was a problem hiding this comment.
Заменила на Exec с последующей обработкой ошибки.
cmd/user/main.go
Outdated
|
|
||
| query, args, err := builderDelete.ToSql() | ||
| if err != nil { | ||
| log.Fatalf("failed to build query: %v", err) |
There was a problem hiding this comment.
Заменила Fatalf на просто печать сообщения и возврат ошибки без завершения работы приложения.
Added: