Skip to content

Commit 940703b

Browse files
authored
Merge pull request #121 from yyg-max/master
fix: Handle the ban of LinuxDo accounts
2 parents 25263f8 + 514848e commit 940703b

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

internal/apps/oauth/err.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
package oauth
2626

2727
const (
28-
UnAuthorized = "未登录"
29-
InvalidState = "非法登录请求"
30-
BannedUser = "用户已被封禁"
28+
UnAuthorized = "未登录"
29+
InvalidState = "非法登录请求"
30+
BannedAccount = "账号已被封禁"
3131
)

internal/apps/oauth/utils.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,11 @@ func doOAuth(ctx context.Context, code string) (*User, error) {
104104
span.SetStatus(codes.Error, err.Error())
105105
return nil, err
106106
}
107+
if !userInfo.Active {
108+
err = errors.New(BannedAccount)
109+
span.SetStatus(codes.Error, err.Error())
110+
return nil, err
111+
}
107112

108113
// save to db
109114
var user User
@@ -142,7 +147,7 @@ func doOAuth(ctx context.Context, code string) (*User, error) {
142147
}
143148
} else {
144149
if !user.IsActive {
145-
err = errors.New(BannedUser)
150+
err = errors.New(BannedAccount)
146151
span.SetStatus(codes.Error, err.Error())
147152
return nil, err
148153
}

0 commit comments

Comments
 (0)