From 2ac76c7937ff462f99c5713ab99c6230a11cbcf5 Mon Sep 17 00:00:00 2001 From: Jesse Morgan Date: Tue, 8 Apr 2014 23:04:40 -0700 Subject: Fixing 500 on New Account Login Bug. AccountRedirectResource was expecting the user to have a UserRecord, but new users did not have one. This caused JsonRequestProvider to throw an IOException and AccountRedirectResource to return 500. JsonRequestProvider now throws a NotFoundException on 404 and AccountRedirectResource properly handles it. --- src/com/p4square/grow/frontend/JsonRequestProvider.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'src/com/p4square/grow/frontend/JsonRequestProvider.java') diff --git a/src/com/p4square/grow/frontend/JsonRequestProvider.java b/src/com/p4square/grow/frontend/JsonRequestProvider.java index a04294d..bf3b2b3 100644 --- a/src/com/p4square/grow/frontend/JsonRequestProvider.java +++ b/src/com/p4square/grow/frontend/JsonRequestProvider.java @@ -51,7 +51,11 @@ public class JsonRequestProvider extends JsonEncodedProvider implements Pr representation.release(); } - throw new IOException("Could not get object. " + response.getStatus()); + if (Status.CLIENT_ERROR_NOT_FOUND.equals(response.getStatus())) { + throw new NotFoundException("Could not get object. " + response.getStatus()); + } else { + throw new IOException("Could not get object. " + response.getStatus()); + } } return decode(representation.getText()); -- cgit v1.2.3