Change Details

Fixes regression introduced in D8887. This is a solution for: [ENG-4926](https://linear.app/comm/issue/ENG-4926/client-backup-upload-issue). At the time code gets to run, the outer function `generateRandomString` may have already returned. This means that `size` might have been destroyed, and there is undefined reference. That's why this function generates backup ID with length 0 and as an effect cause `Argon2` to throw `salt too short`. On dev everything works, this happens only on dev when code is more optimized. Also added casting to correct type.

Fixes regression introduced in D8887. This is a solution for [ENG-4926](https://linear.app/comm/issue/ENG-4926/client-backup-upload-issue). When the code gets to run, the outer function `generateRandomString` may have already returned. This means that `size` might have been destroyed, and there is an undefined reference. That's why this function generates `backupID` with length 0 and as an effect causes `Argon2` to throw `salt too short`. On dev everything worked, everything happens only on prod when code is more optimized. I also added casting to the correct type.

Fixes regression introduced in D8887. This is a solution for: [ENG-4926](https://linear.app/comm/issue/ENG-4926/client-backup-upload-issue). At the timeWhen the code gets to run, the outer function `generateRandomString` may have already returned. This means that `size` might have been destroyed, and there is an undefined reference. That's why this function generates `backup IDID` with length 0 and as an effect causes `Argon2` to throw `salt too short`. On dev everything worksed, thiseverything happens only on devprod when code is more optimized. AI also added casting to the correct type.