ICEpush
  1. ICEpush
  2. PUSH-29

Registered javascript callbacks are continually retried if an exception is thrown in the callback (Client-side Exception-triggered Push Storm)

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 2.0.0
          • Component/s: JavaScript Client
          • Labels:
            None
          • Environment:
            n/a

            Description

            If a registered javascript callback function throws an exception the ICEpush bridge will continue to retry calling that function. This can result in a storm of activity and crash the browser.

              Activity

              Ascending order - Click to sort in descending order
              Philip Breau created issue -
              Philip Breau made changes -
              Field Original Value New Value
              Summary Registered javascript callbacks are continually retried in they throw an exception Registered javascript callbacks are continually retried if an exception is thrown in the callback
              Hide
              Permalink
              Philip Breau added a comment -

              test case instructions:

              • upack in /integration/jquery/samples
              • build war target
              • open <server:port>/push-29/index.html
              • click 'Listen to Groups' button
              • create a new group
              • note storm of requests caused by the callback listener continually recalled when it throws an exception
              Show
              Philip Breau added a comment - test case instructions: upack in /integration/jquery/samples build war target open <server:port>/push-29/index.html click 'Listen to Groups' button create a new group note storm of requests caused by the callback listener continually recalled when it throws an exception
              Philip Breau made changes -
              Attachment push-29.zip [ 12190 ]
              Philip Breau made changes -
              Summary Registered javascript callbacks are continually retried if an exception is thrown in the callback Registered javascript callbacks are continually retried if an exception is thrown in the callback (Client-side Excpetion-triggered Push Storm)
              Philip Breau made changes -
              Summary Registered javascript callbacks are continually retried if an exception is thrown in the callback (Client-side Excpetion-triggered Push Storm) Registered javascript callbacks are continually retried if an exception is thrown in the callback (Client-side Exception-triggered Push Storm)
              Mircea Toma made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #20492 Thu Feb 04 17:54:14 MST 2010 mircea.toma PUSH-29 Stop callback exceptions from propagating. Log callback exceptions. Test if callback is of function type.
              Files Changed
              Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/application.js
              Hide
              Permalink
              Mircea Toma added a comment -

              Stop callback exceptions from propagating. Log callback exceptions. Test if callback is of function type.

              Show
              Mircea Toma added a comment - Stop callback exceptions from propagating. Log callback exceptions. Test if callback is of function type.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 2.0.0 [ 10233 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Philip Breau
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: