fix: remove Lambda@Edge" (#38)

ttoss · Feb 23, 2022 · 12a0e8c · 12a0e8c
1 parent eaa9a20
commit 12a0e8c
Show file tree

Hide file tree

Showing 7 changed files with 76 additions and 1,381 deletions.
diff --git a/packages/cli/jest.config.js b/packages/cli/jest.config.js
@@ -2,7 +2,7 @@ module.exports = {
   collectCoverage: true,
   coverageThreshold: {
     global: {
-      branches: 34,
+      branches: 32,
       functions: 54,
       lines: 59,
       statements: 62,

diff --git a/packages/cli/src/deploy/cicd/lambdas/ecsTaskReport.handler.ts b/packages/cli/src/deploy/cicd/lambdas/ecsTaskReport.handler.ts
@@ -75,16 +75,24 @@ export const getEcsTaskTags = async ({
 
     const task = tasks?.[0];
 
+    console.log({ tasks, cluster, ecsTaskArn });
+
     if (!task) {
       return undefined;
     }
 
     return task.tags;
-  } catch {
+  } catch (error) {
+    console.error(error);
     return undefined;
   }
 };
 
+getEcsTaskTags({
+  ecsTaskArn:
+    'arn:aws:ecs:us-east-1:483684946879:task/CarlinCicdCarlinMonorepo-RepositoryTasksECSCluster-1J6saGT91hCr/f70d559c47804d6383df170712d3e455',
+}).then();
+
 /**
  * - MainTagFound: means that the main has a tag, so the main pipeline should
  * be skipped because the push was only to update versions and changelogs.

diff --git a/packages/cli/src/deploy/staticApp/command.spec.ts b/packages/cli/src/deploy/staticApp/command.spec.ts
@@ -74,8 +74,6 @@ describe('handling methods', () => {
       acm: faker.random.word(),
       aliases: [faker.random.word()],
       cloudfront: true,
-      csp: {},
-      gtmId: faker.random.word(),
       spa: true,
     };
 
@@ -92,8 +90,6 @@ describe('handling methods', () => {
       acm: faker.random.word(),
       aliases: [faker.random.word()],
       cloudfront: true,
-      csp: {},
-      gtmId: faker.random.word(),
       spa: true,
     };
 
@@ -139,12 +135,6 @@ describe('should set cloudfront', () => {
       {
         spa: true,
       },
-      {
-        csp: {},
-      },
-      {
-        gtmId: faker.random.word(),
-      },
       {
         acm: 'some string',
       },

diff --git a/packages/cli/src/deploy/staticApp/command.ts b/packages/cli/src/deploy/staticApp/command.ts
@@ -34,21 +34,6 @@ export const options = {
     require: false,
     type: 'boolean',
   },
-  csp: {
-    coerce: (value: any) => {
-      if (value === 'false') {
-        return false;
-      }
-
-      return value;
-    },
-    describe: 'CSP headers to be added to Lambda@Edge origin response.',
-  },
-  'gtm-id': {
-    describe: 'Id of the GTM container.',
-    require: false,
-    type: 'string',
-  },
   'hosted-zone-name': {
     required: false,
     describe: `Is the name of a Route 53 hosted zone. If this value is provided, ${NAME} creates the subdomains defined on \`--aliases\` option. E.g. if you have a hosted zone named "sub.domain.com", the value provided may be "sub.domain.com".`,