Bug 793246 (JBEPP-331)

Summary: Logo portlet on Dashboard - url validation
Product: [JBoss] JBoss Enterprise Portal Platform 5 Reporter: Michal Vanco <mvanco>
Component: PortalAssignee: claprun <claprun>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 5.0.0.CR01CC: epp-bugs, jmorgan, mvecera, theute
Target Milestone: ---   
Target Release: 5.2.1.GA   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/JBEPP-331
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
URLs provided to the logo portlets were improperly validated, excluding absolute URLs when they should have been allowed. This caused an error with the default URL. The URL validation scheme has been modified to allow absolute URLs, which corrects the issue.
Story Points: ---
Clone Of: Environment:
EPP5 CR1
Last Closed: 2012-02-02 20:04:55 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:
Embargoed:

Description Michal Vanco 2010-04-29 14:18:24 UTC
project_key: JBEPP

I wanted to change logo image in logo portlet and put back original and found out following problem:
 - click Dashboard
 - Edit Layout
 - edit Logo portlet
 - do not change anything (logo has relative url)
 - Save -> Information message: The field "logoUrl" must match the format "URL". 

But default is "/eXoResources/skin/sharedImages/DashboardIcon.png" which do not correspond URL pattern.

Should be relative paths permitted or not?

Comment 2 Jared MORGAN 2011-11-17 23:47:26 UTC
Release Notes Docs Status: Added: Not Required


Comment 3 Thomas Heute 2012-01-12 09:25:00 UTC
Labels: Added: EPP_5_2_1_Candidate


Comment 4 Chris Laprun 2012-02-02 20:04:55 UTC
Needs to be more thoroughly tested…

Comment 5 Chris Laprun 2012-02-02 20:04:55 UTC
Release Notes Docs Status: Removed: Not Required Added: Not Yet Documented
Release Notes Text: Added: URLs provided to the logo portlets were improperly validated, excluding absolute URLs when they should have been allowed, resulting in an error with the default URL. The URL validation scheme has been modified to allow absolute URLs, thus resolving the issue.


Comment 6 Thomas Heute 2012-02-09 10:08:41 UTC
Labels: Removed: EPP_5_2_1_Candidate 


Comment 7 Jared MORGAN 2012-03-22 03:10:11 UTC
    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:
URLs provided to the logo portlets were improperly
validated, excluding absolute URLs when they should have been allowed. This caused an error with the default URL. The URL validation scheme has been modified to allow absolute URLs, which corrects the issue.