Skip to content

Commit

Permalink
remove "I" from verification to avoid ambiguity
Browse files Browse the repository at this point in the history
  • Loading branch information
kentcdodds committed Jun 10, 2024
1 parent 5fa0dd7 commit 7a4da05
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/routes/_auth+/verify.server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ export async function prepareVerification({

const { otp, ...verificationConfig } = generateTOTP({
algorithm: 'SHA256',
// Leaving off 0 and O on purpose to avoid confusing users.
charSet: 'ABCDEFGHIJKLMNPQRSTUVWXYZ123456789',
// Leaving off 0, O, and I on purpose to avoid confusing users.
charSet: 'ABCDEFGHJKLMNPQRSTUVWXYZ123456789',
period,
})
const verificationData = {
Expand Down

0 comments on commit 7a4da05

Please sign in to comment.