7digital/SevenDigital.Api.Wrapper — pull request #35 by knocte (2012-11-13 12:25:09+00:00)
Title: Exceptions: Include Headers property in ApiException

For debugging purposes, headers are useful (in case, you need for example
to inspect 'X-7d-sys-origin'?

I have been conservative in this change because otherwise I would maybe break WebTeams usage :)

What I mean, specifically:
- Now that ApiException has all 3 properties from the Response class, maybe we should just store the Response object as a property! But not sure WebTeam likes this because it increasis coupling.
- We should remove the Response ctor that takes no Headers parameter, just in case it is misused (right now it's only used for tests, but tests could just use the full ctor and pass a null or empty IDictionary).

Let me know what you think.
Thanks