Skip to content

Commit

Permalink
fix(core): update notifications device token validation length (#4657)
Browse files Browse the repository at this point in the history
  • Loading branch information
dolcalmi authored Nov 15, 2024
1 parent 23f78d1 commit 60f5af1
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions core/api/src/domain/users/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,15 @@ export const checkedToNonEmptyLanguage = (
}

export const checkedToDeviceToken = (token: string): DeviceToken | ValidationError => {
// token from firebase have a length of 163
const correctLength = 163
if (token.length !== correctLength) {
// iOS tokens: 64 chars (min)
// Android tokens: up to 200 chars (max)
// Firebase: 142 chars but it is variable
const minLength = 64
const maxLength = 200

if (token.length < minLength || token.length > maxLength) {
return new InvalidDeviceTokenError(
`wrong length, expected ${correctLength}, got ${token.length}`,
`Invalid token length. Expected between ${minLength}-${maxLength}, got ${token.length}`,
)
}

Expand Down

0 comments on commit 60f5af1

Please sign in to comment.