Uploaded image for project: 'OpenNMS'
  1. OpenNMS
  2. NMS-3322

XMPPNotificationStrategy (or JavaMailNotificationStrategy) does not utilise the "Numeric Message" field -nm

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.6.4
    • Fix Version/s: 1.9.90
    • Security Level: Default (Default Security Scheme)
    • Labels:
      None
    • Environment:
      Operating System: Linux
      Platform: PC
    • Bugzilla Id:
      3277

      Description

      There are messaging systems used in Notifcations that can handle richer content than others and you would like to get for instance formatted emails with links back to events or alarms and SMS messages that have a 160 char limit where you need a terse message. You can choose the field to extract the message from when using a Custom NotificationCommmand, like calling a shell script. The predefined Java Notification Strategies have fixed fields for checking for message content. However when you wish to get a rolling list of XMPP Notifications outputted to a large screen in your NOC, you'd like to get short messages as in SMS
      Could the predfined Java Notification Strategies such as XMPPNotificationStategy have the private method
      private String[] parseArgument(List<Argument> arguments)

      be amended to check for the numeric message switch as well? i.e.

      private String[] parseArguments(List<Argument> arguments) throws Exception {
      String[] parsedArgs = new String[XMPP_MAX];
      for (int i = 0; i < arguments.size(); i++) {
      Argument arg = arguments.get;
      if (NotificationManager.PARAM_XMPP_ADDRESS.equals(arg.getSwitch()))

      { parsedArgs[XMPP_TO] = arg.getValue(); }

      else if (NotificationManager.PARAM_TEXT_MSG.equals(arg.getSwitch()))

      { parsedArgs[XMPP_MESSAGE] = arg.getValue(); } else if (NotificationManager.PARAM_NUM_MSG.equals(arg.getSwitch())) { parsedArgs[XMPP_MESSAGE] = arg.getValue(); }

      }

      Thanks
      Patrick

        Attachments

          Activity

            People

            • Assignee:
              seth Seth Leger
              Reporter:
              ptuite@airspeed.ie Patrick Tuite
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: