ICEpush
  1. ICEpush
  2. PUSH-65

ICEpush Portal support

          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

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Ted Goddard added a comment -

              Assigning to Ken to re-assign.

              Show
              Ted Goddard added a comment - Assigning to Ken to re-assign.
              Hide
              Permalink
              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.
              Hide
              Permalink
              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.
              Hide
              Permalink
              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 .

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: