Skip to content

Fix ghunt/parsers/people.py#598

Open
xvkzl wants to merge 1 commit into
mxrch:masterfrom
xvkzl:master
Open

Fix ghunt/parsers/people.py#598
xvkzl wants to merge 1 commit into
mxrch:masterfrom
xvkzl:master

Conversation

@xvkzl
Copy link
Copy Markdown

@xvkzl xvkzl commented Jun 2, 2026

Fixed error!

THE ERROR WAS
GHunt 2.3.4 crashes when processing certain accounts due to an unhandled missing container key in coverPhoto metadata.

Traceback ends at:

self.coverPhotos[cover_photo_data["metadata"]["container"]] = person_cover_photo

with:

KeyError: 'container'

It appears some coverPhoto entries returned by Google's API no longer contain metadata.container.

Suggested fix:

container = cover_photo_data.get("metadata", {}).get("container")

if container:
    self.coverPhotos[container] = person_cover_photo

This prevents GHunt from crashing when the field is absent and preserves existing behavior when it is present.

Environment:

  • GHunt 2.3.4
  • Windows
  • Python 3.14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant