Author: norman.richards(a)jboss.com Date: 2008-12-23 15:30:44 -0500 (Tue, 23 Dec 2008) New Revision: 9836 Modified: trunk/src/main/org/jboss/seam/security/openid/OpenId.java trunk/src/main/org/jboss/seam/security/openid/OpenIdPhaseListener.java Log: fix logging Modified: trunk/src/main/org/jboss/seam/security/openid/OpenId.java =================================================================== --- trunk/src/main/org/jboss/seam/security/openid/OpenId.java 2008-12-23 20:00:11 UTC (rev 9835) +++ trunk/src/main/org/jboss/seam/security/openid/OpenId.java 2008-12-23 20:30:44 UTC (rev 9836) @@ -8,6 +8,8 @@ import javax.faces.context.FacesContext; import javax.servlet.http.HttpServletRequest; +import org.jboss.seam.log.*; + import org.jboss.seam.ScopeType; import org.jboss.seam.annotations.Create; import org.jboss.seam.annotations.Install; @@ -35,6 +37,8 @@ public class OpenId implements Serializable { + private transient LogProvider log = Logging.getLogProvider(OpenId.class); + String id; String validatedId; @@ -113,7 +117,7 @@ return authReq.getDestinationUrl(true); } catch (OpenIDException e) { - e.printStackTrace(); + log.error(e); } return null; @@ -172,14 +176,12 @@ AuthSuccess authSuccess = (AuthSuccess) verification.getAuthResponse(); - System.out.println("*** EXT: " + authSuccess.getExtensions()); if (authSuccess.hasExtension(AxMessage.OPENID_NS_AX)) { FetchResponse fetchResp = (FetchResponse) authSuccess .getExtension(AxMessage.OPENID_NS_AX); List emails = fetchResp.getAttributeValues("email"); String email = (String) emails.get(0); - System.out.println("XXX email is " + email); } return verified.getIdentifier(); Modified: trunk/src/main/org/jboss/seam/security/openid/OpenIdPhaseListener.java =================================================================== --- trunk/src/main/org/jboss/seam/security/openid/OpenIdPhaseListener.java 2008-12-23 20:00:11 UTC (rev 9835) +++ trunk/src/main/org/jboss/seam/security/openid/OpenIdPhaseListener.java 2008-12-23 20:30:44 UTC (rev 9836) @@ -13,11 +13,13 @@ import org.jboss.seam.Component; import org.jboss.seam.navigation.Pages; +import org.jboss.seam.log.*; @SuppressWarnings("serial") public class OpenIdPhaseListener implements PhaseListener { + private transient LogProvider log = Logging.getLogProvider(OpenIdPhaseListener.class); @SuppressWarnings("unchecked") public void beforePhase(PhaseEvent event) @@ -33,8 +35,7 @@ try { sendXRDS(); } catch (IOException e) { - // TODO Auto-generated catch block - e.printStackTrace(); + log.error(e); } return; }