Allow specifiy user different from guardian.user in chat services such as Chat::CreateMessage

Lhc_fl · November 12, 2023, 12:16pm

Now that PostCreator can specify user as post’s creator and guardian to verify posting permissions, I would like Discourse Chat to have similar behavior, which would allow plugins to help send messages from a user (whether it is in the chat channel or not) with the guardian of the system user.

github.com

discourse/discourse/blob/main/plugins/chat/app/services/chat/create_message.rb#L129


      
            reply.thread == thread
          end
          
          def fetch_uploads(contract:, guardian:, **)
            return [] if !SiteSetting.chat_allow_uploads
            guardian.user.uploads.where(id: contract.upload_ids)
          end
          
          def instantiate_message(channel:, guardian:, contract:, uploads:, thread:, reply:, **)
            channel.chat_messages.new(
              user: guardian.user,
              last_editor: guardian.user,
              in_reply_to: reply,
              message: contract.message,
              uploads: uploads,
              thread: thread,
            )
          end
          
          def save_message(message_instance:, **)
            message_instance.save!

github.com

discourse/discourse/blob/cdbe0f74e8659661281decdc9b54ce5e6aa53075/lib/post_creator.rb#L86


      
          
          def spam?
            @spam
          end
          
          def skip_validations?
            @opts[:skip_validations]
          end
          
          def guardian
            @guardian ||= @opts[:guardian] || Guardian.new(@user)
          end
          
          def valid?
            @topic = nil
            @post = nil
          
            if @user.suspended? && !skip_validations?
              errors.add(:base, I18n.t(:user_is_suspended))
              return false
            end

Usage: In Discourse Automation and Chat bridge, you need to send chat messages from plugin. But currently this script will most likely fail to be sent because the user you specify (e.g. DiscoBot) may not be in the channel.

github.com

discourse/discourse/blob/cdbe0f74e8659661281decdc9b54ce5e6aa53075/plugins/chat/plugin.rb#L454


      
          
              script do |context, fields, automation|
                sender = User.find_by(username: fields.dig("sender", "value")) || Discourse.system_user
                channel = Chat::Channel.find_by(id: fields.dig("chat_channel_id", "value"))
          
                placeholders = { channel_name: channel.title(sender) }.merge(context["placeholders"] || {})
          
                creator =
                  ::Chat::CreateMessage.call(
                    chat_channel_id: channel.id,
                    guardian: sender.guardian,
                    message: utils.apply_placeholders(fields.dig("message", "value"), placeholders),
                  )
          
                if creator.failure?
                  Rails.logger.warn "[discourse-automation] Chat message failed to send:\n#{creator.inspect_steps.inspect}\n#{creator.inspect_steps.error}"
                end
              end
            end
          end

I know the team is very busy, so I can do this by myself. I hope the Discourse team thinks this can be added to the core, then I will take the time to modify the code here and submit a pull request.

merefield · November 13, 2023, 2:52pm

Discourse Chatbot solved a similar challenge with current code without needing any specific changes in core, have you taken a look at my solution?

github.com

merefield/discourse-chatbot/blob/f782223d5f6306e865a1604548fd11d4833b5c7c/lib/discourse_chatbot/reply_creator.rb#L7


      
          # frozen_string_literal: true
          module ::DiscourseChatbot
            class ReplyCreator
          
              def initialize(options = {})
                @author = ::User.find_by(id: options[:bot_user_id])
                @guardian = Guardian.new(@author)
                @reply_to = options[:reply_to_message_or_post_id]
                @reply_to_post_number = options[:original_post_number]
                @topic_or_channel_id = options[:topic_or_channel_id]
                @message_body = options[:reply]
                @is_private_msg = options[:is_private_msg]
                @inner_thoughts = options[:inner_thoughts]
                if @message_body.blank?
                  @message_body = '...'
                end
              end

github.com

merefield/discourse-chatbot/blob/f782223d5f6306e865a1604548fd11d4833b5c7c/lib/discourse_chatbot/message/message_reply_creator.rb#L12


      
          module ::DiscourseChatbot
            class MessageReplyCreator < ReplyCreator
          
              def initialize(options = {})
                super(options)
              end
          
              def create
                ::DiscourseChatbot.progress_debug_message("5. Creating a new Chat Nessage...")
                begin
                  Chat::CreateMessage.call(
                    chat_channel_id: @topic_or_channel_id,
                    guardian: @guardian,
                    message: @message_body,
                  )
          
                  presence = PresenceChannel.new("/chat-reply/#{@topic_or_channel_id}")
                  presence.leave(user_id: @author.id, client_id: "12345")
          
                  ::DiscourseChatbot.progress_debug_message("6. The Message has been created successfully")
                rescue => e

Lhc_fl · November 14, 2023, 2:36am

thank you so much for your suggestion! let me see it

joffreyjaffeux · December 4, 2023, 7:53pm

Yes as @merefield said, a chat service accepts a guardian, and you can set the guardian as you want.

joffreyjaffeux · December 8, 2023, 7:54pm

This topic was automatically closed 4 days after the last reply. New replies are no longer allowed.

Topic		Replies	Views
Chat message failed to send because of channel membership bug automation , chat	0	215	October 26, 2023
Add Option to Disable User-User Chat feature chat	28	2814	December 13, 2022
Discourse Chat Bridge (currently only supports Telegram) plugin	22	1763	March 26, 2024
Discourse Version 3.1 releases	2	6366	August 1, 2023
Better Control Over Incoming Chats feature chat	7	386	January 3, 2024

Allow specifiy user different from guardian.user in chat services such as Chat::CreateMessage

Related Topics