Fix KeyError: 'container' crash across all Person parser fields + profile photo mis-attribution

[Iskandeur]

Problem

#597 (closed as completed) and #591 report KeyError: 'container' crashing ghunt email. #592 fixed this for coverPhoto only and got merged, but people are still hitting the crash on other fields (see continued reports in #597 after #592 merged, e.g. from @annithehunter and @bourhanb) — because metadata.container can be missing on email, name, readOnlyProfileInfo, sourceIds, and inAppReachability entries too, not just coverPhoto. All of these use the same unguarded data["metadata"]["container"] pattern.

This PR generalizes the fix started in #592 to every field that does this lookup, and also fixes a separate real bug found while auditing the same function.

What changed in ghunt/parsers/people.py

1. Guard every metadata.container access, not just coverPhoto. Entries missing container are skipped (not collected under a synthetic "unknown" key). A fallback key was considered (and is what Fix KeyError when metadata.container is missing in People parser #593/fix: key profile photos by their own container instead of the outer profile-info loop's #602 use), but risks silently colliding two unrelated container-less entries under the same key, overwriting one with the other — skipping preserves data integrity for the entries that do have a valid container, per the reasoning validated in Fix KeyError when metadata.container is missing in People parser #593's thread by @iam-rubber-ducky.
2. Fixed profile photo mis-attribution: person_data["photo"] was being looped inside the readOnlyProfileInfo loop and keyed by profile_data's container instead of the photo's own metadata.container. This meant every photo got attributed to whichever profile container the outer loop happened to be on at the time, silently dropping/misassigning photos when there are multiple containers. Moved the photo loop out and keyed by its own container (same root cause as fix: key profile photos by their own container instead of the outer profile-info loop's #602, independently confirmed while auditing this function).
3. Guarded coverPhoto's imageUrl lookup against being absent (would otherwise raise AttributeError on .split() after the container fix stops the KeyError).

Relation to existing PRs

• Supersedes Fix KeyError when metadata.container is missing in People parser #593 (same scope, tested and confirmed working by @iam-rubber-ducky in that thread) and Fix ghunt/parsers/people.py #598 (smaller version of the same fix) — happy to close this in favor of either if you prefer that diff, the intent is the same.
• Additionally includes the fix from fix: key profile photos by their own container instead of the outer profile-info loop's #602 (profile photo container mis-attribution), found while extending the same defensive pattern to every field in this function.

Testing

Verified against a real account previously triggering KeyError: 'container' on ghunt email — completes without crashing, and profile photo container mapping is now correct.