| Summary: | LoginRedirectFilter only detects redirection for hardcoded "/portal/sso" context | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [JBoss] JBoss Enterprise Portal Platform 5 | Reporter: | Tomas Kyjovsky <tkyjovsk> | ||||
| Component: | Portal, Site Publisher | Assignee: | mposolda | ||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | |||||
| Severity: | high | Docs Contact: | |||||
| Priority: | high | ||||||
| Version: | unspecified | CC: | epp-bugs, tkyjovsk | ||||
| Target Milestone: | --- | ||||||
| Target Release: | 5.2.0.ER06 | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| URL: | http://jira.jboss.org/jira/browse/JBEPP-988 | ||||||
| Whiteboard: | LoginRedirectFilter OpenSSO SSO SitePublisher sso-agent | ||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2011-11-09 16:10:53 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Attachments: |
|
||||||
I'd suggest either a) adding a new init-param for the LoginRedirectFilter which would customize request URI, or b) detecting the current portal container's context (which possibly could be done via the AbstractFilter.getContainer() method) I'm attaching a patched LoginRedirectFilter which implements option b) from the above comment and redirects logins from "/${portalContainerName}/sso" depending on the current portal container.
Attachment: Added: LoginRedirectFilter.java Labels: Removed: LoginRedirectFilter sso-agent Added: LoginRedirectFilter OpenSSO SSO SitePublisher sso-agent Release Notes Text: Added: Fixed in SSO 1.1.0-GA by commit rev. 7960 . Instead of "/portal/sso", we are using request.getContextPath() + "/sso" right now. Moved comment from RN Text field: Fixed in SSO 1.1.0-GA by commit rev. 7960 . Instead of "/portal/sso", we are using request.getContextPath() + "/sso" right now. Release Notes Docs Status: Added: Not Required Release Notes Text: Removed: Fixed in SSO 1.1.0-GA by commit rev. 7960 . Instead of "/portal/sso", we are using request.getContextPath() + "/sso" right now. |
project_key: JBEPP LoginRedirectFilter only detects login redirection for a hardcoded URL context "/portal/sso". This makes it not to work properly with other non-default portal containers, for example "ecmdemo" which is shipped with the SitePublisher extension. Project: org.gatein.sso:sso-agent Class: org.gatein.sso.agent.filter.LoginRedirectFilter ... private boolean isLoginInProgress(HttpServletRequest request) { String action = request.getRequestURI(); if (action != null && action.equals("/portal/sso")) { return true; } return false; } ...