502 error on preview deployment.

In my branch feature/event-input-form, deployment to hosting preview channel always failed with 502 error.

  /opt/hostedtoolcache/node/14.13.1/x64/bin/npx firebase-tools hosting:channel:deploy pr22-feature_event-input- --expires 10d --project ??? --json
  ***
    "status": "error",
    "error": "HTTP Error: 502, Unknown Error"
  ***
  ***
    "status": "error",
    "error": "HTTP Error: 502, Unknown Error"
  ***

But in another branch named feature/event-input, deployed successfully.

I think CHANNEL_ID ( pr22-feature_event-input- ) which end with - cause the error.

In this code, clipped branch name might have - at end of string.

https://github.com/FirebaseExtended/action-hosting-deploy/blob/79d1e17528573ce1a02632923c928f62009df3c3/src/getChannelId.ts#L25

Anyone know such rules about CHANNEL_ID?

ref: https://firebase.google.com/docs/hosting/manage-hosting-resources

[WiNloSt]

I use channelId as @mather commented. It worked really well.

Also, in case someone wants to still use a similar channel ID like this action produces. I set my action file like this.

      - id: set-firebase-hosting-channel-id
        name: set Firebase preview channel projectId
        run: echo "::set-output name=channel-id::$(echo pr${{ github.event.number }}-$(echo ${{ github.event.pull_request.head.ref }} | cut -c1-20) | sed -e 's/[-_]$//g' | sed -e 's/[^a-zA-Z0-9\.\-]/_/g')"

      # then you can use the channelId like so
      - uses: FirebaseExtended/action-hosting-deploy@v0
        with:
          channelId: ${{ steps.set-firebase-hosting-channel-id.outputs.channel-id }}

This removes the trailing _ or - that would cause the deployment to fail.

[mather]

Workaround: set channelId option. https://github.com/FirebaseExtended/action-hosting-deploy#channelid-string