From 64f73f082e9615d5ebb5672f66beff037118d075 Mon Sep 17 00:00:00 2001
From: Aldred Benedict <45155724+aldredb@users.noreply.github.com>
Date: Thu, 9 Sep 2021 08:48:35 +0800
Subject: [PATCH] Change ECR Login Command

The current command `aws ecr get-login --no-include-email` is no longer valid. Image push will fail with error `no basic auth credentials`
---
 content/prerequisites/bootstrapsh.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/content/prerequisites/bootstrapsh.md b/content/prerequisites/bootstrapsh.md
index 9e055be..ba7ffa4 100644
--- a/content/prerequisites/bootstrapsh.md
+++ b/content/prerequisites/bootstrapsh.md
@@ -142,7 +142,7 @@ cat > ~/environment/scripts/build-containers <<-"EOF"
 CRYSTAL_ECR_REPO=$(jq < cfn-output.json -r '.CrystalEcrRepo')
 NODEJS_ECR_REPO=$(jq < cfn-output.json -r '.NodeJSEcrRepo')
 
-$(aws ecr get-login --no-include-email)
+$(aws ecr get-login-password | docker login --username AWS --password-stdin $CRYSTAL_ECR_REPO)
 
 docker build -t crystal-service ecsdemo-crystal
 docker tag crystal-service:latest $CRYSTAL_ECR_REPO:vanilla