Update DigitalAssetLinks.ts

[hslee2008]

Problem (?) I had before

After using bubblewrap for the first time, I had 1 issue that took me days to fix.
It was the Digital Asset Links JSON file.

The problem was the fingerprints were spread out on one item in arrays.

[
  {
    "relation": [
      "delegate_permission/common.handle_all_urls"
    ],
    "target": {
      "namespace": "android_app",
      "package_name": "app.web.test.twa",
      "sha256_cert_fingerprints": ["fingerprint1"]
    }
  },
  {
    "relation": [
      "delegate_permission/common.handle_all_urls"
    ],
    "target": {
      "namespace": "android_app",
      "package_name": "app.web.test.twa",
      "sha256_cert_fingerprints": ["fingerprint2"]
    }
  }
]

This was how the file was generated.

Fix

After looking at few sources, I came to conclusion that the assetlinks.json file should be something like below.

1. Example Format in AssetLinks docs - Google
2. Example File in AssetLinks docs - Google
3. Also, looking at few examples such as Google Docs AssetLinks.json, we can see that the fingerprints are contained in arrays by package names.
4. Checking out why my site was not validated in Google Play Console

Note: I could not recreate the error message Google Play Console told me to update my asssetlinks.json file to contain all the fingerprints in one array. I will edit this pull request again if I manage to do so.

[
  {
    "relation": [
      "delegate_permission/common.handle_all_urls"
    ],
    "target": {
      "namespace": "android_app",
      "package_name": "app.web.test.twa",
      "sha256_cert_fingerprints": [
        "fingerprint1",
        "fingerprint2"
      ]
    }
  }
]

Todos

• Update corresponding testing files
• Recreate Google Play Console Error Message

[andreban]

Looping in @PEConn, as he may remember the details better. IIRC, listing all the fingerprints in the array is the approach thought to be right initially, but then found that the current implementation, which is documented here, is the correct one.

[PEConn]

It turns out we were originally correct and our "correction" was unnecessary. According to this - "This field supports multiple fingerprints, which can be used to support different versions of your app, such as debug and production builds.".

So, putting all the fingerprints in a single array should work. (However, my understanding is that having separate entries should also work.)

[andreban]

Ah, thanks for clarifying @PEConn. PR looks good to me then.

[andreban]

LGTM, thank you!