Merge pull request #17 from RobotsAndPencils/devicetoken

allow larger device tokens
RobotsAndPencils · Feb 5, 2016 · 24a0dd9 · 24a0dd9
2 parents 10c0bce + 4c5faef
commit 24a0dd9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/push/device_token.go b/push/device_token.go
@@ -5,7 +5,7 @@ import "encoding/hex"
 // IsDeviceTokenValid checks if s is a hexadecimal token of the correct length.
 func IsDeviceTokenValid(s string) bool {
  // TODO: In 2016, they may be growing up to 100 bytes (200 hexadecimal digits).
- if len(s) != 64 {
+ if len(s) < 64 || len(s) > 200 {
  return false
  }
  _, err := hex.DecodeString(s)

diff --git a/push/device_token_test.go b/push/device_token_test.go
@@ -25,6 +25,7 @@ func TestInvalidDeviceTokens(t *testing.T) {
 func TestValidDeviceToken(t *testing.T) {
  tokens := []string{
  "c2732227a1d8021cfaf781d71fb2f908c61f5861079a00954a5453f1d0281433",
+ "c2732227a1d8021cfaf781d71fb2f908c61f5861079a00954ac2732227a1d8021cfaf781d71fb2f908c61f5861079a00954ac2732227a1d8021cfaf781d71fb2f908c61f5861079a00954ac2732227a1d8021cfaf781d71fb2f908c61f5861079a00954a",
  }
 
  for _, token := range tokens {