UserId module - invalid refreshInSeconds value silently ignored

[abazylewicz-id5]

Type of issue

question/possibly a bug

Description

When the partner sets refreshInSeconds value which is not a number (eg. a string with a number "3600", or even a string with number operation like "8 * 3600") it seems to be silently ignored by the userId module. Maybe a log warning/error should be issued?

We have a correct value in our examples, also we reached out to our partners that have incorrect configuration, but we still see a steady stream of incorrect values.

Steps to reproduce

Use a userId module with a string refreshInSeconds:

pbjs.setConfig({
  userSync: {
    userIds: [{
      name: 'id5Id',
      params: {
        partner: 173,           
      },
      storage: {
        type: 'html5',         
        name: 'id5id',     
        expires: 90,            
        refreshInSeconds: "7200"   // some partners use the string value for refreshInSeconds for some reason
      }
    }],
    auctionDelay: 50           
  }
});