Closed andrewcoll closed 9 years ago
Was just about to look at this myself and see there is already a PR in place. OSS is the best! Hope this gets merged in and the nuget package gets updated soon :-)
Thanks for this - and sorry for the delay. I fixed up some stylistic things - I move to only null-checking the configuration value to match other places where we look for strings in the configuration eg, User-Agent
.
I could be convinced that we should do a null-or-empty type check instead, but I'd want to do it universally.
Good call @WestDiscGolf - I pushed out release 0.1.2, so there should be an updated nuget package now. Thanks!
Passes an OAuth token in the header if it's provided in the config.