ICEpush
  1. ICEpush
  2. PUSH-352

When WNS properties aren't set the WNS Notification Provider should be off

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-4.0.0.GA
    • Fix Version/s: EE-4.0.0.GA
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEfaces-EE, ICEpush-EE, EPS

      Description

      When the WNS-specific properties for the WNS Notification Provider haven't been set, the WNS Notification Provider should be off instead of on. In addition some of the WARNING messages should be lowered to FINE level.

        Activity

        Hide
        Jack Van Ooststroom added a comment -

        Sending core-ee/src/main/java/com/icesoft/icepush/WNSNotificationProvider.java
        Transmitting file data .
        Committed revision 41290.

        Show
        Jack Van Ooststroom added a comment - Sending core-ee/src/main/java/com/icesoft/icepush/WNSNotificationProvider.java Transmitting file data . Committed revision 41290.
        Hide
        Jack Van Ooststroom added a comment -

        When one or both of the required properties, namely com.icesoft.icepush.wns.packageSid and com.icesoft.icepush.wns.clientSecret, haven't been set the WNS Notification Provider is now off instead of off as in this scenario it will not be added as an Extension. In addition, the WARNING level messages for having not set these properties have been lowered to FINE level. Marking this one as FIXED.

        Show
        Jack Van Ooststroom added a comment - When one or both of the required properties, namely com.icesoft.icepush.wns.packageSid and com.icesoft.icepush.wns.clientSecret , haven't been set the WNS Notification Provider is now off instead of off as in this scenario it will not be added as an Extension. In addition, the WARNING level messages for having not set these properties have been lowered to FINE level. Marking this one as FIXED.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: