My problem is very similar to this question here. I have an AuthenticationService class that makes an HttpClient PostAsync() and never returns the result when I am running it from the ASP project, but when I implement it in a Console app it works just fine.
This is my Authentication Service class:
public class AuthenticationService : BaseService
{
public async Task<Token> Authenticate (User user, string url)
{
string json = JsonConvert.SerializeObject(user);
StringContent content = new StringContent(json, Encoding.UTF8, "application/json");
HttpResponseMessage response = await _client.PostAsync(url, content);
string responseContent = await response.Content.ReadAsStringAsync();
Token token = JsonConvert.DeserializeObject<Token>(responseContent);
return token;
}
}
And it is here where it hangs: HttpResponseMessage response = await _client.PostAsync(url, content);
Here is my Controller calling the service:
public ActionResult Signin(User user)
{
// no token needed to be send - we are requesting one
Token token = _authenticationService.Authenticate(user, ApiUrls.Signin).Result;
return View();
}
Here is an example of how I have been testing the service using a Console app, and it runs just fine.
class Program
{
static void Main()
{
AuthenticationService auth = new AuthenticationService();
User u = new User()
{
email = "<a href="https://getridbug.com/cdn-cgi/l/email-protection" class="__cf_email__" data-cfemail="d4b1b9b5bdb894bcbba0b9b5bdb8fab7bbb9">[email protected]</a>",
password = "password123"
};
Token newToken = auth.Authenticate(u, ApiUrls.Signin).Result;
Console.Write("Content: " + newToken.user._id);
Console.Read();
}
}
Answers:
Thank you for visiting the Q&A section on Magenaut. Please note that all the answers may not help you solve the issue immediately. So please treat them as advisements. If you found the post helpful (or not), leave a comment & I’ll get back to you as soon as possible.
Method 1
Since you are using .Result, this will end up causing a deadlock in your code. The reason this is working in a console application is because console applications don’t have contexts, but ASP.NET apps do (see Stephen Cleary’s Don’t Block on Async Code). You should make the Signin method in your controller async and await the call to _authenticationService.Authenticate to resolve the deadlock issue.
Method 2
Since you are using .Result or .Wait or await this will end up causing a deadlock in your code.
you can use ConfigureAwait(false) in async methods for preventing deadlock
like this:
string responseContent = await response.Content.ReadAsStringAsync().ConfigureAwait(false);
you can use
ConfigureAwait(false)wherever possible for Don’t Block Async Code .
Method 3
In case someone comes and needs to see code I just change the controller to something like this:
/***
*** Added async and Task<ActionResult>
****/
public async Task<ActionResult> Signin(User user)
{
//no token needed - we are requesting one
// added await and remove .Result()
Token token = await _authenticationService.Authenticate(user, ApiUrls.Signin);
return RedirectToAction("Index", "Dashboard", token.user);
}
Thank you all for your quick response!
All methods was sourced from stackoverflow.com or stackexchange.com, is licensed under cc by-sa 2.5, cc by-sa 3.0 and cc by-sa 4.0