Fixing a Sonarqube bug regarding HttpResponseMessage in c#

Question

I have code something like this:

HttpResponseMessage response = new HttpResponseMessage();
try
{
    string url = ConfigurationManager.AppSettings["url"];
    using (HttpRequestMessage requestMessage = new HttpRequestMessage(HttpMethod.Get, url))
    {
        requestMessage.Headers.Add("Accept", "application/json");                                
        response = await SendHttpRequest(requestMessage).ConfigureAwait(false);                                
    }
    if (response != null && response.IsSuccessStatusCode)
    {
        // do other things
    }
    else
    {
        log.Error("Http response failure with Status Code : " + response.StatusCode"); // sonarqube showing a bug here
    }
}
catch (Exception ex2)
{
    // log the exception                        
}


public virtual async Task<HttpResponseMessage> SendHttpRequest(HttpRequestMessage httpRequest)
{
    HttpResponseMessage response = null;
    using (HttpClient client = new HttpClient())
    {
        response = await client.SendAsync(httpRequest).ConfigureAwait(false);
    }
    return response;
}

When I run Sonarqube analysis on this code, it is throwing a bug at the commented line in the code, saying "'response' is null on at least one execution path" How can I make this go away. I have intialized HttpResponseMessage , but still this is throwing a bug.
Any help in this regard will be helpful.

Buddhabhushan Kamble · Accepted Answer · 2019-10-19 08:18:02Z

1

Please check the below code

 if (response != null && response.IsSuccessStatusCode)
{
    // do other things
}
else
{
    log.Error("Http response failure with Status Code : " + response.StatusCode"); // sonarqube showing a bug here
}

In your code SonarQube thinks that there are chances of null reference exception at response.StatusCode because in if condition you checked it for not null. IF it is null then it will go to else where you are accessing property of null object.

Just to resolve the SonarQube error please update your line with below

log.Error("Http response failure with Status Code : " + response?.StatusCode"); // sonarqube showing a bug here

Hopefully this will work

answered Oct 19, 2019 at 8:18

Buddhabhushan Kamble

2901 silver badge7 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

CrazyCoder Over a year ago

Thanks, this solved my issue. Can you also may be help me in resolving this issues I posted : stackoverflow.com/q/58446681/8930751 or stackoverflow.com/q/58405873/8930751 . Many thanks in advance.

Collectives™ on Stack Overflow

Fixing a Sonarqube bug regarding HttpResponseMessage in c#

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related