go-park-mail-ru/2018_2_42

2018_2_42's Issues

Много инитов - плохо, нужен 1

2018_2_42/authorization_server/accessor/insert.go

Line 111 in c238175

func init() {

Использовать менеджер зависимостей dep.

С dep можно будет сделать так:

RUN go get github.com/golang/dep/cmd/dep

WORKDIR ${GOPATH}/src/github.com/go-park-mail-ru/2018_2_42/authorization_server
COPY Gopkg.toml .
COPY Gopkg.lock .
RUN dep ensure -vendor-only

COPY . .

Это даже было в дз №5.

Разбить handlers.go на несколько файлов

почему мидлвары в хендлерах?

2018_2_42/authorization_server/handlers/handlers.go

Line 49 in 1413ac6

func CommonMiddleware(next http.Handler) http.Handler {

РК 1

• хендлеры в отдельный пакет (0)

• хардкод доступов к бд

• перенести api doc в swagger (2)

Не юзать через глобальную переменную

2018_2_42/authorization_server/accessor/insert.go

Line 22 in c238175

var Db DB // собственный тип, что бы прикреплять к нему функции с бизнес логикой.

Вызывать привязать мидлвары ко всем хэндлерам сразу, без вызова каждый раз

2018_2_42/authorization_server/main.go

Line 12 in c238175

http.Handle("/api/v1/user", handlers.CORSMiddleware(

Функция должна влезать в 1 экран, убрать повторяющиеся куски

Вызывать из sql файла

2018_2_42/authorization_server/accessor/insert.go

Line 50 in c238175

begin transaction;