Skip to content

Commit

Permalink
cdn: Cope with depots with no name
Browse files Browse the repository at this point in the history
App ID 1628350 "Steam Linux Runtime - Sniper" contains depot 1628351,
which has no name, causing steam.client.cdn.CDNClient to crash with
a KeyError while retrieving manifests from that depot. Fall back to
using the depot ID as a placeholder name.

Signed-off-by: Simon McVittie <smcv@collabora.com>
  • Loading branch information
smcv authored and rossengeorgiev committed May 3, 2022
1 parent be5d6ad commit 8e01e6d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions steam/client/cdn.py
Original file line number Diff line number Diff line change
Expand Up @@ -740,7 +740,7 @@ def async_fetch_manifest(app_id, depot_id, manifest_gid, decrypt, name):
and depot_id not in self.licensed_depot_ids
and depot_id not in self.licensed_app_ids):
self._LOG.debug("No license for depot %s (%s). Skipping...",
repr(depot_info['name']),
repr(depot_info.get('name', depot_id)),
depot_id,
)
continue
Expand Down Expand Up @@ -769,7 +769,7 @@ def async_fetch_manifest(app_id, depot_id, manifest_gid, decrypt, name):
depot_id,
manifest_gid,
decrypt,
depot_info['name'],
depot_info.get('name', depot_id),
))

# collect results
Expand Down

0 comments on commit 8e01e6d

Please sign in to comment.