Revert "Revert "[expo-assets] search for key without file extension (…

[jkhales]

…#12624)""

This reverts commit 58a7adb.

Why

The true cause of #13558 was a5dbb5b

Reverting this triggered a fall back measure which fixed the issue in an incomplete way. In detail (thanks @esamelson): Before the revert, we were feeding a dictionary of asset locations that looked like this:

{
  hash1: '/path/to/file/hash1',
  hash2: '/path/to/file/hash2'
}

The iOS AV module was directed to the correct file, but then crashed as these files are missing an extension. Once the revert landed, the local assets could not be found and so they were downloaded from an external URL, avoiding a crash.

Now that assets are saved with an extension, we can reintroduce this safely.

How

re-revert.

Test Plan

• mp3's work No sound on iOS in production #13558 (comment)
• assets loaded locally
• classic updates work

Checklist

• Documentation is up to date to reflect these changes (eg: https://docs.expo.io and README.md).
• This diff will work correctly for expo build (eg: updated @expo/xdl).
• This diff will work correctly for expo prebuild & EAS Build (eg: updated a module plugin).

[expo-bot]

The review previously left here is no longer valid, jump to the latest one 👉 #13789 (review)

[expo-bot]

Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines.

Looks like I have nothing to complain about 👏 Keep up the good work! 💪

Generated by ExpoBot 🤖 against 6dbaaaf