Bug 1393655

Summary: HTML character codes while accessing the vm/templates page under a folder which has '/' in name
Product: Red Hat CloudForms Management Engine Reporter: Aziza Karol <akarol>
Component: ProvidersAssignee: Adam Grare <agrare>
Status: CLOSED ERRATA QA Contact: Nandini Chandra <nachandr>
Severity: low Docs Contact:
Priority: medium    
Version: 5.7.0CC: akarol, cpelland, gblomqui, hkataria, jfrey, jhardy, mmojzis, mpovolny, nachandr, obarenbo, simaishi
Target Milestone: GA   
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:vm
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-03-01 13:07:41 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Attachments:
Description Flags
snapshot
none
screenshot none

Description Aziza Karol 2016-11-10 05:53:26 UTC
Created attachment 1219177 [details]
snapshot

Description of problem:


Version-Release number of selected component (if applicable):
5.7.0.10-beta3.20161109111947_9a61b18 

How reproducible:
100%

Steps to Reproduce:
see attached screenshot. clicking on folder name "Dev/Admin" in accordion displays "Dev %2f Admin" in the vm/templates page.

Actual results:


Expected results:


Additional info:

Comment 2 Dávid Halász 2017-03-09 09:42:54 UTC
Should be consistent in the latest build, could you please check it out? With using the same provider, I see in the database "Dev %2f Admin" and it shows the same in the treeview/title...

Comment 3 Aziza Karol 2017-03-14 10:36:02 UTC
David,

This issue is still reproducible in 5.8.0.4.20170307183144_745a3d9. See attached screenshot.

Thanks,
Aziza

Comment 4 Aziza Karol 2017-03-14 10:36:57 UTC
Created attachment 1262853 [details]
screenshot

Comment 5 Dávid Halász 2017-03-15 10:05:29 UTC
Checked in the DB: SELECT name FROM ems_folders WHERE id = XX;
It is stored as: "Dev %2f Admin"

Aziza verified for me that in the VmWare UI it is displayed as "Dev / Admin", i.e. the provider client writes the value into the DB wrong.

Greg, can I pass this to your team?

Comment 6 Greg Blomquist 2017-05-02 20:06:15 UTC
Adam, this sounds sort of familiar.  Can you take a peek at this and see if it's fixed?

Comment 7 Adam Grare 2017-05-02 20:10:53 UTC
Yeah this is probably the same issue as https://bugzilla.redhat.com/show_bug.cgi?id=1411527

Problem is that VMware returns it in this fashion from the API so we need to actively decode these.  We already do for a number of different objects, I'll need to add this for folders as well.

Comment 9 CFME Bot 2017-05-04 18:33:49 UTC
New commit detected on ManageIQ/manageiq-providers-vmware/master:
https://github.com/ManageIQ/manageiq-providers-vmware/commit/4cfc246f9b6bca7a91bd7b05d5bd8a596c089665

commit 4cfc246f9b6bca7a91bd7b05d5bd8a596c089665
Author:     Adam Grare <agrare>
AuthorDate: Wed May 3 14:16:37 2017 -0400
Commit:     Adam Grare <agrare>
CommitDate: Wed May 3 14:16:37 2017 -0400

    Decode slash in folder/datacenter names
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1393655

 app/models/manageiq/providers/vmware/infra_manager/refresh_parser.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comment 10 Adam Grare 2017-05-04 18:34:52 UTC
For testing this will now handle folders, datacenters, datastores, and clusters with '/' in the name.

Comment 11 Nandini Chandra 2017-10-11 16:20:27 UTC
Verified in 5.9.0.1

Comment 14 errata-xmlrpc 2018-03-01 13:07:41 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:0380