Merge pull request #265 from csfloat/fix/resiliance-against-missing-fields

Improves Resilience From Missing Steam Response Fields

Author: Step7750

src/lib/alarms/trade_history.ts

@@ -80,7 +80,7 @@ async function getTradeHistoryFromAPI(): Promise<TradeHistoryStatus[]> {
     }
 
     const data = (await resp.json()) as TradeHistoryAPIResponse;
-    return data.response.trades
+    return (data.response?.trades || [])
         .filter((e) => e.status === 3) // Ensure we only count _complete_ trades (k_ETradeStatus_Complete)
         .filter((e) => !e.time_escrow_end || new Date(parseInt(e.time_escrow_end) * 1000).getTime() < Date.now())
         .map((e) => {

src/lib/alarms/trade_offer.ts

@@ -237,7 +237,7 @@ async function getSentTradeOffersFromAPI(): Promise<OfferStatus[]> {
     }
 
     const data = (await resp.json()) as TradeOffersAPIResponse;
-    return data.response.trade_offers_sent.map(offerStateMapper);
+    return (data.response?.trade_offers_sent || []).map(offerStateMapper);
 }
 
 async function getSentAndReceivedTradeOffersFromAPI(): Promise<{
@@ -260,8 +260,8 @@ async function getSentAndReceivedTradeOffersFromAPI(): Promise<{
 
     const data = (await resp.json()) as TradeOffersAPIResponse;
     return {
-        received: data.response.trade_offers_received.map(offerStateMapper),
-        sent: data.response.trade_offers_sent.map(offerStateMapper),
+        received: (data.response?.trade_offers_received || []).map(offerStateMapper),
+        sent: (data.response?.trade_offers_sent || []).map(offerStateMapper),
         steam_id: access.steam_id,
     };
 }