error occurred when swift Pseudo-folder include "="

[liyuanjian]

when swift Pseudo-folder include "=" (this kind of usage often use in hive partition), hirizon link will transfer "=" to "%253D", for example, my Pseudo-folder name is "A=B", if I want enter "A=B", actually enter "A%253DB" through horizon

[Rajasi Kulkarni (rajasi-kulkarni)]

This is happening because Horizon is using Django version 1.6.As explained in bug #1231904, in earlier Django version the reverse() did not urlquote parameters. Now with Django 1.6 , the reverse() function can handle the issue of special characters in urls.

Currently when a Pseudo-folder with name ‘A=B’ is created , the folder name is first getting encoded to ‘A%3DB’ because of the code which is provided as fix for Bug #1231904 and then Django again encodes the ‘%’ sign to ‘%25’ .Hence the Horizon link converts it to ‘A=%253DB’. This is the issue with any special character in container name or Pseudo folder name.

It is specified in bug #1231904 we will have to remove all urlquote when Horizon will use Django 1.6. To fix the issue I will remove urlquote() wherever unneccessary.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/101161

[OpenStack Infra (hudson-openstack)]

Change abandoned by Rajasi Kulkarni (<email address hidden>) on branch: master
Review: https://review.openstack.org/101161
Reason: Facing merge conflict issues while uploading code for patch set 2. Hence abandoning this change and will upload the updated code.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/104121

[Romain Hardouin (romain-hardouin)]

Rajasi,

Following the discussion we've had by email, here is the situation:
George Peristerakis is working on a patch that will work with Django 1.6 as well as older versions.
It does not work with older versions *right now* (even if unit tests pass) but I talked to him today and he will make it work.
So, maybe we could flag this ticket as duplicate? (even if it older than the other one?)

Bug: https://bugs.launchpad.net/horizon/+bug/1347734
Patch: https://review.openstack.org/#/c/109009/

[Rajasi Kulkarni (rajasi-kulkarni)]

Can we hold on for marking this ticket as duplicate. After the bug #1347734 is fixed, then we can mark this as duplicate.

[OpenStack Infra (hudson-openstack)]

Change abandoned by David Lyle (<email address hidden>) on branch: master
Review: https://review.openstack.org/104121
Reason: No updates since July. If this is still being worked on restore the review and rebase.