Windows.Web.Http.HttpClient#GetAsync throws an incomplete exception when invalid credentials are used with basic authentication

Question

I am working on a Windows Runtime Component which makes API calls. Until earlier today I used the HttpClient and related models from System.Net but switched over to Windows.Web instead to leverage the WinRT streams.

Aside from changing the using statements, swapping HttpContent to IHttpContent and using the WindowsRuntimeExtensions to change my IInputStream to Stream for JSON.NET, I didn't have to do anything special. However suddenly 3 out of my 16 tests fail whereas previously everything worked.

All 3 (integration) tests validate that I receive an error response when logging in with invalid credentials. There are other tests that include logging in as well (but with valid credentials) and they work just fine. The given error message is of type AggregateException and has as message

System.AggregateException: One or more errors occurred. ---> System.Exception: Element not found.

A dialog cannot be displayed because the parent window handle has not been set.

The exception contains HRESULT values. The outerexception has value -2146233088 which corresponds to 0x80131500 while the innerexception has -2147023728 which corresponds to 0x80070490. Neither of those are a known error code on the MSDN page.

Following investigation:

0x80131500 corresponds to COR_E_EXCEPTION
0x80070490 corresponds to ERROR_NOT_FOUND

Stacktrace:

Result StackTrace:  
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at xx.Models.Requests.GetRequest.<ExecuteRequestAsync>d__0.MoveNext() in c:\Users\jeroen\Github\Windows-app\xx\xx\Models\Requests\Request.cs:line 17

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at xx.ApiDispatcher.<ExecuteAsync>d__0`2.MoveNext() in c:\Users\jeroen\Github\Windows-app\xx\xx\ApiDispatcher.cs:line 40

 --- End of inner exception stack trace ---

    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
   at System.Threading.Tasks.Task`1.get_Result()
   at xx.ApiDispatcher.Execute[TCallResult,TResponseObject](ApiCall`2 call) in c:\Users\jeroen\Github\Windows-app\xx\xx\ApiDispatcher.cs:line 22

Originally my question was worded somewhat differently because the actual problem seemed to be hidden. I have found out that the GET request by the HttpClient returns back to the caller instead of awaiting the result of the call (and executing the rest of the method).

In my project, executing the line var data = await myHttpClient.GetAsync(url); will return to the calling method with a non-constructed object and subsequent lines that come after the GetAsync() call are simply not executed.

Adding .ConfigureAwait(false) to stop it from going back did not make a difference.

The AggregateException is thrown when a user tries to login with invalid credentials. For some reason the HttpClient decides to throw an exception then without giving me a return value I could use. The problem here is that it does not tell me what kind of exception: catching COMException, TaskCanceledException, AggregateException and Exception only trigger the latter.

I have also found out that asynchronous integration tests do not work well with the multithreaded MSTest environment, so that explains several other failed tests that I had (but worked just fine individually)

I also, finally, have an example that demonstrates the problem (but I can't provide a webservice that takes basic auth)!

[TestMethod]
public void TestMethod3()
{
    Assert.IsTrue(new Test().Do().AsTask().Result);
}

public sealed class Test
{
   public IAsyncOperation<bool> Do()
   {
       return DoSomething().AsAsyncOperation();
   } 

   private async Task<bool> DoSomething()
   {
       var client = new HttpClient();
       var info = "[email protected]:nopass";
       var token = Convert.ToBase64String(Encoding.UTF8.GetBytes(info));
       client.DefaultRequestHeaders.Authorization = new HttpCredentialsHeaderValue("Basic", token);

       var data = await client.GetAsync(new Uri("https://mytestdomain/v2/apikey?format=Json"));
       return true;
   }
}

Executing this code with a valid password will return true while an invalid password will throw an AggregateException.

Right now I am working around the problem by catching a general Exception around the call to GetAsync() but this is very rudimentary and I'd like to know why this incomplete exception is thrown in the first place.

Could you try setting different authentication credentials: client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Basic", token); — Yuval Itzchakov
– Yuval Itzchakov, Commented Jun 24, 2014 at 21:02
@YuvalItzchakov: that is part of the System.Net.Http namespace and not compatible with the Windows.Web.Http.HttpClient unless I am overlooking something. — Jeroen Vannevel
– Jeroen Vannevel, Commented Jun 24, 2014 at 21:06
Have you tried creating a HttpHandler and settings its Credentials property, then passing it to HttpClient? var client = new HttpClient(handler) — Yuval Itzchakov
– Yuval Itzchakov, Commented Jun 24, 2014 at 21:17

Yuval Itzchakov · Accepted Answer · 2016-10-09 19:20:36Z

32

+250

After reconstructing your example and playing around, I figured out what happens.

var data = await client.GetAsync(new Uri("https://mytestdomain/v2/apikey?format=Json"));

The GetAsync method invokes the HTTP request with invalid credentials. What happens is that the returned request tries to look for a window where you can enter the correct credentials, but doesn't find one. Hence it throws an Element Not Found while searching for that window.

This can be fixed by creating a HttpBaseProtocolFilter and setting the AllowUI property to false and then passing it to the HttpClient:

private async Task<bool> DoSomething()
{
    var httpBaseFilter = new HttpBaseProtocolFilter
    {
        AllowUI = false
    };

    var client = new HttpClient(httpBaseFilter);
    var info = "[email protected]:nopass";
    var token = Convert.ToBase64String(Encoding.UTF8.GetBytes(info));
    client.DefaultRequestHeaders.Authorization = new HttpCredentialsHeaderValue("Basic", token);

    var data = await client.GetAsync(new Uri("https://mytestdomain/v2/apikey?format=Json"));
    return true;
}

Response after adding HttpBaseProtocolFilter

edited Oct 9, 2016 at 19:20

answered Jun 25, 2014 at 13:35

Yuval Itzchakov

150k32 gold badges275 silver badges333 bronze badges

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

3 Comments

Jeroen Vannevel Over a year ago

I appreciate the thorough help! You are entirely correct, this solved it. I'll leave the bounty open for some more time so there will be more views for your answer but rest assured that you'll get it.

tavier Over a year ago

Does this behavior depend on the platform of the device sending the request? When I send this request from Win RT tablet I get "Element not found" error (and setting AllowUI to false worked there) but in WP device I got the correct error: UnAuthorized (even without setting AllowUI to false).

Yuval Itzchakov Over a year ago

@Ashish Not sure, I've never tried dispatching a request from WinRT.

James Newton-King · Accepted Answer · 2016-05-05 23:07:38Z

1

Setting AllowUI on HttpBaseProtocolFilter to false will stop this error.

If however you do want a dialog box to display, allowing the user to enter credentials, then the web request needs to be started on the UI thread.

answered May 5, 2016 at 23:07

James Newton-King

49.2k25 gold badges113 silver badges132 bronze badges

Comments

Joshua I · Accepted Answer · 2014-06-25 11:26:47Z

0

I think the problem is that the exception is actually thrown by your call to EnsureSuccessStatusCode.

Did you try adding HttpResponseMessage class EnsureSuccessStatusCode() method and check.

http://msdn.microsoft.com/en-us/library/system.net.http.httpresponsemessage.ensuresuccessstatuscode.aspx

After this line:

var data = await client.GetAsync(new Uri("https://mytestdomain/v2/apikey?format=Json"));
date.EnsureSuccessStatusCode();

answered Jun 25, 2014 at 11:26

Joshua I

5603 gold badges8 silver badges22 bronze badges

1 Comment

Yuval Itzchakov Over a year ago

The exception is thrown at the await point. It has nothing to do with EnsureSuccessStatusCode

Collectives™ on Stack Overflow

Windows.Web.Http.HttpClient#GetAsync throws an incomplete exception when invalid credentials are used with basic authentication

3 Answers 3

3 Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related