Skip to content

Commit f614692

Browse files
committed
refactor(auth): Remove user allowlist filtering
- Delete `isAllowedUser` function for user authorization - Remove environment variable `OAUTH_ALLOWED_USERS` from docker-compose - Eliminate user whitelist check in OAuth callback handler
1 parent 929d131 commit f614692

File tree

2 files changed

+0
-18
lines changed

2 files changed

+0
-18
lines changed

docker-compose.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,4 @@ services:
99
environment:
1010
- TZ=Asia/Shanghai
1111
- OAUTH_CLIENT_ID=your_client_id
12-
- OAUTH_ALLOWED_USERS=user1,user2,user3
1312
restart: always

internal/handler/auth.go

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -166,17 +166,6 @@ func (h *ProxyHandler) LoginHandler(w http.ResponseWriter, r *http.Request) {
166166
http.Redirect(w, r, authURL, http.StatusTemporaryRedirect)
167167
}
168168

169-
// isAllowedUser 检查用户是否在允许列表中
170-
func isAllowedUser(username string) bool {
171-
allowedUsers := strings.Split(os.Getenv("OAUTH_ALLOWED_USERS"), ",")
172-
for _, allowed := range allowedUsers {
173-
if strings.TrimSpace(allowed) == username {
174-
return true
175-
}
176-
}
177-
return false
178-
}
179-
180169
// OAuthCallbackHandler 处理 OAuth 回调
181170
func (h *ProxyHandler) OAuthCallbackHandler(w http.ResponseWriter, r *http.Request) {
182171
code := r.URL.Query().Get("code")
@@ -225,12 +214,6 @@ func (h *ProxyHandler) OAuthCallbackHandler(w http.ResponseWriter, r *http.Reque
225214
return
226215
}
227216

228-
// 检查用户是否在允许列表中
229-
if !isAllowedUser(userInfo.Username) {
230-
http.Error(w, "Unauthorized user", http.StatusUnauthorized)
231-
return
232-
}
233-
234217
// 生成内部访问令牌
235218
internalToken := h.auth.generateToken()
236219
h.auth.addToken(internalToken, userInfo.Username, tokenExpiry)

0 commit comments

Comments
 (0)