lib/mailgun/webhooks/webhooks.rb in mailgun-ruby-1.2.13 vs lib/mailgun/webhooks/webhooks.rb in mailgun-ruby-1.2.14

@@ -1,10 +1,11 @@
 module Mailgun
 
   # A Mailgun::Webhooks object is a simple CRUD interface to Mailgun Webhooks.
   # Uses Mailgun
   class Webhooks
+    ACTIONS = %w(accepted clicked complained delivered opened permanent_fail temporary_fail unsubscribed).freeze
 
     # Public creates a new Mailgun::Webhooks instance.
     #   Defaults to Mailgun::Client
     def initialize(client = Mailgun::Client.new)
       @client = client
@@ -29,11 +30,11 @@
     #
     # Returns a String of the url for the identified webhook or an
     #   empty String if one is not set
     def info(domain, action)
       res = @client.get("domains/#{domain}/webhooks/#{action}")
-      res.to_h['webhook']['url'] || ''
+      res.to_h['webhook']['urls'] || ''
     rescue NoMethodError
       ''
     end
     alias_method :get_webhook_url, :info
 
@@ -44,11 +45,11 @@
     # url    - A String of the url of the webhook
     #
     # Returns a Boolean of whether the webhook was created
     def create(domain, action, url = '')
       res = @client.post("domains/#{domain}/webhooks", id: action, url: url)
-      res.to_h['webhook']['url'] == url && res.to_h['message'] == 'Webhook has been created'
+      res.to_h['webhook']['urls'].include?(url) && res.to_h['message'] == 'Webhook has been created'
     end
     alias_method :add, :create
     alias_method :add_webhook, :create
 
     # Public: Sets all webhooks to the same URL
@@ -56,19 +57,34 @@
     # domain - A String of the domain name
     # url    - A String of the url to set all webhooks to
     #
     # Returns true or false
     def create_all(domain, url = '')
-      %w(accepted clicked complained delivered opened permanent_fail temporary_fail unsubscribed).each do |action|
+      ACTIONS.each do |action|
         add_webhook domain, action, url
       end
       true
     rescue
       false
     end
     alias_method :add_all_webhooks, :create_all
 
+    # Public: Update webhook
+    #
+    # domain - A String of the domain name (ex. domain.com)
+    # action - A String of the action to create a webhook for
+    # url    - A String of the url of the webhook
+    #
+    # Returns a Boolean of whether the webhook was updated
+    def update(domain, action, url = '')
+      fail Mailgun::ParameterError('Domain not provided to update webhooks') unless domain
+      fail Mailgun::ParameterError('Action not provided to identify webhook to update') unless action
+      res = @client.put("domains/#{domain}/webhooks/#{action}", id: action, url: url)
+      res.to_h['webhook']['urls'] == url && res.to_h['message'] == 'Webhook has been updated'
+    end
+    alias_method :update_webhook, :update
+
     # Public: Delete a specific webhook
     #
     # domain - The required String of domain name
     # action - The required String of the webhook action to delete
     #
@@ -88,10 +104,10 @@
     # domain - A required String of the domain to remove all webhooks for
     #
     # Returns a Boolean on the success
     def remove_all(domain)
       fail Mailgun::ParameterError('Domain not provided to remove webhooks from') unless domain
-      %w(accepted clicked complained delivered opened permanent_fail temporary_fail unsubscribed).each do |action|
+      ACTIONS.each do |action|
         delete_webhook domain, action
       end
     end
     alias_method :delete_all, :remove_all
     alias_method :delete_all_webooks, :remove_all