Remove fake email scope value

Author: motoki317

docs/v3-api.yaml

@@ -5289,12 +5289,6 @@ components:
           description: 更新日時
         traq:
           $ref: '#/components/schemas/OIDCTraqUserInfo'
-        email:
-          type: string
-          description: メールアドレス (フェイク値)
-        email_verified:
-          type: boolean
-          description: メールアドレスの確認が取れているか (フェイク値)
       required:
         - sub
     OIDCTraqUserInfo:

migration/current.go

@@ -45,7 +45,7 @@ func Migrations() []*gormigrate.Migration {
 		v32(), // ユーザーの表示名上限を32文字に
 		v33(), // 未読テーブルにチャンネルIDカラムを追加 / インデックス類の更新 / 不要なレコードの削除
 		v34(), // 未読テーブルのcreated_atカラムをメッセージテーブルを元に更新 / カラム名を変更
-		v35(), // OIDC実装のため、openid, profile, emailロール、get_oidc_userinfo権限を追加
+		v35(), // OIDC実装のため、openid, profileロール、get_oidc_userinfo権限を追加
 	}
 }
 

migration/v35.go

@@ -5,7 +5,7 @@ import (
 	"gorm.io/gorm"
 )
 
-// v35  OIDC実装のため、openid, profile, emailロール、get_oidc_userinfo権限を追加
+// v35  OIDC実装のため、openid, profileロール、get_oidc_userinfo権限を追加
 func v35() *gormigrate.Migration {
 	return &gormigrate.Migration{
 		ID: "35",
@@ -28,17 +28,6 @@ func v35() *gormigrate.Migration {
 						},
 					},
 				},
-				{
-					Name:        "email",
-					Oauth2Scope: true,
-					System:      true,
-					Permissions: []v35RolePermission{
-						{
-							Role:       "profile",
-							Permission: "get_oidc_userinfo",
-						},
-					},
-				},
 			}
 			for _, role := range roles {
 				err := db.Create(&role).Error

model/oauth2.go

@@ -31,7 +31,7 @@ type AccessScopes map[AccessScope]struct{}
 
 // SupportedAccessScopes 対応するスコープ一覧を返します
 func SupportedAccessScopes() []string {
-	return []string{"read", "write", "manage_bot", "openid", "profile", "email"}
+	return []string{"read", "write", "manage_bot", "openid", "profile"}
 }
 
 // Value database/sql/driver.Valuer 実装

router/oauth2/token_endpoint.go

@@ -59,7 +59,7 @@ func (h *Handler) issueIDToken(client *model.OAuth2Client, token *model.OAuth2To
 		"exp": token.Deadline().Unix(),
 		"iat": token.CreatedAt.Unix(),
 	}
-	// Extra claims according to scopes (profile, email)
+	// Extra claims according to scopes (profile)
 	userInfo, err := h.OIDC.GetUserInfo(userID, token.Scopes)
 	if err != nil {
 		return "", err

service/oidc/userinfo.go

@@ -71,10 +71,6 @@ func (s *Service) GetUserInfo(userID uuid.UUID, scopes ScopeChecker) (map[string
 			"home_channel": user.GetHomeChannel(),
 		}
 	}
-	if scopes.Contains("email") {
-		claims["email"] = user.GetName() + "@example.com"
-		claims["email_verified"] = false
-	}
 
 	return claims, nil
 }

service/rbac/role/email.go

@@ -48,11 +48,6 @@ func GetSystemRoles() Roles {
 			oauth2Scope: true,
 			permissions: permission.PermissionsFromArray(profilePerms),
 		},
-		Email: &systemRole{
-			name:        Email,
-			oauth2Scope: true,
-			permissions: permission.PermissionsFromArray(emailPerms),
-		},
 	}
 }