[seam-commits] Seam SVN: r9836 - trunk/src/main/org/jboss/seam/security/openid.

[seam-commits at lists.jboss.org]

Author: norman.richards at 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;
         }