Add files via upload

auth.js

@@ -104,7 +104,7 @@ class AuthService {
         });
     }
 
-    async authenticateLDAP(username, password) {
+async authenticateLDAP(username, password) {
     try {
         const response = await fetch(process.env.LDAP_AUTH_URL, {
             method: 'POST',
@@ -129,15 +129,16 @@ class AuthService {
         console.error('LDAP authentication error:', error);
         return null;
     }
-    }
+}
 
-    async processLDAPUser(ldapData) {
+async processLDAPUser(ldapData) {
     const { username, full_name, groups, description } = ldapData;
     
     // Определяем роль пользователя на основе групп
     const allowedGroups = process.env.ALLOWED_GROUPS ? 
         process.env.ALLOWED_GROUPS.split(',').map(g => g.trim()) : [];
     
+    // ВАЖНО: Проверяем актуальные группы при каждом входе
     const isAdmin = groups && groups.some(group => 
         allowedGroups.includes(group)
     );
@@ -156,7 +157,7 @@ class AuthService {
                 login: username,
                 name: full_name || username,
                 email: `${username}@school25.ru`,
-                    role: role,
+                role: role, // Всегда обновляем роль из актуальных групп
                 auth_type: 'ldap',
                 groups: groups ? JSON.stringify(groups) : '[]',
                 description: description || '',
@@ -164,7 +165,7 @@ class AuthService {
             };
 
             if (existingUser) {
-                    // Обновляем существующего пользователя
+                // Всегда обновляем роль, даже если пользователь уже существует
                 db.run(
                     `UPDATE users SET 
                      name = ?, email = ?, role = ?, groups = ?, description = ?, last_login = datetime('now'),
@@ -175,7 +176,7 @@ class AuthService {
                         if (err) {
                             reject(err);
                         } else {
-                                // Возвращаем полные данные пользователя
+                            console.log(`Обновлены данные LDAP пользователя ${username}. Роль: ${userData.role}, Группы: ${groups}`);
                             resolve({ 
                                 id: existingUser.id,
                                 login: userData.login,
@@ -201,6 +202,7 @@ class AuthService {
                         if (err) {
                             reject(err);
                         } else {
+                            console.log(`Создан новый LDAP пользователь ${username}. Роль: ${userData.role}, Группы: ${groups}`);
                             resolve({
                                 id: this.lastID,
                                 login: userData.login,
@@ -218,7 +220,7 @@ class AuthService {
             }
         });
     });
-    }
+}
 
     async authenticate(login, password) {
         // Сначала пробуем локальную авторизацию

server.js

@@ -367,35 +367,52 @@ app.post('/api/logout', (req, res) => {
     });
 });
 
+// В server.js обновите маршрут /api/user
 app.get('/api/user', (req, res) => {
     if (req.session.user) {
-        // Обновляем данные пользователя из базы на случай изменений
+        // Для LDAP пользователей всегда проверяем актуальную роль
-        authService.getUserById(req.session.user.id)
+        if (req.session.user.auth_type === 'ldap') {
-            .then(user => {
+            db.get("SELECT groups FROM users WHERE id = ?", [req.session.user.id], (err, user) => {
-                if (user) {
+                if (err || !user) {
-                    const updatedUser = {
-                        id: user.id,
-                        login: user.login,
-                        name: user.name,
-                        email: user.email,
-                        role: user.role,
-                        auth_type: user.auth_type,
-                        groups: user.groups ? (typeof user.groups === 'string' ? JSON.parse(user.groups) : user.groups) : []
-                    };
-                    
-                    // Обновляем сессию
-                    req.session.user = updatedUser;
-                    res.json({ user: updatedUser });
-                } else {
-                    // Пользователь не найден в базе - разлогиниваем
                     req.session.destroy();
-                    res.status(401).json({ error: 'Пользователь не найден' });
+                    return res.status(401).json({ error: 'Пользователь не найден' });
                 }
-            })
+                
-            .catch(err => {
+                // Парсим группы
-                console.error('Ошибка получения пользователя:', err);
+                let groups = [];
-                res.status(500).json({ error: 'Ошибка сервера' });
+                try {
+                    groups = JSON.parse(user.groups || '[]');
+                } catch (e) {
+                    groups = [];
+                }
+                
+                // Проверяем группы
+                const allowedGroups = process.env.ALLOWED_GROUPS ? 
+                    process.env.ALLOWED_GROUPS.split(',').map(g => g.trim()) : [];
+                
+                const isAdmin = groups.some(group => allowedGroups.includes(group));
+                const actualRole = isAdmin ? 'admin' : 'teacher';
+                
+                // Обновляем роль если изменилась
+                if (req.session.user.role !== actualRole) {
+                    console.log(`Обновлена роль пользователя ${req.session.user.login} с ${req.session.user.role} на ${actualRole}`);
+                    
+                    // Обновляем в базе
+                    db.run(
+                        "UPDATE users SET role = ?, updated_at = datetime('now') WHERE id = ?",
+                        [actualRole, req.session.user.id]
+                    );
+                    
+                    // Обновляем в сессии
+                    req.session.user.role = actualRole;
+                }
+                
+                res.json({ user: req.session.user });
             });
+        } else {
+            // Для локальных пользователей просто возвращаем данные
+            res.json({ user: req.session.user });
+        }
     } else {
         res.status(401).json({ error: 'Не аутентифицирован' });
     }