Fix deprecated aws-cli login command

get-login -> get-login-password

README.md

@@ -1,4 +1,5 @@
 # Docker Build & Push Action
+
 <p><a href="https://github.com/fourteenfish/docker-build-push"><img alt="GitHub Actions status" src="https://github.com/fourteenfish/docker-build-push/workflows/Tests/badge.svg"></a></p>
 
 Builds a Docker image and pushes it to the private registry of your choosing.
@@ -94,7 +95,7 @@ By default, if you do not pass a `tag` input this action will use an algorithm b
 | /refs/heads/master              | 1234567    | master-1234567       |                                                        |
 | /refs/heads/SOME-feature        | 1234567    | some-feature-1234567 |                                                        |
 
-Tags prepended with 'relsease', such as `release/myapp-v1.0.0`, will return a Docker tag of just the part after the last dash, eg. v1.0.0. This can be useful when you have a 'mono-repository' containing multiple apps, but want to trigger github actions for specific apps. So, a github action for 'myapp' can be set up to trigger only on tags that filter for that app:
+Tags prepended with 'release', such as `release/myapp-v1.0.0`, will return a Docker tag of just the part after the last dash, eg. v1.0.0. This can be useful when you have a 'mono-repository' containing multiple apps, but want to trigger github actions for specific apps. So, a github action for 'myapp' can be set up to trigger only on tags that filter for that app:
 
 ```yaml
 on:

dist/index.js

@@ -614,7 +614,9 @@ const login = () => {
   if (isEcr(registry)) {
     const region = getRegion(registry);
     core.info(`Logging into ECR region ${region}...`);
-    cp.execSync(`$(aws ecr get-login --region ${region} --no-include-email)`);
+    cp.execSync(
+      `aws ecr get-login-password --region ${region} | docker login --username AWS --password-stdin ${registry}`
+    );
   } else if (username && password) {
     core.info(`Logging into Docker registry ${registry}...`);
     cp.execSync(`docker login -u ${username} --password-stdin ${registry}`, {

src/docker.js

@@ -86,7 +86,9 @@ const login = () => {
   if (isEcr(registry)) {
     const region = getRegion(registry);
     core.info(`Logging into ECR region ${region}...`);
-    cp.execSync(`$(aws ecr get-login --region ${region} --no-include-email)`);
+    cp.execSync(
+      `aws ecr get-login-password --region ${region} | docker login --username AWS --password-stdin ${registry}`
+    );
   } else if (username && password) {
     core.info(`Logging into Docker registry ${registry}...`);
     cp.execSync(`docker login -u ${username} --password-stdin ${registry}`, {

tests/docker.test.js

@@ -168,7 +168,9 @@ describe('core and cp methods', () => {
 
       docker.login();
 
-      expect(cp.execSync).toHaveBeenCalledWith(`$(aws ecr get-login --region us-east-1 --no-include-email)`);
+      expect(cp.execSync).toHaveBeenCalledWith(
+        `aws ecr get-login-password --region us-east-1 | docker login --username AWS --password-stdin ${registry}`
+      );
     });
 
     test("returns undefined if empty login and doesn't execute command", () => {