From 9fff9fa9c1fa7fb29be576a5d95d8d3e87878772 Mon Sep 17 00:00:00 2001 From: amee Date: Fri, 3 Apr 2026 15:38:00 +0530 Subject: [PATCH] fix: _get_client_ip --- app/routes/admin/modules.py | 6 +++--- app/routes/admin/tenant_modules.py | 6 +++--- app/routes/auth/role.py | 6 +++--- app/routes/auth/tenant.py | 6 +++--- app/routes/auth/user.py | 6 +++--- 5 files changed, 15 insertions(+), 15 deletions(-) diff --git a/app/routes/admin/modules.py b/app/routes/admin/modules.py index 2218e5c..1f02196 100644 --- a/app/routes/admin/modules.py +++ b/app/routes/admin/modules.py @@ -43,7 +43,7 @@ def create_module( description=f"Module '{result.module_name}' created", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), new_values=module_data.model_dump() ) return result @@ -81,7 +81,7 @@ def update_module( description=f"Module '{result.module_name}' updated", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values=old_values, new_values=module_data.model_dump(exclude_unset=True) ) @@ -110,7 +110,7 @@ def delete_module( description=f"Module '{module_name}' deleted", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request) + ip_address=get_client_ip(request) ) return result diff --git a/app/routes/admin/tenant_modules.py b/app/routes/admin/tenant_modules.py index e178fed..c4a18f9 100644 --- a/app/routes/admin/tenant_modules.py +++ b/app/routes/admin/tenant_modules.py @@ -40,7 +40,7 @@ def assign_module_to_tenant( description=f"Module assigned to tenant {tenant_id}", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), new_values=assignment_data.model_dump(mode='json') ) return result @@ -66,7 +66,7 @@ def update_tenant_module( description=f"Tenant module assignment updated", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), new_values=update_data.model_dump(exclude_unset=True) ) return result @@ -91,6 +91,6 @@ def remove_module_from_tenant( description=f"Tenant module assignment deleted", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), ) return result \ No newline at end of file diff --git a/app/routes/auth/role.py b/app/routes/auth/role.py index f39b04d..2f1e176 100644 --- a/app/routes/auth/role.py +++ b/app/routes/auth/role.py @@ -39,7 +39,7 @@ def create_role( description=f"Role '{result.role_name}' created", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), new_values={ "role_name": result.role_name, "tenant_id": str(result.tenant_id) if result.tenant_id else None, @@ -95,7 +95,7 @@ def update_role( description=f"Role '{result.role_name}' updated", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values=old_values, new_values=new_values ) @@ -125,7 +125,7 @@ def delete_role( description=f"Role '{result.role_name}' deleted", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values={"role_name": role_name} ) return result diff --git a/app/routes/auth/tenant.py b/app/routes/auth/tenant.py index a00f7d3..97c2b41 100644 --- a/app/routes/auth/tenant.py +++ b/app/routes/auth/tenant.py @@ -32,7 +32,7 @@ def create_tenant( description=f"Tenant '{result.tenant_name}' onboarded", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), new_values=tenant_data.model_dump(mode='json') ) return result @@ -92,7 +92,7 @@ def update_tenant( description=f"Tenant '{result.tenant_name}' updated", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values=old_values, new_values=new_values ) @@ -122,7 +122,7 @@ def delete_tenant( description=f"Tenant '{existing.tenant_name}' removed", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values={"tenant_name": existing.tenant_name, "domain": existing.tenant_domain} ) return result diff --git a/app/routes/auth/user.py b/app/routes/auth/user.py index 3b32751..e2612f7 100644 --- a/app/routes/auth/user.py +++ b/app/routes/auth/user.py @@ -32,7 +32,7 @@ def create_user( description=f"User '{result.email}' created", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), new_values={ "email": result.email, "tenant_id": str(result.tenant_id) if result.tenant_id else None, @@ -82,7 +82,7 @@ def update_user( description=f"User '{result.email}' updated", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values=old_values, new_values=user_data.model_dump(mode='json', exclude_unset=True) ) @@ -111,7 +111,7 @@ def delete_user( description=f"User '{user_email}' deleted", performed_by_id=str(current_user.id), performed_by_email=current_user.email, - ip_address=_get_client_ip(request), + ip_address=get_client_ip(request), old_values={"email": user_email} ) return result