From 699f97929abe8d83f18d0a002ce8e69ae5aadded Mon Sep 17 00:00:00 2001
From: Ben Alex <ben.alex@springsource.com>
Date: Wed, 1 Dec 2004 02:09:15 +0000
Subject: [PATCH] Fix bug where class should delegate to setDetails method -
 not set the details directly.

---
 .../ui/webapp/AuthenticationProcessingFilter.java             | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/org/acegisecurity/ui/webapp/AuthenticationProcessingFilter.java b/core/src/main/java/org/acegisecurity/ui/webapp/AuthenticationProcessingFilter.java
index 46cfc0b3a7..a4414585d6 100644
--- a/core/src/main/java/org/acegisecurity/ui/webapp/AuthenticationProcessingFilter.java
+++ b/core/src/main/java/org/acegisecurity/ui/webapp/AuthenticationProcessingFilter.java
@@ -77,7 +77,9 @@ public class AuthenticationProcessingFilter extends AbstractProcessingFilter {
 
         UsernamePasswordAuthenticationToken authRequest = new UsernamePasswordAuthenticationToken(username,
                 password);
-        authRequest.setDetails(request.getRemoteAddr());
+
+        // Allow subclasses to set the "details" property
+        setDetails(request, authRequest);
 
         return this.getAuthenticationManager().authenticate(authRequest);
     }