Skip to content

Commit

Permalink
fix(lambda): Removed ternary operator
Browse files Browse the repository at this point in the history
  • Loading branch information
kevinseq1 authored and RonWang committed Feb 26, 2021
1 parent b1bbb2c commit 33fcf1f
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions lib/clients/aws-client/iam-client.js
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,7 @@ module.exports = class IAMClient extends AbstractAwsClient {
*/
_generateIAMRoleName(skillName) {
const roleNamePrefix = process.env.ASK_DEPLOY_ROLE_PREFIX || 'ask-lambda';
const validSkillName = skillName.replace(/_/g, '-');
const truncatedSkillName = (validSkillName.length > 39) ? validSkillName.substr(0, 39 - 1) : validSkillName;
return `${roleNamePrefix}-${truncatedSkillName}-${Date.now()}`;
const validSkillName = skillName.replace(/_/g, '-').substr(0, 39 - 1);
return `${roleNamePrefix}-${validSkillName}-${Date.now()}`;
}
};

0 comments on commit 33fcf1f

Please sign in to comment.