jellyfin / TMDbLib

C#.Net library for TheMovieDB
MIT License
344 stars 128 forks source link

Small changes from TMDb cause bug #411

Closed uponatime2019 closed 1 year ago

uponatime2019 commented 1 year ago

https://api.themoviedb.org/3/trending/movie/Week?api_key= will cause a request error {"success":false,"status_code":5,"status_message":"Invalid parameters: Your request parameters are incorrect."}

Week should lower case to make api working

SirSparkles commented 1 year ago

Might be the same issue I am seeing?

TMDbLib.Objects.Exceptions.GeneralHttpException: TMDb returned an unexpected HTTP error at TMDbLib.Rest.RestRequest.SendInternal(HttpMethod method, CancellationToken cancellationToken) at TMDbLib.Rest.RestRequest.Get[T](CancellationToken cancellationToken) at TMDbLib.Rest.RestRequestExtensions.GetOfT[T](RestRequest request, CancellationToken cancellationToken) at TMDbLib.Client.TMDbClient.GetTrendingMoviesAsync(TimeWindow timeWindow, Int32 page, CancellationToken cancellationToken)

SirSparkles commented 1 year ago

I have done a quick fix at: https://github.com/LordMike/TMDbLib/pull/416/files

SirSparkles commented 1 year ago

Can we release this fix please?