aarondcoleman / Fitbit.NET

Fitbit .NET API Client Library
MIT License
193 stars 138 forks source link

GetHeartRateTimeSeries is missing a null guard #234

Open joshfitabase opened 6 years ago

joshfitabase commented 6 years ago

This method used to have a null guard, but it got removed during the resolution of a merge conflict. It's important because without it, null userId can be passed to GetHeartRateTimeSeries resulting in a corrupted URL.

In practice, we've observed that HttpClient has the behavior of switching from https:// to http:// when urls contain // (as in /user//activities/ )