servlet-spec
  1. servlet-spec
  2. SERVLET_SPEC-19

Add forward and include events to RequestListener

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Activity

      Hide
      Rajiv Mordani added a comment -

      Are you looking for requestForwarded(ServletRequestEvent sre) and requestIncluded(ServletRequestEvent sre)? Sounds reasonable. Are there any other such events needed?

      Show
      Rajiv Mordani added a comment - Are you looking for requestForwarded(ServletRequestEvent sre) and requestIncluded(ServletRequestEvent sre)? Sounds reasonable. Are there any other such events needed?
      Hide
      markt_asf added a comment -

      Yes, that is what I was looking for.

      This is the Tomcat bug report that triggered this request:
      https://issues.apache.org/bugzilla/show_bug.cgi?id=50789
      also
      https://issues.apache.org/bugzilla/show_bug.cgi?id=49991

      It might be worth checking with the CDI folks if this is sufficient or if there is anything else they'd like.

      Show
      markt_asf added a comment - Yes, that is what I was looking for. This is the Tomcat bug report that triggered this request: https://issues.apache.org/bugzilla/show_bug.cgi?id=50789 also https://issues.apache.org/bugzilla/show_bug.cgi?id=49991 It might be worth checking with the CDI folks if this is sufficient or if there is anything else they'd like.
      Hide
      Rajiv Mordani added a comment -

      As discussed in the EG, this would break compatibility with existing listeners. Also it isn't clear to me why CDI needs these events. For now I am not going to add these events to a new listener either.

      Show
      Rajiv Mordani added a comment - As discussed in the EG, this would break compatibility with existing listeners. Also it isn't clear to me why CDI needs these events. For now I am not going to add these events to a new listener either.
      Hide
      markt_asf added a comment -

      I'm fine with that.

      If the CDI folks need this then they need to speak up and make their case.

      Show
      markt_asf added a comment - I'm fine with that. If the CDI folks need this then they need to speak up and make their case.

        People

        • Assignee:
          Rajiv Mordani
          Reporter:
          markt_asf
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: