Wicket
  1. Wicket
  2. WICKET-4869

Wicket-Atmosphere track message length

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.0.0, 6.3.0
    • Fix Version/s: 6.4.0
    • Component/s: wicket-atmosphere
    • Labels:
      None
    • Environment:
      Linux 3.0.0-13-generic #22-Ubuntu, OpenJDK Runtime Environment (IcedTea6 1.11.5), Tomcat 7.0.28

      Description

      When activating a PerRequestBroadcastFilter, the methods annotated with @Subscribe are unable to contribute to the AjaxRequestTarget.

      At the Filter level, the messages are OK but the client receives an empty <ajax-response>.

      A quickstart is attached where target/WEB-INF/lib is provided because jquery.atmosphere.js has been modified in order to activate track message length.

      Feel free to ask for more info.

      Regards,

      Pierre Goupil

      1. quickstart.tar.gz
        8.74 MB
        Zala Goupil

        Activity

        Hide
        Zala Goupil added a comment -

        At the root of the quickstart, there's a tomcat-logs.txt file containing all messages from Tomcat, just in case.

        Show
        Zala Goupil added a comment - At the root of the quickstart, there's a tomcat-logs.txt file containing all messages from Tomcat, just in case.
        Hide
        Emond Papegaaij added a comment -

        I've just pushed some updates to wicket-atmosphere and also updated the quickstart at https://github.com/papegaaij/wicket-atmosphere-quickstart . With these changes, message length tracking is enabled and seems to work (I can push very large messages). Can you verify that this also works for you? Make sure you use the same filter configuration from the web.xml as in the quickstart.

        Show
        Emond Papegaaij added a comment - I've just pushed some updates to wicket-atmosphere and also updated the quickstart at https://github.com/papegaaij/wicket-atmosphere-quickstart . With these changes, message length tracking is enabled and seems to work (I can push very large messages). Can you verify that this also works for you? Make sure you use the same filter configuration from the web.xml as in the quickstart.
        Hide
        Zala Goupil added a comment -

        Good afternoon,

        Well done! Silly question: where is org.apache.wicket.atmosphere.TrackMessageSizeFilter from web.xml? I can't find it in the code of wicket-atmosphere.

        Plus, I can't see any change in the GitHub of Apache Wicket, is there anything that I should know?

        Regards,

        Pierre

        Show
        Zala Goupil added a comment - Good afternoon, Well done! Silly question: where is org.apache.wicket.atmosphere.TrackMessageSizeFilter from web.xml? I can't find it in the code of wicket-atmosphere. Plus, I can't see any change in the GitHub of Apache Wicket, is there anything that I should know? Regards, Pierre
        Hide
        Emond Papegaaij added a comment -

        The github repo is synced once a day as far as I know. You can use our public Apache git repository at https://git-wip-us.apache.org/repos/asf/wicket.git or wait a day.

        Show
        Emond Papegaaij added a comment - The github repo is synced once a day as far as I know. You can use our public Apache git repository at https://git-wip-us.apache.org/repos/asf/wicket.git or wait a day.
        Hide
        Zala Goupil added a comment -

        I've just tried it and... it works! Great work, thank you. You can close this ticket.

        Show
        Zala Goupil added a comment - I've just tried it and... it works! Great work, thank you. You can close this ticket.
        Hide
        Emond Papegaaij added a comment -

        Message length tracking is now enabled.

        Note All users upgrading to wicket-atmosphere 0.6 need to add the following configuration to the web.xml:

        <init-param>
        <param-name>org.atmosphere.cpr.broadcastFilterClasses</param-name>
        <param-value>org.apache.wicket.atmosphere.TrackMessageSizeFilter</param-value>
        </init-param>

        Show
        Emond Papegaaij added a comment - Message length tracking is now enabled. Note All users upgrading to wicket-atmosphere 0.6 need to add the following configuration to the web.xml: <init-param> <param-name>org.atmosphere.cpr.broadcastFilterClasses</param-name> <param-value>org.apache.wicket.atmosphere.TrackMessageSizeFilter</param-value> </init-param>

          People

          • Assignee:
            Emond Papegaaij
            Reporter:
            Zala Goupil
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development