GitHub user Parth-Brahmbhatt opened a pull request:
https://github.com/apache/incubator-storm/pull/242
STORM-439: Replace purl.js with jquery url plugin to support special cha...
...racters in topology names.
BugFix: Remove double decoding from the ui server, currently the decoding
happens as part of clout.core.clj and ui.core.clj. Removed the decoding from
ui.core.clj.
Encode all url components on server side. Added the encodedId as response
param to ensure the url from client side uses encoded component but the user
still sees unencoded value.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Parth-Brahmbhatt/incubator-storm STORM-439
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/242.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #242
----
commit c0eec64c74a97ff8a1ba28488a5c2775f4f81bfd
Author: Parth Brahmbhatt <[email protected]>
Date: 2014-08-26T04:12:50Z
STORM-439: Replace purl.js with jquery url plugin to support special
characters in topology names.
BugFix: Remove double decoding from the ui server, currently the decoding
happens as part of clout.core.clj and ui.core.clj. Removed the decoding from
ui.core.clj.
Encode all url components on server side. Added the encodedId as response
param to ensure the url from client side uses encoded component but the user
still sees unencoded value.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
