From 8fd06c4c8cb280e7b671329ef35ba1490ac8a6d7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Karol=20S=C3=B3jko?= <karolsojko@protonmail.com>
Date: Wed, 19 Oct 2022 11:42:52 +0200
Subject: [PATCH] fix: docker tags on publishing workflow

---
 .github/workflows/server-application.yml | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/server-application.yml b/.github/workflows/server-application.yml
index 9eaf73c86..879e30ed6 100644
--- a/.github/workflows/server-application.yml
+++ b/.github/workflows/server-application.yml
@@ -225,10 +225,6 @@ jobs:
 
     - name: Publish Docker image
       uses: docker/build-push-action@v3
-      env:
-        ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
-        ECR_REPOSITORY: ${{ inputs.service_name }}
-        IMAGE_TAG: ${{ github.sha }}
       with:
         builder: ${{ steps.buildx.outputs.name }}
         context: ${{ steps.bundle.outputs.dir }}
@@ -237,8 +233,8 @@ jobs:
         push: true
         tags: |
           standardnotes/${{ inputs.service_name }}:latest
-          $ECR_REGISTRY/$ECR_REPOSITORY:$IMAGE_TAG
-          $ECR_REGISTRY/$ECR_REPOSITORY:latest
+          ${{ steps.login-ecr.outputs.registry }}/${{ inputs.service_name }}:${{ github.sha }}
+          ${{ steps.login-ecr.outputs.registry }}/${{ inputs.service_name }}:latest
 
   deploy-web:
     needs: publish