Fixup notification sending

app/models/concerns/foreman_notification_send/notification_extensions.rb

@@ -18,8 +18,10 @@ module ForemanNotificationSend
         sender.send_notification(self)
       end
 
-      NotificationTarget.select { |target| target.should_send?(self) }
-                        .each { |target| target.send(self) }
+      #NotificationTarget.select { |target| target.should_send?(self) }
+      #                  .each { |target| target.send(self) }
+    rescue StandardError => ex
+      Foreman::Logging.exception "Failed to send notification #{self}", ex
     end
 
     def level_to_symbol

app/services/foreman_notification_send/sender_matrix.rb

@@ -4,7 +4,8 @@ require 'matrix_sdk/api'
 module ForemanNotificationSend
   class SenderMatrix < SenderBase
     def initialize(hs_url:, access_token:, room:, msgtype: 'm.notice')
-      raise ArgumentError, 'access_token must be a Matrix room ID/Alias' unless access_token.is_a?(MXID) && access_token.room?
+      room = MatrixSdk::MXID.new room unless room.is_a?(MatrixSdk::MXID) 
+      raise ArgumentError, 'room must be a Matrix room ID/Alias' unless room.room?
 
       @hs_url = hs_url
       @access_token = access_token

foreman_notification_send.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |s|
   s.executables   = s.files.grep(%r{^exe/}) { |f| File.basename(f) }
   s.require_paths = ['lib']
 
-  s.add_dependency 'matrix_sdk', '~> 1'
+  s.add_dependency 'matrix_sdk', '~> 2'
 
   s.add_development_dependency 'bundler', '~> 1.16'
   s.add_development_dependency 'minitest', '~> 5.0'