JWT proto with login & registration #1

Open
Mephi wants to merge 10 commits from feature/jwt into master
Owner

qwe

qwe
d3m0k1d approved these changes 2026-06-13 19:08:40 +00:00
Dismissed
d3m0k1d left a comment
Owner

В общем все расписано так же пока нет ci используй просто golangci-lint run --fix

В общем все расписано так же пока нет ci используй просто golangci-lint run --fix
README.md Outdated
@@ -1,2 +1,155 @@
# Control-plane
Owner

Вот это хорошо но лучше это или в wiki вынести

Вот это хорошо но лучше это или в wiki вынести
cmd/main.go Outdated
@@ -17,0 +44,4 @@
if err != nil {
log.Fatalf("failed to create postgres pool: %v", err)
}
defer pool.Close()
Owner

Тут полюбому проверка на ошибки нужна линтер на такое не пройдет

Тут полюбому проверка на ошибки нужна линтер на такое не пройдет
cmd/main.go Outdated
@@ -17,0 +71,4 @@
loginLimiter := auth.NewRateLimiter(10, time.Minute)
authMW := auth.AuthMiddleware([]byte(cfg.JWTSecret))
go func() {
Owner

Не уверен что хорошее решение но допустим

Не уверен что хорошее решение но допустим
cmd/main.go Outdated
@@ -26,2 +97,3 @@
})
r.Run(":8080")
api := r.Group("/api/auth")
Owner

Давай версионирование api сразу то есть базовый путь /api/v1/{handler/group}

Давай версионирование api сразу то есть базовый путь /api/v1/{handler/group}
cmd/main.go Outdated
@@ -28,0 +116,4 @@
orgs.DELETE("/:id", orgHandler.Delete)
}
srv := &http.Server{
Owner

В gin есть стандартный запуст через router.run() если не ошибаюсь посмотри в доке

В gin есть стандартный запуст через router.run() если не ошибаюсь посмотри в доке
@@ -0,0 +1,108 @@
# JWT Аутентификация — AegisGuard API
Owner

Аналогично вынести лучше в wiki или вообще такие подробности не обязательны в Readme

Аналогично вынести лучше в wiki или вообще такие подробности не обязательны в Readme
@@ -0,0 +82,4 @@
c.JSON(http.StatusOK, resp)
}
// @Summary Refresh epta token
Owner

Так как api внутренний можешь эти комменты писать на русском а так же лучше нормальное описание

Так как api внутренний можешь эти комменты писать на русском а так же лучше нормальное описание
@@ -0,0 +113,4 @@
c.JSON(http.StatusOK, resp)
}
// @Summary Logout epta
Owner

Нормальное описание надо для фронтов)

Нормальное описание надо для фронтов)
@@ -0,0 +143,4 @@
c.JSON(http.StatusOK, gin.H{"message": "logged out successfully"})
}
// @Summary Get epta current user
Owner

Аналогично

Аналогично
@@ -0,0 +179,4 @@
c.JSON(http.StatusOK, UserResponse{User: *user})
}
// @Summary Change epta password
Owner

-_-

-_-
@@ -0,0 +226,4 @@
c.JSON(http.StatusOK, gin.H{"message": "password changed successfully"})
}
// @Summary Update epta profile
Owner

-_-

-_-
@@ -0,0 +1,77 @@
package auth
Owner

Не уверен что это вообще нужно ratelimit пока что не важен и тем более есть nginx

Не уверен что это вообще нужно ratelimit пока что не важен и тем более есть nginx
@@ -0,0 +1,101 @@
package auth
Owner

Все sql репозитории вынести в отдельный пакет

Все sql репозитории вынести в отдельный пакет
@@ -0,0 +1,77 @@
package org
Owner

Аналогично с прошлым репо

Аналогично с прошлым репо
Mephi marked this conversation as resolved
@@ -0,0 +1,20 @@
CREATE TABLE IF NOT EXISTS users (
Owner

Нет goose up и тд миграции не сработают

Нет goose up и тд миграции не сработают
@@ -0,0 +1,11 @@
-- +goose Up
Owner

А тут уже есть)

А тут уже есть)
d3m0k1d closed this pull request 2026-06-13 19:08:53 +00:00
d3m0k1d reopened this pull request 2026-06-13 19:09:47 +00:00
d3m0k1d requested review from d3m0k1d 2026-06-13 19:09:56 +00:00
d3m0k1d requested changes 2026-06-13 19:10:33 +00:00
d3m0k1d left a comment
Owner

Все расписанно в прошлом ревью

Все расписанно в прошлом ревью
d3m0k1d requested review from shinyzero0 2026-06-13 19:21:05 +00:00
shinyzero0 added 8 commits 2026-06-14 00:12:52 +00:00
shinyzero0 force-pushed feature/jwt from e4c86fab5d to 9da532e9dc 2026-06-14 00:12:52 +00:00 Compare
Mephi added 1 commit 2026-06-14 16:41:50 +00:00
refactor: migrate from raw pgx to GORM, unify ErrNoRows, cleanup auth
ci / build (push) Successful in 2m58s
ci / build (pull_request) Successful in 2m47s
2da484d781
Mephi added 1 commit 2026-06-14 17:15:00 +00:00
feat: add API versioning , translate swagger, remove rate limiter
ci / build (push) Successful in 2m42s
ci / build (pull_request) Successful in 2m45s
ff355ad1d9
Some checks are pending
ci / build (push) Successful in 2m42s
ci / build (pull_request) Successful in 2m45s
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/jwt:feature/jwt
git checkout feature/jwt
Sign in to join this conversation.
No Reviewers
No Label
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: HellreigN/Control-plane#1