Suggestion from Jack that the set of allowed redirect URLs be configurable.

Similar to PUSH-297, this configuration could be based on

public void addGroupMember(final String groupName, final String pushId, PushConfiguration config)

where the PushConfiguration contains cloudPush = "true".

The corresponding JavaScript API would be

ice.push.addGroupMember(group, pushId, options);

where options contains cloudPush = "true".

If cloudPush is not enabled for a given pushId at the time it is added to a group, that pushID will not be parked when the push connection goes offline.

Note that Cloud Push will now be disabled by default, so the parameter is only needed to enable Cloud Push.

Looks like this is doable for 4.0. However, my suggestion of "that the set of allowed redirect URLs be configurable" might need to be addressed in a separate JIRA. I'm not sure if my suggestion was related to this JIRA. It was related to avoid being directed to any (malicious) page through Cloud Push.

Sending core/src/main/java/org/icepush/Browser.java
Sending core/src/main/java/org/icepush/LocalPushGroupManager.java
Sending core/src/main/java/org/icepush/NoopPushGroupManager.java
Sending core/src/main/java/org/icepush/PushContext.java
Sending core/src/main/java/org/icepush/PushGroupManager.java
Sending core/src/main/java/org/icepush/PushID.java
Sending core/src/main/java/org/icepush/servlet/BrowserBoundServlet.java
Transmitting file data .......
Committed revision 42135.

Sending eps/src/main/java/com/icesoft/push/DynamicPushGroupManager.java
Sending eps/src/main/java/com/icesoft/push/LocalPushGroupManager.java
Sending eps/src/main/java/com/icesoft/push/RemotePushGroupManager.java
Adding eps/src/main/java/com/icesoft/push/SerializablePushConfiguration.java
Sending eps/src/main/java/com/icesoft/push/SerializablePushNotification.java
Sending eps/src/main/java/com/icesoft/push/messaging/PushMessageService.java
Transmitting file data ......
Committed revision 39198.

Sending src/main/java/com/icesoft/notify/client/NotificationClient.java
Sending src/main/java/com/icesoft/notify/common/data/NotificationAsAService.java
Sending src/main/java/com/icesoft/notify/common/data/NotificationMarshallerJSON.java
Sending src/main/java/com/icesoft/notify/common/data/NotificationUnmarshallerJSON.java
Sending src/main/java/com/icesoft/notify/common/data/PushConfiguration.java
Sending src/main/java/com/icesoft/notify/push/LocalPushGroupManager.java
Sending src/main/java/com/icesoft/notify/push/servlet/BrowserBoundServlet.java
Sending src/main/java/com/icesoft/notify/server/AbstractNotificationResourceRequestHandler.java
Sending src/main/java/com/icesoft/notify/server/NotificationContext.java
Sending src/main/java/com/icesoft/notify/server/NotificationService.java
Sending src/main/resources/com/icesoft/notify/common/data/notification-as-a-service.xsd
Sending src/main/test/com/icesoft/notify/NotificationClientTest.java
Transmitting file data ............
Committed revision 39199.

The new addGroupMember(String, String, PushConfiguration) method, and its related logic, has been added to ICEpush, ICEpush-EE, EPS, NaaS and the NotificationServiceClient. I also added two additional tests to the NaaS test-suite. Support at the JavaScript-level still needs to be added.

Sending core/src/main/javascript/application.js
Transmitting file data .
Committed revision 42148.

Added support to the ICEfaces JavaScript Bridge.

It does look like that first comment quoting Jack is not intended for this JIRA.

Sending src/main/java/com/icesoft/notify/push/PushID.java
Transmitting file data .
Committed revision 39300.

NaaS now adapts the new method accordingly.

Marking this one as FIXED.