From 628227f5e71729d1a85e8d1f4bab70cbc77c1338 Mon Sep 17 00:00:00 2001
From: Luke Taylor <luke.taylor@springsource.com>
Date: Thu, 6 Dec 2007 16:53:35 +0000
Subject: [PATCH] Corrected out of date comment (constructor doesn't create a
 session). Removed unnecessary default constructor.

---
 .../security/ui/WebAuthenticationDetails.java          | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/core/src/main/java/org/springframework/security/ui/WebAuthenticationDetails.java b/core/src/main/java/org/springframework/security/ui/WebAuthenticationDetails.java
index 5ac2120046..eaa8e2647f 100644
--- a/core/src/main/java/org/springframework/security/ui/WebAuthenticationDetails.java
+++ b/core/src/main/java/org/springframework/security/ui/WebAuthenticationDetails.java
@@ -38,10 +38,8 @@ public class WebAuthenticationDetails implements SessionIdentifierAware, Seriali
     //~ Constructors ===================================================================================================
 
     /**
-     * NB: This constructor will cause a <code>HttpSession</code> to be created
-     * (this is considered reasonable as all Spring Security authentication
-     * requests rely on <code>HttpSession</code> to store the
-     * <code>Authentication</code> between requests.
+     * Records the remote address and will also set the session Id if a session
+     * already exists (it won't create one).
      *
      * @param request that the authentication request was received from
      */
@@ -54,10 +52,6 @@ public class WebAuthenticationDetails implements SessionIdentifierAware, Seriali
         doPopulateAdditionalInformation(request);
     }
 
-    protected WebAuthenticationDetails() {
-        throw new IllegalArgumentException("Cannot use default constructor");
-    }
-
     //~ Methods ========================================================================================================
 
     /**