michidk / TwitchCSharp

Twitch C# Wrapper for the Twitch v3 REST API
Apache License 2.0
43 stars 12 forks source link

Refactored TwitchAuthenticatedClient #5

Closed jerotire closed 8 years ago

jerotire commented 8 years ago

Removed override function GetRequest from TwitchAuthenticatedClient. Instead adding the Authorization header as a default header to restClient.

michidk commented 8 years ago

I personally like that code design with the extra GetRequest-method more but because in all the other clients, the headers a directly set in the constructor (like in your commit), it makes sense to change that one as well.