Author: johnh
Date: Fri Feb 12 01:04:46 2010
New Revision: 909196
URL: http://svn.apache.org/viewvc?rev=909196&view=rev
Log:
Clean up OAuthRequest URI parsing exception case.
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
URL:
http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java?rev=909196&r1=909195&r2=909196&view=diff
==============================================================================
---
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
(original)
+++
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
Fri Feb 12 01:04:46 2010
@@ -500,7 +500,13 @@
case URL_ONLY:
break;
case URL_AND_FORM_PARAMS:
- params.addAll(sanitize(OAuth.decodeForm(base.getPostBodyAsString())));
+ try {
+
params.addAll(sanitize(OAuth.decodeForm(base.getPostBodyAsString())));
+ } catch (IllegalArgumentException e) {
+ // Occurs if OAuth.decodeForm finds an invalid URL to decode.
+ throw
responseParams.oauthRequestException(OAuthError.INVALID_REQUEST,
+ "Could not decode body", e);
+ }
break;
case URL_AND_BODY_HASH:
try {
