AbpAuditLogs - insert Ipv4 instead of Ipv6 into CleintAddress and Normalize table field TenantName as we are already gettinf tenantId from tenant table? #2983

5 Answer(s)

• 

  0

  maliming created 4 years ago

  Support Team Fullstack Developer

  hi

  You can override the GetClientIpAddress method to use IPV6 address.

  https://github.com/abpframework/abp/blob/dev/framework/src/Volo.Abp.AspNetCore/Volo/Abp/AspNetCore/WebClientInfo/HttpContextWebClientInfoProvider.cs#L30

  Normalize Auditlogs table as it has both TenantId and TenantName fields?

  What do you mean? Can you explain?

  Markdown supported.

  Copy, paste, or drag & drop images and files (max 100 MB per file, 100 MB total per post)

  Save Cancel
• 

  0

  ronaksbhavsar created 4 years ago

  hi

  You can override the GetClientIpAddress method to use IPV6 address.

  https://github.com/abpframework/abp/blob/dev/framework/src/Volo.Abp.AspNetCore/Volo/Abp/AspNetCore/WebClientInfo/HttpContextWebClientInfoProvider.cs#L30

  Normalize Auditlogs table as it has both TenantId and TenantName fields?

  What do you mean? Can you explain?

  Can't we get tenantName from tenantid as it has foreign key relation, Do we required to add tenantname?

  Markdown supported.

  Copy, paste, or drag & drop images and files (max 100 MB per file, 100 MB total per post)

  Save Cancel
• 

  0

  maliming created 4 years ago

  Support Team Fullstack Developer

  hi

  The** tenant id** may be changed by code, so the caller is responsible for passing the tenant name.

  using(currentTenant.Change(tenantId))
  {
  
  }
  
  

  Markdown supported.

  Copy, paste, or drag & drop images and files (max 100 MB per file, 100 MB total per post)

  Save Cancel
• 

  0

  ronaksbhavsar created 4 years ago

  hi

  The** tenant id** may be changed by code, so the caller is responsible for passing the tenant name.

  using(currentTenant.Change(tenantId)) 
  { 
   
  } 
   
  

  Okay Thanks

  Markdown supported.

  Copy, paste, or drag & drop images and files (max 100 MB per file, 100 MB total per post)

  Save Cancel
• 

  0

  maliming created 4 years ago

  Support Team Fullstack Developer

  You're welcome. : )

  Markdown supported.

  Copy, paste, or drag & drop images and files (max 100 MB per file, 100 MB total per post)

  Save Cancel