Merge pull request #3181 from Anai-Guo/fix-audit-log-missing-action

fix(server): five audit_log calls omit the required action argument

Author: bghira

simpletuner/simpletuner_sdk/server/routes/approvals.py

@@ -419,7 +419,8 @@ async def approve_request(
     # Audit log
     if request:
         await audit_log(
-            event_type=AuditEventType.JOB_APPROVED,
+            AuditEventType.JOB_APPROVED,
+            f"Job {request.job_id} approved by '{user.username}'",
             actor_id=user.id,
             target_type="approval_request",
             target_id=str(request_id),
@@ -484,7 +485,8 @@ async def reject_request(
     # Audit log
     if request:
         await audit_log(
-            event_type=AuditEventType.JOB_REJECTED,
+            AuditEventType.JOB_REJECTED,
+            f"Job {request.job_id} rejected by '{user.username}'",
             actor_id=user.id,
             target_type="approval_request",
             target_id=str(request_id),
@@ -570,7 +572,8 @@ async def bulk_approve_requests(
 
                     # Audit log
                     await audit_log(
-                        event_type=AuditEventType.JOB_APPROVED,
+                        AuditEventType.JOB_APPROVED,
+                        f"Job {request.job_id} approved by '{user.username}' (bulk)",
                         actor_id=user.id,
                         target_type="approval_request",
                         target_id=str(request_id),
@@ -634,7 +637,8 @@ async def bulk_reject_requests(
 
                     # Audit log
                     await audit_log(
-                        event_type=AuditEventType.JOB_REJECTED,
+                        AuditEventType.JOB_REJECTED,
+                        f"Job {request.job_id} rejected by '{user.username}' (bulk)",
                         actor_id=user.id,
                         target_type="approval_request",
                         target_id=str(request_id),

simpletuner/simpletuner_sdk/server/services/cloud/credential_resolver.py

@@ -124,7 +124,8 @@ async def get_credential(
                         from .audit import AuditEventType, audit_log
 
                         await audit_log(
-                            event_type=AuditEventType.CREDENTIAL_USED,
+                            AuditEventType.CREDENTIAL_USED,
+                            f"Credential '{provider}/{credential_name}' used",
                             actor_id=user_id,
                             target_type="credential",
                             target_id=f"{provider}/{credential_name}",