Add Error Status Code about alexa.net.profile HOT 5 OPEN

Hi @ncipollina - thanks for raising your comment.

Because exception handling is an area where everyone has different opinions - the extension doesn't try and hide or wrap any exceptions. If you receive a 403 your skill code should be able to catch that WebException and deal with it accordingly.

There are error handlers within https://github.com/stoiveyp/Alexa.NET.RequestHandlers which can help isolate and maintain error logic, but if you have suggestions on a better way of handling that it'd be great to have a discussion on those.

from alexa.net.profile.

The problem is that HttpClient doesn't throw a WebException, it throws HttpRequestException, which does not include the status code.

from alexa.net.profile.

Ahh okay. If it's not part of the underlying exception given to us from .NET - I'm not sure I understand how we could return it consistently?

from alexa.net.profile.

I would handle it by calling the EnsureSuccessStatusCode method, and if the status is an error, return a custom exception with the status code. The consumer of the API could use that to know that it failed due to a permissions or other issue.

from alexa.net.profile.

At this point we're implementing a custom exception structure as well as altering the way we deal with the requests at an HttpClient level.

I'm happy to keep it as a future feature but short term I'd recommend using the custom client implementation (link below). Thanks for the information though.

https://github.com/stoiveyp/Alexa.NET.CustomerProfile/blob/master/Alexa.Net.CustomerProfile/CustomerProfileClient.cs#L31

from alexa.net.profile.