⚠️ 应用类型约束:当前实现仅支持「钉钉登录-企业内部应用」(DingTalk 开放平台
internal_app 类型)。第三方个人应用、第三方企业应用类型暂不支持——OAuth 流程
相同但 corp 校验、跨企业行为不同。backend 通过 DingTalkAppKind 校验对非
internal_app 类型 fail-closed(硬约束)。
钉钉 OAuth 登录主链
- 4 步 OAuth 链:ExchangeCodeForUserToken / GetUnionIdByUserToken /
GetUserIdByUnionId / GetStaffInfoByUserId;app token 缓存
- pending session 机制持久化 OAuth 中间态;cookie-only token 持久化
- 三种分流:bind_login_required / email_completion / choose_account_action
- corp_restriction_policy 支持 none + internal_only;stale "whitelist" 在
加载层与写入层均静默 coerce 为 none + slog.Warn
- bypass_registration 开关:企业内部模式豁免全局 REGISTRATION_DISABLED
- isReservedEmail / signup_source / canUnbindProvider / OAuth pending flow
等横切点支持 dingtalk provider
- migration 136:4 表 CHECK 约束加入 'dingtalk' provider 值
internal_only 模式同步企业邮箱/姓名/部门到用户属性
- SyncCorpEmail / SyncDisplayName / SyncDept 三个独立开关 + 对应
SyncXxxAttrKey 目标属性 key(默认 dingtalk_email / dingtalk_name /
dingtalk_department);非 internal_only policy 在写入层与加载层均
coerce 为 false,admin handler 与 setting_service 双层兜底
- 同步语义:首次注册写 users.username(昵称优先 → 企业姓名 fallback),
之后每次登录刷新 3 个属性;空值也写入以覆盖旧值
- 邮箱三级 fallback:org_email > email > extension["企业邮箱"]
(钉钉自定义字段 JSON)
- 部门路径递归向上拼接,跳过 dept_id=1 选首个真实子部门,剥离根组织名
- GetUnionIdByUserToken 同时返回 OIDC /contact/users/me 的 nick 字段;
新增 GetDeptInfo 调用 OAPI /topapi/v2/department/get
- AuthHandler 注入 UserAttributeService;OAuth pending flow 在
createPendingOAuthAccount / bindPendingOAuthLogin 分别派发到
AfterRegistration(syncUsername=true)/ AfterLogin
- migration 137 seed dingtalk_email/name/department 三个用户属性定义
附带修复(同集成路径暴露的两个 OAuth 注册回归)
- LoginOrRegisterOAuthWithTokenPair 新建用户分支用 inferLegacySignupSource
覆写 caller 显式传入的 signupSource,导致 dingtalk/linuxdo/oidc/wechat
渠道授权按 email 渠道读取;改为只在 caller 未显式传入时回退邮箱推断
- mergeProviderDefaultGrantSettings 把 parse fallback 默认值
(Concurrency=5 / Balance=0) 当作"未配置"哨兵,admin 显式设 5 时被误判
退回全局默认(复现:全局默认 1 + 渠道默认并发 5 + grant_on_signup → 新
用户实际 concurrency=1);去掉哨兵,admin 任何 >=0 值都覆盖 globalDefaults
前端
- DingTalk Login / Callback / EmailCompletion / ChoiceAccount / Error
视图;router + auth API client
- admin SettingsView:corp policy radio(none / internal_only)+ bypass
注册开关 + i18n;internal_only 下展示三同步开关 + 目标 attr key 下拉
(拉取 user attribute definitions),展示 fieldEmail /
qyapi_get_department_list 钉钉权限申请提示
- Profile:S1 主动绑定 / S5 解绑钉钉按钮 + 合成邮箱防自锁
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
384 lines
9.3 KiB
Go
384 lines
9.3 KiB
Go
//go:build unit
|
|
|
|
package middleware
|
|
|
|
import (
|
|
"context"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"testing"
|
|
|
|
"github.com/Wei-Shaw/sub2api/internal/config"
|
|
"github.com/Wei-Shaw/sub2api/internal/service"
|
|
"github.com/gin-gonic/gin"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
type bmSettingRepo struct {
|
|
values map[string]string
|
|
}
|
|
|
|
func (r *bmSettingRepo) Get(_ context.Context, _ string) (*service.Setting, error) {
|
|
panic("unexpected Get call")
|
|
}
|
|
|
|
func (r *bmSettingRepo) GetValue(_ context.Context, key string) (string, error) {
|
|
v, ok := r.values[key]
|
|
if !ok {
|
|
return "", service.ErrSettingNotFound
|
|
}
|
|
return v, nil
|
|
}
|
|
|
|
func (r *bmSettingRepo) Set(_ context.Context, _, _ string) error {
|
|
panic("unexpected Set call")
|
|
}
|
|
|
|
func (r *bmSettingRepo) GetMultiple(_ context.Context, _ []string) (map[string]string, error) {
|
|
panic("unexpected GetMultiple call")
|
|
}
|
|
|
|
func (r *bmSettingRepo) SetMultiple(_ context.Context, settings map[string]string) error {
|
|
if r.values == nil {
|
|
r.values = make(map[string]string, len(settings))
|
|
}
|
|
for key, value := range settings {
|
|
r.values[key] = value
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func (r *bmSettingRepo) GetAll(_ context.Context) (map[string]string, error) {
|
|
panic("unexpected GetAll call")
|
|
}
|
|
|
|
func (r *bmSettingRepo) Delete(_ context.Context, _ string) error {
|
|
panic("unexpected Delete call")
|
|
}
|
|
|
|
func newBackendModeSettingService(t *testing.T, enabled string) *service.SettingService {
|
|
t.Helper()
|
|
|
|
repo := &bmSettingRepo{
|
|
values: map[string]string{
|
|
service.SettingKeyBackendModeEnabled: enabled,
|
|
},
|
|
}
|
|
svc := service.NewSettingService(repo, &config.Config{})
|
|
require.NoError(t, svc.UpdateSettings(context.Background(), &service.SystemSettings{
|
|
BackendModeEnabled: enabled == "true",
|
|
}))
|
|
|
|
return svc
|
|
}
|
|
|
|
func stringPtr(v string) *string {
|
|
return &v
|
|
}
|
|
|
|
func TestBackendModeUserGuard(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
nilService bool
|
|
enabled string
|
|
role *string
|
|
wantStatus int
|
|
}{
|
|
{
|
|
name: "disabled_allows_all",
|
|
enabled: "false",
|
|
role: stringPtr("user"),
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "nil_service_allows_all",
|
|
nilService: true,
|
|
role: stringPtr("user"),
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_admin_allowed",
|
|
enabled: "true",
|
|
role: stringPtr("admin"),
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_user_blocked",
|
|
enabled: "true",
|
|
role: stringPtr("user"),
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_no_role_blocked",
|
|
enabled: "true",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_empty_role_blocked",
|
|
enabled: "true",
|
|
role: stringPtr(""),
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
}
|
|
|
|
for _, tc := range tests {
|
|
tc := tc
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
gin.SetMode(gin.TestMode)
|
|
|
|
r := gin.New()
|
|
if tc.role != nil {
|
|
role := *tc.role
|
|
r.Use(func(c *gin.Context) {
|
|
c.Set(string(ContextKeyUserRole), role)
|
|
c.Next()
|
|
})
|
|
}
|
|
|
|
var svc *service.SettingService
|
|
if !tc.nilService {
|
|
svc = newBackendModeSettingService(t, tc.enabled)
|
|
}
|
|
|
|
r.Use(BackendModeUserGuard(svc))
|
|
r.GET("/test", func(c *gin.Context) {
|
|
c.JSON(http.StatusOK, gin.H{"ok": true})
|
|
})
|
|
|
|
w := httptest.NewRecorder()
|
|
req := httptest.NewRequest(http.MethodGet, "/test", nil)
|
|
r.ServeHTTP(w, req)
|
|
|
|
require.Equal(t, tc.wantStatus, w.Code)
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBackendModeAuthGuard(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
nilService bool
|
|
enabled string
|
|
path string
|
|
wantStatus int
|
|
}{
|
|
{
|
|
name: "disabled_allows_all",
|
|
enabled: "false",
|
|
path: "/api/v1/auth/register",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "nil_service_allows_all",
|
|
nilService: true,
|
|
path: "/api/v1/auth/register",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_login",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/login",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_login_2fa",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/login/2fa",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_logout",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/logout",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_refresh",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/refresh",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_linuxdo_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/linuxdo/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_linuxdo_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/linuxdo/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_wechat_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/wechat/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_wechat_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/wechat/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_wechat_payment_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/wechat/payment/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_wechat_payment_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/wechat/payment/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_oidc_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/oidc/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_oidc_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/oidc/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_github_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/github/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_github_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/github/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_google_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/google/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_google_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/google/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_dingtalk_oauth_start",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/dingtalk/start",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_allows_dingtalk_oauth_callback",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/dingtalk/callback",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_dingtalk_complete_registration",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/dingtalk/complete-registration",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_dingtalk_create_account",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/dingtalk/create-account",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_dingtalk_bind_login",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/dingtalk/bind-login",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_oauth_pending_exchange",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/pending/exchange",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_oauth_pending_send_verify_code",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/pending/send-verify-code",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_oauth_pending_create_account",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/pending/create-account",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_oauth_pending_bind_login",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/pending/bind-login",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_provider_bind_login",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/oidc/bind-login",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_provider_create_account",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/wechat/create-account",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_allows_legacy_complete_registration",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/oauth/linuxdo/complete-registration",
|
|
wantStatus: http.StatusOK,
|
|
},
|
|
{
|
|
name: "enabled_blocks_register",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/register",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
{
|
|
name: "enabled_blocks_forgot_password",
|
|
enabled: "true",
|
|
path: "/api/v1/auth/forgot-password",
|
|
wantStatus: http.StatusForbidden,
|
|
},
|
|
}
|
|
|
|
for _, tc := range tests {
|
|
tc := tc
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
gin.SetMode(gin.TestMode)
|
|
|
|
r := gin.New()
|
|
|
|
var svc *service.SettingService
|
|
if !tc.nilService {
|
|
svc = newBackendModeSettingService(t, tc.enabled)
|
|
}
|
|
|
|
r.Use(BackendModeAuthGuard(svc))
|
|
r.Any("/*path", func(c *gin.Context) {
|
|
c.JSON(http.StatusOK, gin.H{"ok": true})
|
|
})
|
|
|
|
w := httptest.NewRecorder()
|
|
req := httptest.NewRequest(http.MethodGet, tc.path, nil)
|
|
r.ServeHTTP(w, req)
|
|
|
|
require.Equal(t, tc.wantStatus, w.Code)
|
|
})
|
|
}
|
|
}
|