History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: UP-2092
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Eric Dalquist
Reporter: Andrew Wills
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
uPortal

PortalHttpServletRequest Causes StackOverflow When RemoteUserPersonManager is Configured

Created: 17/Jun/08 05:23 PM   Updated: 15/Sep/08 06:46 PM
Component/s: Framework
Affects Version/s: 3.0.0, 3.0.1
Fix Version/s: 3.1.0-M1, 3.0.2

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown


 Description  « Hide
The following 3 sections of code combine to cause a StackOverflowError...

#####


// RemoteUserPersonManager.java @line 54...

        // If the user has authenticated with the server which has implemented web authentication,
        // the REMOTE_USER environment variable will be set.
        String remoteUser = request.getRemoteUser();

// PortalHttpServletRequest.java

    public String getRemoteUser() {
        final Principal userPrincipal = this.getUserPrincipal();
        if (userPrincipal == null) {
            return null;
        }
 
        return userPrincipal.getName();
    }
 
[...]

    public Principal getUserPrincipal() {
        final IPerson person = this.personManager.getPerson(this);
        if (person == null || person.isGuest()) {
            return null;
        }
       
        return person;
    }


 All   Comments   Work Log   Change History      Sort Order:
Eric Dalquist [18/Jun/08 11:12 AM]
Fixed issue by modifying PortalHttpServletRequest to pass the underlying request instead of itself to avoid possible recursive loops.