Permalink
Browse files

Javadocs corrections

  • Loading branch information...
1 parent 11162d4 commit b9974875ef23e79ac31514db60fee4843eb9aea2 @dsyer dsyer committed Dec 10, 2012
@@ -16,22 +16,21 @@
package org.springframework.security.oauth2.provider;
-import org.springframework.security.oauth2.common.exceptions.OAuth2Exception;
/**
- * A service that provides the details about an oauth 2client.
+ * A service that provides the details about an OAuth2 client.
*
* @author Ryan Heaton
*/
public interface ClientDetailsService {
/**
- * Load a client by the client id. This method must NOT return null.
+ * Load a client by the client id. This method must not return null.
*
* @param clientId The client id.
* @return The client details.
- * @throws OAuth2Exception If the client account is locked, expired, disabled, or for any other reason.
+ * @throws ClientRegistrationException If the client account is locked, expired, disabled, or for any other reason.
*/
- ClientDetails loadClientByClientId(String clientId) throws OAuth2Exception;
+ ClientDetails loadClientByClientId(String clientId) throws ClientRegistrationException;
}
@@ -39,8 +39,9 @@
import org.springframework.web.context.request.ServletWebRequest;
/**
- * Default implementation of {@link OAuth2ExceptionRenderer} that can render the exceptions as JSON. If the caller only
- * accepts a specific media type (like XML) then this implementation will not handle it.
+ * Default implementation of {@link OAuth2ExceptionRenderer} that can render the exceptions using message converters
+ * (just like regular Spring MVC endpoints). If the caller sends an appropriate Accept header he should get the right
+ * result as long as an appropriate message converter is provided.
*
* @author Dave Syer
*

0 comments on commit b997487

Please sign in to comment.