ICEpush
  1. ICEpush
  2. PUSH-81

Reconsider the uriPrefix and uriSuffix implementation (configuration.icepush)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha3
    • Fix Version/s: 2.0-Beta
    • Component/s: Push Library, Push Server
    • Labels:
      None
    • Environment:
      ICEpush
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)

      Description

      We should reconsider the uriPrefix and uriSuffix strategy which currently relates to the configuration.icepush request. With this implementation there seems to be a nasty cashing issue. We should consider switching to HTTP permanent redirect for instance and step away from the requirement of having the developer change the application by adding the script tag in order to get the configuration.icepush.

        Activity

        Jack Van Ooststroom created issue -
        Hide
        Jack Van Ooststroom added a comment -

        Assigning to Mircea for investigation.

        Show
        Jack Van Ooststroom added a comment - Assigning to Mircea for investigation.
        Jack Van Ooststroom made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21380 Mon May 10 08:21:03 MDT 2010 mircea.toma PUSH-81 Reorganize code to allow definition of private functions outside of AsyncConnection object.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/connection.async.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21381 Mon May 10 08:31:03 MDT 2010 mircea.toma PUSH-81 Define constants for cookie names.
        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
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21382 Mon May 10 08:36:20 MDT 2010 mircea.toma PUSH-81 Add logging message.
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/connection.async.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21465 Tue May 25 05:59:54 MDT 2010 mircea.toma PUSH-81 Seed the definition of dynamic configuration.
        Files Changed
        Commit graph ADD /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21466 Tue May 25 06:26:13 MDT 2010 mircea.toma PUSH-81 Fix attribute value lookup. Add configuration object unit test.
        Files Changed
        Commit graph ADD /projects/bridge-support/src/test/javascript/configuration.html
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21467 Tue May 25 06:52:40 MDT 2010 mircea.toma PUSH-81 Add multiple value lookup.
        Files Changed
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21468 Tue May 25 07:06:04 MDT 2010 mircea.toma PUSH-81 Use private lookupValues function for looking up raw values.
        Files Changed
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21469 Tue May 25 07:23:05 MDT 2010 mircea.toma PUSH-81 Fix lookupValues function. Add minimal test for valueAsStrings operator.
        Files Changed
        Commit graph MODIFY /projects/bridge-support/src/test/javascript/configuration.html
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21470 Tue May 25 07:24:26 MDT 2010 mircea.toma PUSH-81 Use defaultValues variable for test.
        Files Changed
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21478 Wed May 26 07:27:24 MDT 2010 mircea.toma PUSH-81 Re-configure bridge through the blocking connection. Create dynamic XML based configuration to make possibile the reconfiguration of the bridge at any time.
        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/servlet/BrowserBoundServlet.java
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/ConfigurationServer.java
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/command.js
        Commit graph MODIFY /scratchpads/snowplow/core/build.javascript.xml
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/application.js
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/servlet/MainServlet.java
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/oo.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21479 Wed May 26 07:59:32 MDT 2010 mircea.toma PUSH-81 Update test to use new function name.
        Files Changed
        Commit graph MODIFY /projects/bridge-support/src/test/javascript/configuration.html
        Hide
        Mircea Toma added a comment -

        Changed the bridge to make it possible to re-configure it through the blocking connection. Created custom configuration message/command sent by the server and captured by the bridge. First request after bridge loads is always marked so that server is informed that client might need to be reconfigured.
        Generate and unblock connection only when values in configuration differ from the default values. Changed build to include new dynamic construct used by the bridge.

        Show
        Mircea Toma added a comment - Changed the bridge to make it possible to re-configure it through the blocking connection. Created custom configuration message/command sent by the server and captured by the bridge. First request after bridge loads is always marked so that server is informed that client might need to be reconfigured. Generate and unblock connection only when values in configuration differ from the default values. Changed build to include new dynamic construct used by the bridge.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Mircea Toma added a comment - - edited

        The following configuration parameters were introduced:

        org.icepush.heartbeatTimeout (default value = 50000ms)
        When blocking connection is blocked for more time (defined in milliseconds) than this parameters specifies the bridge assumes that the connection is in trouble (and will retry to reconnect twice before giving up).

        org.icepush.serverErrorRetryTimeouts (default values = 1000ms 2000ms 4000ms)
        A list of space separated intervals (defined in milliseconds) used by the bridge to retry connecting to the server after a server side error is received.

        org.icepush.fileExtension (default value = "")
        This parameter will append the defined file extension to the request URIs used by the bridge to communicate with the server.

        org.icepush.contextPath (no default value)
        This parameter will force the bridge to use the defined context path when initiating the blocking request.

        Show
        Mircea Toma added a comment - - edited The following configuration parameters were introduced: org.icepush.heartbeatTimeout (default value = 50000ms) When blocking connection is blocked for more time (defined in milliseconds) than this parameters specifies the bridge assumes that the connection is in trouble (and will retry to reconnect twice before giving up). org.icepush.serverErrorRetryTimeouts (default values = 1000ms 2000ms 4000ms) A list of space separated intervals (defined in milliseconds) used by the bridge to retry connecting to the server after a server side error is received. org.icepush.fileExtension (default value = "") This parameter will append the defined file extension to the request URIs used by the bridge to communicate with the server. org.icepush.contextPath (no default value) This parameter will force the bridge to use the defined context path when initiating the blocking request.
        Mircea Toma made changes -
        Affects [Documentation (User Guide, Ref. Guide, etc.)]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21501 Thu May 27 08:36:34 MDT 2010 mircea.toma PUSH-81 Move private functions outside method definition closure.
        Files Changed
        Commit graph MODIFY /projects/bridge-support/src/main/javascript/configuration.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21502 Thu May 27 09:56:19 MDT 2010 mircea.toma PUSH-81 Reuse configuration object constructed in MainServlet.
        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/servlet/BrowserBoundServlet.java
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/ConfigurationServer.java
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/BlockingConnectionServer.java
        Ken Fyten made changes -
        Fix Version/s 2.0-Beta [ 10232 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Jack Van Ooststroom
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: