Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0-Beta
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEpush, Portal

      Description

      ICEpush should be supported in portal environments. This should include both core support and a Portal version of the chat sample application.

        Activity

        Ted Goddard created issue -
        Hide
        Ted Goddard added a comment -

        Assigning to Ken to re-assign.

        Show
        Ted Goddard added a comment - Assigning to Ken to re-assign.
        Ted Goddard made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Fix Version/s 1.0-Beta [ 10232 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22002 Thu Jul 22 12:51:59 MDT 2010 ted.goddard relocated icepush.js for direct access as JSF resource (PUSH-65)
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/CodeServer.java
        Commit graph MODIFY /scratchpads/snowplow/core/build.javascript.xml
        Ted Goddard made changes -
        Assignee Ken Fyten [ ken.fyten ] Ted Goddard [ ted.goddard ]
        Hide
        Ted Goddard added a comment -

        icepush.js is now in META-INF/resources/icepush.js so that it can be loaded directly by JSF. Note that this bypasses the following code in CodeServer.java:

        InputStream configuration = new ByteArrayInputStream(("ice.push.configuration.contextPath='" + context.getContextPath() + "';").getBytes("UTF-8"));

        response.writeBodyFrom(new SequenceInputStream(code, configuration));

        However, this approach should not be used to configure the push-server as it requires mixing static and dynamic code, preventing caching, preventing configuration after application startup, and complicating security analysis.

        Show
        Ted Goddard added a comment - icepush.js is now in META-INF/resources/icepush.js so that it can be loaded directly by JSF. Note that this bypasses the following code in CodeServer.java: InputStream configuration = new ByteArrayInputStream(("ice.push.configuration.contextPath='" + context.getContextPath() + "';").getBytes("UTF-8")); response.writeBodyFrom(new SequenceInputStream(code, configuration)); However, this approach should not be used to configure the push-server as it requires mixing static and dynamic code, preventing caching, preventing configuration after application startup, and complicating security analysis.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22014 Fri Jul 23 12:24:43 MDT 2010 ted.goddard check for full listen.icepush URL (PUSH-65)
        Files Changed
        Commit graph MODIFY /scratchpads/snowplow/core/src/main/javascript/connection.async.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22015 Fri Jul 23 12:54:14 MDT 2010 ted.goddard initial Portlet push support via proxied HTTP requests and full listen.icepush URI (PUSH-65)
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/push/servlet/ICEpushResourceHandler.java
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/event/BridgeSetup.java
        Commit graph ADD /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/push/servlet/ProxyHttpServletResponse.java
        Commit graph ADD /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/push/servlet/ProxyHttpServletRequest.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22016 Fri Jul 23 13:01:55 MDT 2010 ted.goddard initial portlet push support (PUSH-65)
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/lib/icepush.jar
        Hide
        Ted Goddard added a comment -

        Need to look at providing the configuration in a way that does not require modifying the icepush.js resource. This may already be done with the configuration request in icepush.js. To verify, we will test with ICEpush EPS with portlets.

        Show
        Ted Goddard added a comment - Need to look at providing the configuration in a way that does not require modifying the icepush.js resource. This may already be done with the configuration request in icepush.js. To verify, we will test with ICEpush EPS with portlets.
        Ted Goddard made changes -
        Assignee Ted Goddard [ ted.goddard ] Mircea Toma [ mircea.toma ]
        Hide
        Mircea Toma added a comment -

        Issue solved by the fixes for ICE-6425 .

        Show
        Mircea Toma added a comment - Issue solved by the fixes for ICE-6425 .
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: