Details

    • New Feature
    • Resolution: Fixed
    • Minor
    • None
    • Professional Edition
    • Alerting

    Description

      The alerting include attached has been tested to work with Clickatel. It is based on the Messagebird script in the current Professional Edition.

      When configured in the interface as such:
      recipient::<<FULL_NUMBER>>||originator::<<NUMBER/NAME>>||user::<<USERNAME>>||password::<<PASSWORD>>||apiid::<<API_ID>>

      The update to includes/definitions/transports.inc.php is:
      $config['alerts']['transports']['clickatell'] = array(
      'name' => 'Clickatell SMS',
      'parameters' => array(
      'originator' => 'Sender of SMS message',
      'apiid' => 'API access key to send SMS',
      'username' => 'API username',
      'password' => 'API password',
      'recipient' => "Recipient's phone number",
      )
      );

      Attachments

        Activity

          [OBS-1596] Clickatel Alerting (SMS Gateway)

          As the transport file is just include()'d as-is, that won't work - there will be no way to get the data out. The definitions file will be used to create a better web interface which prompts you with separate fields instead of having to create the awkward endpoint format yourself.

          Getting some things self contained is being thought about but currently it's not possible.

          sid3windr Tom Laermans added a comment - As the transport file is just include()'d as-is, that won't work - there will be no way to get the data out. The definitions file will be used to create a better web interface which prompts you with separate fields instead of having to create the awkward endpoint format yourself. Getting some things self contained is being thought about but currently it's not possible.
          troykelly Troy Kelly added a comment -

          Appreciate it Tom.

          I'm sure this has been discussed somewhere else - but would it be possible to define the required definition used by "includes/definitions/transports.inc.php" in the includes/alerting file?

          It would allow alerting endpoints to be fully self contained plugins effectively. And removes the reliance on editing the transports file.

          troykelly Troy Kelly added a comment - Appreciate it Tom. I'm sure this has been discussed somewhere else - but would it be possible to define the required definition used by "includes/definitions/transports.inc.php" in the includes/alerting file? It would allow alerting endpoints to be fully self contained plugins effectively. And removes the reliance on editing the transports file.

          Committed in r7233.

          sid3windr Tom Laermans added a comment - Committed in r7233.

          Sorry I was a bit slow in responding, but I see you got it figured out

          Looking good. A few nits were picked, I committed this to the tree. Easiest is to delete your local clickatell.inc.php then run svn, I think.

          Thanks!

          sid3windr Tom Laermans added a comment - Sorry I was a bit slow in responding, but I see you got it figured out Looking good. A few nits were picked, I committed this to the tree. Easiest is to delete your local clickatell.inc.php then run svn, I think. Thanks!
          troykelly Troy Kelly added a comment -

          Using get_http_request now.

          troykelly Troy Kelly added a comment - Using get_http_request now.

          Hi Troy,

          Why did you use curl instead of our functionality to do HTTP requests? We have proxy support and all sorts of other things using our, so transports should also use do_htto_request() so things don't fail unexpectedly.

          The top 5 lines of your file are missing by the way.

          Thanks!

          sid3windr Tom Laermans added a comment - Hi Troy, Why did you use curl instead of our functionality to do HTTP requests? We have proxy support and all sorts of other things using our, so transports should also use do_htto_request() so things don't fail unexpectedly. The top 5 lines of your file are missing by the way. Thanks!

          People

            sid3windr Tom Laermans
            troykelly Troy Kelly
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: