Audit info parameters - WebAPI

Hi, I have custom WebAPI on the remote server. On every request there is a warning in the log file.

WARN|Abp.Auditing.WebAuditInfoProvider|System.Net.Sockets.SocketException (0x80004005): No such host is known at System.Net.Dns.GetAddrInfo(String name) at System.Net.Dns.InternalGetHostByName(String hostName, Boolean includeIPv6) at System.Net.Dns.GetHostAddresses(String hostNameOrAddress) at Abp.Auditing.WebAuditInfoProvider.GetClientIpAddress(HttpContext httpContext) at Abp.Auditing.WebAuditInfoProvider.Fill(AuditInfo auditInfo)

What’s the problem?

[hikalkan]

I cached exceptions for Dns.GetHostAddresses call. New code is like that:

protected virtual string GetClientIpAddress(HttpContext httpContext)
{
    var clientIp = httpContext.Request.ServerVariables["HTTP_X_FORWARDED_FOR"] ??
                    httpContext.Request.ServerVariables["REMOTE_ADDR"];

    try
    {
        foreach (var hostAddress in Dns.GetHostAddresses(clientIp))
        {
            if (hostAddress.AddressFamily == AddressFamily.InterNetwork)
            {
                return hostAddress.ToString();
            }
        }

        foreach (var hostAddress in Dns.GetHostAddresses(Dns.GetHostName()))
        {
            if (hostAddress.AddressFamily == AddressFamily.InterNetwork)
            {
                return hostAddress.ToString();
            }
        }
    }
    catch (Exception ex)
    {
        Logger.Debug(ex.ToString());
    }

    return clientIp;
}

As you see, I did it virtual. So, you can create a new class inherits from WebAuditInfoProvider and override GetClientIpAddress to change code for your application. Then you can replace IAuditInfoProvider service with your own implementation (https://github.com/aspnetboilerplate/aspnetboilerplate/issues/983#issuecomment-217428154).

[automys]

Thanks @tamys, this works!