ICEpush
  1. ICEpush
  2. PUSH-31

Missing notification when handing connection control between windows

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha3
    • Fix Version/s: 2.0-Alpha3
    • Component/s: Samples
    • Labels:
      None
    • Environment:
      ICEpush svn trunk - Tomcat 6

      Description

      From the original email...

      I have completed my initial implementation of the <icep:push> tag, but have run up against an oddity that I need you to look at.

      I have checked everything in, so you should be able to build it. Need to build the following in order:

      core
      integration/common/notify
      integration/jsp/core
      integration/jsp/samples/push

      URL is http://localhost:8080/icepush-jsp-push

      The application works as follows:

      1) You identify yourself with a nickname and enter the system at in.jsp
      2) You are initially IN, but have a button to get you OUT, which navigates you to out.jsp
      3) From out.jsp you have the option to get back in, navigating back to in.jsp.
      4) The IN and OUT lists are rendered in <icep:region> tags, and an <icep:push> tag on each page causes a push with each page navigation.

      This mostly works except I noticed an issue when you have multiple windows in the same session. If you toggle in/out from the window that is not maintaining the blocking connection, everything works fine. But if you submit from the window that is maintaining the blocking connection, the page load cause control to switch to the other window, but the notification associated with the page load never occurs. You can now toggle back and forth without issue in the non-controlling window, but the one notification has gone missing.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20621 Fri Feb 12 09:35:13 MST 2010 mircea.toma PUSH-31 Test for null, older cached bridge code won't send the 'ice.push.window' header.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/BlockingConnectionServer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20555 Wed Feb 10 11:57:48 MST 2010 mircea.toma PUSH-31 Re-send last notifications when blocking connection switches to a different window.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/connection.async.js
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/BlockingConnectionServer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20525 Tue Feb 09 09:52:31 MST 2010 mircea.toma PUSH-31 Improve notification related logging. Make sure that last outbound pushIDs are saved each time a new blocking connection is created.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/connection.async.js
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/application.js
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/BlockingConnectionServer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20523 Tue Feb 09 09:04:18 MST 2010 mircea.toma PUSH-31 Purge notifications for disposed IDs.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/application.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20522 Tue Feb 09 08:02:30 MST 2010 mircea.toma PUSH-31 Fix delistPushIDWithBrowser function to receive list of IDs instead of single ID. Rename functions to better state what parameters they are using.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/application.js

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Steve Maryka
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: