Bug 818565
| Summary: | Portal container name is hardcoded in the LocalizationFilter | ||
|---|---|---|---|
| Product: | [JBoss] JBoss Enterprise Portal Platform 5 | Reporter: | Thomas Heute <theute> |
| Component: | Portal | Assignee: | Nobody <nobody> |
| Status: | VERIFIED --- | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 5.2.1.GA | CC: | epp-bugs, vramik |
| Target Milestone: | --- | ||
| Target Release: | 5.2.2.ER01 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: |
A hard-coded portal container name in the LocalizationFilter caused a NPE (NULL container) if the default context path was changed. The fix introduces changes to the way the portal container name is parsed by the LocalizationFilter, which corrects the issue.
|
Story Points: | --- |
| Clone Of: | Environment: | ||
| Last Closed: | Type: | Bug | |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
Technical note added. If any revisions are required, please edit the "Technical Notes" field
accordingly. All revisions will be proofread by the Engineering Content Services team.
New Contents:
Cause: Portal container name is hardcoded in the LocalizationFilter
Consequence: We will get the exception because NULL container
Fix: Change code
Result: No NULL exception
Verified again at 5.2.2 CR01
Technical note updated. If any revisions are required, please edit the "Technical Notes" field
accordingly. All revisions will be proofread by the Engineering Content Services team.
Diffed Contents:
@@ -1,4 +1 @@
-Cause: Portal container name is hardcoded in the LocalizationFilter
+A hard-coded portal container name in the LocalizationFilter caused a NPE (NULL container) if the default context path was changed. The fix introduces changes to the way the portal container name is parsed by the LocalizationFilter, which corrects the issue.-Consequence: We will get the exception because NULL container
-Fix: Change code
-Result: No NULL exception
|
In LocalizationFilter.java, we hardcode "portal" for getting RootContainer: if (container instanceof RootContainer) container = (ExoContainer) container.getComponentInstance("portal"); This affect if we change default context path as article: http://community.jboss.org/wiki/ChangeGateInContextPath. We will get the exception because NULL container. The code should be change to: if (container instanceof RootContainer) container = (ExoContainer) container.getComponentInstance(req.getContextPath().substring(1));