Mask 16 digit credit card but not 20 digit numbers

deepaltrivedi
Mega Contributor

Hi All - We have a change in requirement where we want to mask 16 digit credit card information in inbound email body but not 20 digit numbers as they are not credit card. I've below script that works perfectly but it masks any number over 12 digits -- i don't want to mask 20 digits number.

 

Thoughts on how i need to modify the bolded portion??

 

(function executeRule(current, previous /*null when async*/) {
var emailVars = [current.body,current.subject,current.body_text];
var arr = [];
for(e = 0; e < emailVars.length; e++) {


var wt = emailVars[e].match(/(https:\/\/.*wetransfer\.com)(.*)[^\s]/g);
var matches = emailVars[e].match(/(https:\/\/.*wetransfer\.com)(.*)[^\s]|([\d]){4}([ -])?([\d]){4}([ -])?([\d]){4}([ -])?([\d]){4}/g);
if(matches){
for(var i = 0; i < 16; i++) {
if(!wt || matches[i] != wt[0]) {
var numCount = 0;
var maskedCredCard = '';
for(var j = 0; j < matches[i].length; j++) {
if(matches[i][j] == '-' ||matches[i][j] == ' '|| numCount >= 12) {
maskedCredCard += matches[i][j];
}
else if(numCount < 12) {
maskedCredCard += '*';
numCount++;
}
}
if(e == 0) {
current.body = current.body.replace(matches[i] ,maskedCredCard);
}
else if(e == 1) {
current.subject = current.subject.replace(matches[i] ,maskedCredCard);
}
else if(e == 2) {
current.body_text = current.body_text.replace(matches[i], maskedCredCard);
}
} //closes WT if statement
}//closes matches for loop


} //closes if there are matches
} //closes loop through email variables
})(current, previous);

1 ACCEPTED SOLUTION

Hi,

 

Replace this below line and check

var numbers = email_body.match(/\d+/g).map(Number);

WITH

var numbers = email_body.match(/\s\d+/g).map(Number);

View solution in original post

26 REPLIES 26

asifnoor
Kilo Patron

Make this change in your code and try once

if(matches[i][j] == '-' ||matches[i][j] == ' '|| numCount >= 12) {

with

if(matches[i][j] == '-' ||matches[i][j] == ' '|| numCount == 16) {

Mark the comment as a correct answer if this works.

that didn't work. it still masked 20 digits number AND instead of masking only 12 of 16 digits - it masked all 16 digits.

 

Try replacing this condition. 

if(matches){

with

if(matches && matches[i].length <=16){

This basically ensures that the digit length is < 16, only then it enters the code. 

Mark comment as a correct answer if it works.

can you share a smaple wetransfer url of how the number contains?