ICEfaces
  1. ICEfaces
  2. ICE-8899

Parameters containing “rvn” lost on redirect

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P04
    • Fix Version/s: EE-1.8.2.GA_P06
    • Component/s: Bridge, Framework
    • Labels:
      None
    • Environment:
      Seam
    • Assignee Priority:
      P1
    • Salesforce Case Reference:

      Description

      Given the following redirect rule specified in pages.xml:
      <redirect view-id="/xhtml/loginator.xhtml">
      <param name="t1" value="hellorvnworld" />
      <param name="t2" value="RkqF3RW4QZzhF0JrlznFr6d99LOIuMW52h3Vp1dPH8R0007XuIsIgoXjaSCuGD6OIOvECogMjt8=" />
      <param name="t3" value="0000000rvn11111111111" />
      <param name="t4" value="rvn" />
      <param name="t5" value="RVN" />
      <param name="t6" value="RkqF3RW4QZzhF0JrlznFr6d99LOIuMW52h3Vp1dPH8Rvn7XuIsIgoXjaSCuGD6OIOvECogMjt8=" />
      </redirect>

      In this example, a parameter with name “t1” gets lost because its value is “hellorvnworld”, which contains “rvn” as a substring. Parameters t1, t3, and t4 will be null when trying to read the http request parameter after redirect.

      This appears to be due to the following code at com.icesoft.util.SeamUtilities:194:
      while(st.hasMoreTokens() ){
      token = st.nextToken();
      if ( (token.indexOf(conversationIdParameter) == -1) &&
      (token.indexOf(conversationParentParameter) == -1) &&
      token.indexOf("rvn") == -1 ) {

      tokenList.add( token );
      }
      }

      which indiscriminately discards request parameters if “rvn” is found anywhere in the value.

      Note, this will also apply to the string “cid” when used as the conversationIdParameter.

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case Reference 5007000000QYryIAAT
        Ken Fyten made changes -
        Assignee Deryk Sinotte [ deryk.sinotte ]
        Fix Version/s EE-1.8.2.GA_P06 [ 10470 ]
        Affects Version/s EE-1.8.2.GA_P05 [ 10331 ]
        Assignee Priority P2 [ 10011 ]
        Ken Fyten made changes -
        Assignee Priority P2 [ 10011 ] P1 [ 10010 ]
        Deryk Sinotte made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Deryk Sinotte
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: