diff options
author | Jesse Morgan <jesse@jesterpm.net> | 2014-04-08 23:04:40 -0700 |
---|---|---|
committer | Jesse Morgan <jesse@jesterpm.net> | 2014-04-08 23:04:40 -0700 |
commit | 45525336cbf2f648016682a1247c548631f91bdd (patch) | |
tree | 2b772dd68120feceefd35774e64ab13a6ee762cd /src/com/p4square/grow/frontend/JsonRequestProvider.java | |
parent | 38c12cf70ef4714a7fc508f7fbaf44487ea971b7 (diff) |
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.
Diffstat (limited to 'src/com/p4square/grow/frontend/JsonRequestProvider.java')
-rw-r--r-- | src/com/p4square/grow/frontend/JsonRequestProvider.java | 6 |
1 files changed, 5 insertions, 1 deletions
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<V> extends JsonEncodedProvider<V> 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()); |