fix(scf): apigw trigger function qualifier

Author: yugasun

src/modules/scf/index.js

@@ -231,6 +231,7 @@ class Scf {
             this.region,
             funcInfo,
             event[eventType],
+            inputs.needSetTraffic,
           );
           try {
             const apigwOutput = await this.apigwClient.deploy(triggerInputs);
@@ -442,13 +443,6 @@ class Scf {
     }
 
     const outputs = funcInfo;
-    if (inputs.tags || inputs.events) {
-      if (!funcInfo) {
-        funcInfo = await this.getFunction(namespace, inputs.name);
-      }
-      await Promise.all([this.deployTags(funcInfo, inputs), this.deployTrigger(funcInfo, inputs)]);
-    }
-
     if (inputs.publish) {
       const { FunctionVersion } = await this.publishVersion({
         functionName: funcInfo.FunctionName,
@@ -459,7 +453,8 @@ class Scf {
       inputs.lastVersion = FunctionVersion;
       outputs.LastVersion = FunctionVersion;
     }
-    if (inputs.traffic !== undefined && inputs.lastVersion) {
+    inputs.needSetTraffic = inputs.traffic !== undefined && inputs.lastVersion;
+    if (inputs.needSetTraffic) {
       await this.updateAliasTraffic({
         functionName: funcInfo.FunctionName,
         region: this.region,
@@ -471,6 +466,13 @@ class Scf {
       outputs.Traffic = inputs.traffic;
     }
 
+    if (inputs.tags || inputs.events) {
+      if (!funcInfo) {
+        funcInfo = await this.getFunction(namespace, inputs.name);
+      }
+      await Promise.all([this.deployTags(funcInfo, inputs), this.deployTrigger(funcInfo, inputs)]);
+    }
+
     console.log(`Deployed funtion ${funcInfo.FunctionName}.`);
     return outputs;
   }

src/modules/scf/utils.js

@@ -6,7 +6,7 @@ const CONFIGS = require('./config');
  * @param {object} funcInfo function information
  * @param {object} inputs yml configuration
  */
-const formatApigwTrigger = (region, funcInfo, inputs) => {
+const formatApigwTrigger = (region, funcInfo, inputs, traffic = false) => {
   const { parameters, name } = inputs;
   const triggerInputs = {};
   triggerInputs.region = region;
@@ -18,6 +18,7 @@ const formatApigwTrigger = (region, funcInfo, inputs) => {
     ep.function = ep.function || {};
     ep.function.functionName = funcInfo.FunctionName;
     ep.function.functionNamespace = funcInfo.Namespace;
+    ep.function.functionQualifier = traffic ? '$DEFAULT' : '$LATEST';
     return ep;
   });
   return {
@@ -159,10 +160,10 @@ const formatCmqTrigger = (region, funcInfo, inputs) => {
   };
 };
 
-const formatTrigger = (type, region, funcInfo, inputs) => {
+const formatTrigger = (type, region, funcInfo, inputs, traffic) => {
   switch (type) {
     case 'apigw':
-      return formatApigwTrigger(region, funcInfo, inputs);
+      return formatApigwTrigger(region, funcInfo, inputs, traffic);
     case 'timer':
       return formatTimerTrigger(region, funcInfo, inputs);
     case 'cos':