Merge pull request 'Fix url on gitea and test ver for auth' (#8) from develop into master
Reviewed-on: #8
This commit was merged in pull request #8.
This commit is contained in:
Binary file not shown.
@@ -4,6 +4,7 @@ import (
|
|||||||
"encoding/json"
|
"encoding/json"
|
||||||
"os"
|
"os"
|
||||||
|
|
||||||
|
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/auth"
|
||||||
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/logger"
|
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/logger"
|
||||||
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/repositories"
|
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/repositories"
|
||||||
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/storage"
|
"gitea.d3m0k1d.ru/d3m0k1d/d3m0k1d.ru/backend/internal/storage"
|
||||||
@@ -66,6 +67,7 @@ func (h *AuthHandlers) LoginGithub(c *gin.Context) {
|
|||||||
// @Failure 500 {object} map[string]string "Exchange failed"
|
// @Failure 500 {object} map[string]string "Exchange failed"
|
||||||
// @Router /callback/github [get]
|
// @Router /callback/github [get]
|
||||||
func (h *AuthHandlers) CallbackGithub(c *gin.Context) {
|
func (h *AuthHandlers) CallbackGithub(c *gin.Context) {
|
||||||
|
var id int
|
||||||
h.logger.Info("CallbackGithub called")
|
h.logger.Info("CallbackGithub called")
|
||||||
|
|
||||||
code := c.Query("code")
|
code := c.Query("code")
|
||||||
@@ -83,6 +85,7 @@ func (h *AuthHandlers) CallbackGithub(c *gin.Context) {
|
|||||||
c.JSON(500, gin.H{"error": "exchange failed", "details": err.Error()})
|
c.JSON(500, gin.H{"error": "exchange failed", "details": err.Error()})
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
client := h.config.Client(c.Request.Context(), token)
|
client := h.config.Client(c.Request.Context(), token)
|
||||||
resp, err := client.Get("https://api.github.com/user")
|
resp, err := client.Get("https://api.github.com/user")
|
||||||
if err != nil {
|
if err != nil {
|
||||||
@@ -90,6 +93,7 @@ func (h *AuthHandlers) CallbackGithub(c *gin.Context) {
|
|||||||
c.JSON(500, gin.H{"error": "get request failed to github", "details": err.Error()})
|
c.JSON(500, gin.H{"error": "get request failed to github", "details": err.Error()})
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
var ghUser storage.UserReg
|
var ghUser storage.UserReg
|
||||||
err = json.NewDecoder(resp.Body).Decode(&ghUser)
|
err = json.NewDecoder(resp.Body).Decode(&ghUser)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
@@ -97,23 +101,41 @@ func (h *AuthHandlers) CallbackGithub(c *gin.Context) {
|
|||||||
c.JSON(500, gin.H{"error": "decode failed", "details": err.Error()})
|
c.JSON(500, gin.H{"error": "decode failed", "details": err.Error()})
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
isreg, err := h.repo.IsRegistered(c.Request.Context(), ghUser.GithubID)
|
isreg, err := h.repo.IsRegistered(c.Request.Context(), ghUser.GithubID)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
h.logger.Error("Database check failed: " + err.Error())
|
||||||
c.JSON(500, gin.H{"error": "database error", "details": err.Error()})
|
c.JSON(500, gin.H{"error": "database error", "details": err.Error()})
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if isreg {
|
|
||||||
c.JSON(200, gin.H{"user": ghUser})
|
|
||||||
|
|
||||||
reg := h.repo.Register(c.Request.Context(), ghUser)
|
if !isreg {
|
||||||
if reg != nil {
|
h.logger.Info("New user, registering: " + ghUser.GithubLogin)
|
||||||
c.JSON(500, gin.H{"error": "database error", "details": reg.Error()})
|
id, err = h.repo.Register(c.Request.Context(), ghUser)
|
||||||
err := h.repo.Register(c.Request.Context(), ghUser)
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
c.JSON(500, gin.H{"error": "database error", "details": err.Error()})
|
h.logger.Error("Registration failed: " + err.Error())
|
||||||
h.logger.Error("Database eer in gh callback handler")
|
c.JSON(500, gin.H{"error": "registration failed", "details": err.Error()})
|
||||||
|
return
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
user := storage.User{
|
||||||
|
ID: id,
|
||||||
|
GithubID: ghUser.GithubID,
|
||||||
|
GithubLogin: ghUser.GithubLogin,
|
||||||
|
Email: ghUser.Email,
|
||||||
|
AvatarURL: ghUser.AvatarURL,
|
||||||
|
}
|
||||||
|
jwtToken, err := auth.GenerateJWT(user)
|
||||||
|
if err != nil {
|
||||||
|
h.logger.Error("JWT generation failed: " + err.Error())
|
||||||
|
c.JSON(500, gin.H{"error": "token generation failed", "details": err.Error()})
|
||||||
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
h.logger.Info("Authentication successful for user: " + ghUser.GithubLogin)
|
||||||
|
|
||||||
|
c.JSON(200, gin.H{
|
||||||
|
"token": jwtToken,
|
||||||
|
"user": ghUser,
|
||||||
|
})
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -20,16 +20,24 @@ func NewAuthRepository(db *sql.DB) AuthRepository {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func (a *authRepository) Register(ctx context.Context, user storage.UserReg) error {
|
func (a *authRepository) Register(ctx context.Context, user storage.UserReg) (int, error) {
|
||||||
|
var id int
|
||||||
_, err := a.db.Exec(
|
_, err := a.db.Exec(
|
||||||
"INSERT INTO users(email, github_id, github_login, avatar_url) VALUES(?, ?, ?, ?)",
|
"INSERT INTO users(email, github_id, github_login, avatar_url) VALUES(?, ?, ?, ?)",
|
||||||
)
|
)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
a.logger.Error("error scan: " + err.Error())
|
a.logger.Error("error request: " + err.Error())
|
||||||
return err
|
return 0, err
|
||||||
}
|
}
|
||||||
|
row := a.db.QueryRow("SELECT id FROM users WHERE github_id = ?", user.GithubID)
|
||||||
|
err = row.Scan(&id)
|
||||||
|
if err != nil {
|
||||||
|
a.logger.Error("error scan: " + err.Error())
|
||||||
|
return 0, err
|
||||||
|
}
|
||||||
|
|
||||||
a.logger.Info("User registered:", "email", user.Email)
|
a.logger.Info("User registered:", "email", user.Email)
|
||||||
return nil
|
return id, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
func (a *authRepository) IsRegistered(ctx context.Context, github_id int) (bool, error) {
|
func (a *authRepository) IsRegistered(ctx context.Context, github_id int) (bool, error) {
|
||||||
|
|||||||
@@ -15,6 +15,6 @@ type PostRepository interface {
|
|||||||
}
|
}
|
||||||
|
|
||||||
type AuthRepository interface {
|
type AuthRepository interface {
|
||||||
Register(ctx context.Context, user storage.UserReg) error
|
Register(ctx context.Context, user storage.UserReg) (int, error)
|
||||||
IsRegistered(ctx context.Context, github_id int) (bool, error)
|
IsRegistered(ctx context.Context, github_id int) (bool, error)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -12,7 +12,7 @@ export default function About() {
|
|||||||
💻 Github
|
💻 Github
|
||||||
</a>
|
</a>
|
||||||
<a
|
<a
|
||||||
href="https://git.d3m0k1d.ru"
|
href="https://gitea.d3m0k1d.ru"
|
||||||
target="_blank"
|
target="_blank"
|
||||||
rel="noopener noreferrer"
|
rel="noopener noreferrer"
|
||||||
className="px-3 py-1 border border-base-content/20 rounded hover:border-[hsl(270,73%,63%)] hover:text-[hsl(270,73%,63%)] transition-all font-mono"
|
className="px-3 py-1 border border-base-content/20 rounded hover:border-[hsl(270,73%,63%)] hover:text-[hsl(270,73%,63%)] transition-all font-mono"
|
||||||
|
|||||||
Reference in New Issue
Block a user