ICEfaces
  1. ICEfaces
  2. ICE-8270

Add configuration parameter to disable URL encoding

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P04
          • Fix Version/s: EE-1.8.2.GA_P05
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEfaces EE 1.8.2 P04, Seam
          • Assignee Priority:
            P1
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

            Description

            A customer had found an issue where they are adding in request parameters to the URL for a redirect. Sometimes these request parameters contain characters that should be encoded (colons, pluses, etc). When doing the redirect, Seam will properly encode these characters. But, Icefaces' BridgeExternalContext will mangle these encoded params.

            This feature request is to add a parameter that would disable the URL encoding we are doing. A parameter such as:

            com.icesoft.seamRedirectEncoding (default false)

            that will pass the URL through unencoded. They can then set this in their application and others will be unaffected.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case [5007000000MHjy6]
              Ken Fyten made changes -
              Fix Version/s EE-1.8.2.GA_P05 [ 10331 ]
              Assignee Priority P1
              Assignee Ted Goddard [ ted.goddard ]
              Ken Fyten made changes -
              Affects [Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration]
              Ted Goddard made changes -
              Assignee Ted Goddard [ ted.goddard ] Jack van Ooststroom [ jack.van.ooststroom ]
              Hide
              Permalink
              Ted Goddard added a comment -

              com.icesoft.encodeRedirectURLs

              Show
              Ted Goddard added a comment - com.icesoft.encodeRedirectURLs
              Hide
              Permalink
              Ted Goddard added a comment -

              default is true for the above parameter name.

              Show
              Ted Goddard added a comment - default is true for the above parameter name.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #31357 Tue Oct 09 09:55:43 MDT 2012 jack.van.ooststroom Fixed JIRA ICE-8270 : Add configuration parameter to disable URL encoding
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/BridgeExternalContext.java
              Hide
              Permalink
              Jack Van Ooststroom added a comment - - edited

              Sending core/src/com/icesoft/faces/context/BridgeExternalContext.java
              Transmitting file data .
              Committed revision 31357.

              Show
              Jack Van Ooststroom added a comment - - edited Sending core/src/com/icesoft/faces/context/BridgeExternalContext.java Transmitting file data . Committed revision 31357.
              Hide
              Permalink
              Jack Van Ooststroom added a comment - - edited

              Introduced the proposed com.icesoft.encodeRedirectURLs context parameter. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - - edited Introduced the proposed com.icesoft.encodeRedirectURLs context parameter. Marking this one as FIXED.
              Migration made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Jack Van Ooststroom
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: