Fix problem with appid filter on GetOwnedGames.

Author: Justin

src/SteamWebAPI2/Interfaces/PlayerService.cs

@@ -1,10 +1,11 @@
-using Steam.Models.SteamCommunity;
+using Newtonsoft.Json;
+using Steam.Models.SteamCommunity;
 using SteamWebAPI2.Models.SteamCommunity;
 using SteamWebAPI2.Models.SteamPlayer;
 using SteamWebAPI2.Utilities;
 using System;
 using System.Collections.Generic;
-using System.Collections.ObjectModel;
+using System.Net;
 using System.Threading.Tasks;
 
 namespace SteamWebAPI2.Interfaces
@@ -45,7 +46,7 @@ public PlayerService(string steamWebApiKey, ISteamWebInterface steamWebInterface
             }
 
             var steamWebResponseModel = AutoMapperConfiguration.Mapper.Map<
-                ISteamWebResponse<PlayingSharedGameResultContainer>, 
+                ISteamWebResponse<PlayingSharedGameResultContainer>,
                 ISteamWebResponse<ulong?>>(steamWebResponse);
 
             return steamWebResponseModel;
@@ -145,16 +146,20 @@ public async Task<ISteamWebResponse<OwnedGamesResultModel>> GetOwnedGamesAsync(u
             if (includeFreeGames.HasValue) { includeFreeGamesBit = includeFreeGames.Value ? 1 : 0; }
 
             List<SteamWebRequestParameter> parameters = new List<SteamWebRequestParameter>();
-            parameters.AddIfHasValue(includeFreeGamesBit, "include_played_Free_games");
-            parameters.AddIfHasValue(steamId, "steamid");
-            parameters.AddIfHasValue(includeAppInfoBit, "include_appinfo");
 
-            // join the app ids by commas since that's what the Steam Web API expects
-            if (appIdsToFilter != null)
+            var inputJsonObj = new
             {
-                string appIdsDelimited = String.Join(",", appIdsToFilter);
-                parameters.AddIfHasValue(appIdsDelimited, "appids_filter");
-            }
+                steamid = steamId,
+                include_played_Free_games = includeFreeGamesBit,
+                include_appinfo = includeAppInfoBit,
+                appids_filter = appIdsToFilter
+            };
+            var inputJson = JsonConvert.SerializeObject(inputJsonObj, new JsonSerializerSettings()
+            {
+                NullValueHandling = NullValueHandling.Ignore
+            });
+
+            parameters.AddIfHasValue(WebUtility.UrlEncode(inputJson), "input_json");
 
             var steamWebResponse = await steamWebInterface.GetAsync<OwnedGamesResultContainer>("GetOwnedGames", 1, parameters);
 

src/SteamWebAPI2/Utilities/SteamWebRequest.cs

@@ -1,8 +1,8 @@
 using Newtonsoft.Json;
-using SteamWebAPI2.Utilities;
 using System;
 using System.Collections.Generic;
 using System.Diagnostics;
+using System.Linq;
 using System.Net;
 using System.Net.Http;
 using System.Threading.Tasks;
@@ -122,14 +122,16 @@ private async Task<ISteamWebResponse<T>> SendWebRequestAsync<T>(HttpMethod httpM
                 httpResponse = await httpClient.PostAsync(command).ConfigureAwait(false);
             }
 
+            var headers = httpResponse.Content?.Headers;
+
             // extract http headers that we care about
             SteamWebResponse<T> steamWebResponse = new SteamWebResponse<T>()
             {
-                ContentLength = httpResponse.Content.Headers.ContentLength,
-                ContentType = httpResponse.Content.Headers.ContentType.MediaType,
-                ContentTypeCharSet = httpResponse.Content.Headers.ContentType.CharSet,
-                Expires = httpResponse.Content.Headers.Expires,
-                LastModified = httpResponse.Content.Headers.LastModified,
+                ContentLength = headers?.ContentLength,
+                ContentType = headers?.ContentType?.MediaType,
+                ContentTypeCharSet = headers?.ContentType?.CharSet,
+                Expires = headers?.Expires,
+                LastModified = headers?.LastModified,
             };
 
             // deserialize the content if we have any as indicated by the response code
@@ -152,7 +154,7 @@ private async Task<ISteamWebResponse<T>> SendWebRequestAsync<T>(HttpMethod httpM
         /// <param name="methodVersion">Example: 1</param>
         /// <param name="parameters">Example: { key: 8A05823474AB641D684EBD95AB5F2E47 } </param>
         /// <returns></returns>
-        private string BuildRequestCommand(string interfaceName, string methodName, int methodVersion, IList<SteamWebRequestParameter> parameters)
+        private string BuildRequestCommand(string interfaceName, string methodName, int methodVersion, IEnumerable<SteamWebRequestParameter> parameters)
         {
             Debug.Assert(!String.IsNullOrWhiteSpace(interfaceName));
             Debug.Assert(!String.IsNullOrWhiteSpace(methodName));
@@ -166,7 +168,7 @@ private string BuildRequestCommand(string interfaceName, string methodName, int
             string commandUrl = String.Format("{0}/{1}/{2}/v{3}/", steamWebApiBaseUrl, interfaceName, methodName, methodVersion);
 
             // if we have parameters, join them together with & delimiter and append them to the command URL
-            if (parameters != null && parameters.Count > 0)
+            if (parameters != null && parameters.Count() > 0)
             {
                 string parameterString = String.Join("&", parameters);
                 commandUrl += String.Format("?{0}", parameterString);