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 | 2ac76c7937ff462f99c5713ab99c6230a11cbcf5 (patch) | |
tree | d90fcd86c8a030fdf78f7c67d88120047e8b111b /src/com/p4square/grow/frontend/JsonRequestProvider.java | |
parent | dcb9d839c9a30adc833af51e5c0f5a0df8175ce2 (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()); |