From 5090c07846d5f88feb6ca7e5bd28ae115d861e83 Mon Sep 17 00:00:00 2001 From: Moxie Marlinspike Date: Fri, 4 Jan 2019 00:18:13 -0800 Subject: [PATCH] Update constraints on account attributes name --- .../textsecuregcm/entities/AccountAttributes.java | 2 +- .../textsecuregcm/tests/controllers/DeviceControllerTest.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/whispersystems/textsecuregcm/entities/AccountAttributes.java b/src/main/java/org/whispersystems/textsecuregcm/entities/AccountAttributes.java index 4f5911fe7..c104c472c 100644 --- a/src/main/java/org/whispersystems/textsecuregcm/entities/AccountAttributes.java +++ b/src/main/java/org/whispersystems/textsecuregcm/entities/AccountAttributes.java @@ -33,7 +33,7 @@ public class AccountAttributes { private int registrationId; @JsonProperty - @Length(max = 50, message = "This field must be less than 50 characters") + @Length(max = 204, message = "This field must be less than 50 characters") private String name; @JsonProperty diff --git a/src/test/java/org/whispersystems/textsecuregcm/tests/controllers/DeviceControllerTest.java b/src/test/java/org/whispersystems/textsecuregcm/tests/controllers/DeviceControllerTest.java index 02155987a..ce5bf7d30 100644 --- a/src/test/java/org/whispersystems/textsecuregcm/tests/controllers/DeviceControllerTest.java +++ b/src/test/java/org/whispersystems/textsecuregcm/tests/controllers/DeviceControllerTest.java @@ -195,7 +195,7 @@ public class DeviceControllerTest { .target("/v1/devices/5678901") .request() .header("Authorization", AuthHelper.getAuthHeader(AuthHelper.VALID_NUMBER, "password1")) - .put(Entity.entity(new AccountAttributes("keykeykeykey", false, 1234, "this is a really long name that is longer than 80 characters", true, true, null), + .put(Entity.entity(new AccountAttributes("keykeykeykey", false, 1234, "this is a really long name that is longer than 80 characters it's so long that it's even longer than 204 characters. that's a lot of characters. we're talking lots and lots and lots of characters. 12345678", true, true, null), MediaType.APPLICATION_JSON_TYPE)); assertEquals(response.getStatus(), 422);