触发器触发后Audit_UserLogOn表插入重复登录记录的问题求助

问题分析 #

你的触发器在用户登录时会插入两条重复的审计记录，核心原因大概率是登录操作触发了两次对用户表的LastActivityDate字段更新，或者原触发器逻辑没有正确处理inserted/deleted表的关联与去重。

解决方案 #

方案1：通过关联与去重保证单条插入

修改触发器的查询逻辑，明确关联inserted和deleted表的UserId，同时用DISTINCT去重，避免重复记录：

INSERT Audit_UserLogOn
    (UserId, LogOnDateTime)
SELECT DISTINCT i.UserId, i.LastActivityDate
FROM inserted i
JOIN deleted d ON i.UserId = d.UserId
WHERE i.LastActivityDate <> d.LastActivityDate

• 用JOIN替代子查询，确保inserted和deleted的记录是同一用户的前后数据
• DISTINCT会自动过滤inserted表中重复的用户登录记录

方案2：通过时间窗口过滤短时间内的重复插入

如果是同一登录操作在极短时间内触发了两次更新，可以通过判断审计表中是否已有最近的相同记录来避免重复：

INSERT Audit_UserLogOn
    (UserId, LogOnDateTime)
SELECT i.UserId, i.LastActivityDate
FROM inserted i
JOIN deleted d ON i.UserId = d.UserId
WHERE i.LastActivityDate <> d.LastActivityDate
AND NOT EXISTS (
    SELECT 1 FROM Audit_UserLogOn 
    WHERE UserId = i.UserId 
    AND LogOnDateTime >= DATEADD(SECOND, -1, i.LastActivityDate)
)

• 这里设置了1秒的时间窗口，可根据实际情况调整（比如改成3秒）
• 确保同一用户同一登录动作只生成一条审计记录

方案3：限制触发器的触发事件类型

如果你的触发器同时绑定了INSERT和UPDATE事件，而登录操作实际是先删后插的逻辑，会导致触发器触发两次。可以只处理UPDATE事件（因为登录是更新LastActivityDate）：

CREATE TRIGGER [YourTriggerName]
ON [YourUserTable]
AFTER UPDATE -- 只绑定UPDATE事件
AS
BEGIN
    SET NOCOUNT ON;

    INSERT Audit_UserLogOn
        (UserId, LogOnDateTime)
    SELECT DISTINCT i.UserId, i.LastActivityDate
    FROM inserted i
    JOIN deleted d ON i.UserId = d.UserId
    WHERE i.LastActivityDate <> d.LastActivityDate
END

排查建议 #

如果不确定具体原因，可以在触发器中临时添加查询语句，查看每次触发时inserted和deleted表的记录：

CREATE TRIGGER [YourTriggerName]
ON [YourUserTable]
AFTER UPDATE
AS
BEGIN
    SET NOCOUNT ON;

    -- 临时查看触发时的数据，排查重复原因
    SELECT 'Inserted' AS Source, * FROM inserted;
    SELECT 'Deleted' AS Source, * FROM deleted;

    INSERT Audit_UserLogOn
        (UserId, LogOnDateTime)
    SELECT DISTINCT i.UserId, i.LastActivityDate
    FROM inserted i
    JOIN deleted d ON i.UserId = d.UserId
    WHERE i.LastActivityDate <> d.LastActivityDate
END

执行登录操作后，查看这些查询的结果，就能明确是inserted表有多条记录，还是触发了多次触发器。

内容的提问来源于stack exchange，提问作者NoHope4Me