Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-4858

JSONInterceptor must not populate value stack directly

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 2.3.x, 2.5.x
    • Fix Version/s: 2.6
    • Component/s: Plugin - JSON
    • Flags:
      Important

      Description

      JSONInterceptor must populate value stack via OGNL to honor our exclusions!

      So according to my current quick view and understanding, I'm planning something like below which also resolves WW-3364: JSONInterceptor could be an extension of ModelDrivenInterceptor via some refactoring of both. From hereafter, JSONPopulator could be removed and JSONInterceptor must give population up to ParametersInterceptor. This simply could be achieved by something like pushing JSONInterceptor.root into top of stack and also moving posted json to http params (or if was impossible some refactoring of ParametersInterceptor).

      Please cast here any objections, votes or idea.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                yasser.zamani Yasser Zamani
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: