Разрешить указывать пользователя, отличного от guardian.user, в сервисах чата, таких как Chat::CreateMessage

Lhc_fl · 12.Ноябрь.2023 12:16:54

Теперь, когда PostCreator может указывать user как автора поста и guardian для проверки прав на публикацию, я хотел бы, чтобы Discourse Chat обладал аналогичной функциональностью. Это позволит плагинам отправлять сообщения от имени пользователя (независимо от того, состоит ли он в чат-канале или нет), используя права системного пользователя.

github.com/discourse/discourse

plugins/chat/app/services/chat/create_message.rb

main


      
            channel.membership_for(guardian.user)
          end
          
          def fetch_reply(params:)
            Chat::Message.find_by(id: params.in_reply_to_id)
          end
          
          def ensure_reply_consistency(channel:, params:, reply:)
            return true if params.in_reply_to_id.blank?
            reply&.chat_channel == channel
          end
          
          def fetch_thread(params:, reply:, channel:, options:)
            return Chat::Thread.find_by(id: params.thread_id) if params.thread_id.present?
            return unless reply
            reply.thread ||
              reply.build_thread(
                original_message: reply,
                original_message_user: reply.user,
                channel: channel,
                force: options.force_thread,

github.com/discourse/discourse

lib/post_creator.rb

cdbe0f74e


      
          
          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

Использование: В Discourse Automation и мосте для чата необходимо отправлять сообщения из плагина. Однако в настоящее время этот скрипт, скорее всего, не будет отправлен, так как указанный вами пользователь (например, DiscoBot) может не состоять в канале.

github.com/discourse/discourse

plugins/chat/plugin.rb

cdbe0f74e


      
          
              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

Я понимаю, что команда очень занята, поэтому я могу сделать это сам. Надеюсь, команда Discourse сочтет возможным добавить эту функцию в ядро, и тогда я потрачу время на внесение изменений в код и отправлю pull request.

merefield · 13.Ноябрь.2023 14:52:34

Discourse Chatbot решил аналогичную задачу с помощью текущего кода без необходимости каких-либо изменений в ядре. Вы уже посмотрели моё решение?

github.com/merefield/discourse-chatbot

lib/discourse_chatbot/reply_creator.rb

f782223d5


      
          # 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

lib/discourse_chatbot/message/message_reply_creator.rb

f782223d5


      
          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 · 14.Ноябрь.2023 02:36:15

Большое спасибо за ваше предложение! Давайте посмотрим.

j.jaffeux · 04.Декабрь.2023 19:53:45

Да, как отметил @merefield, сервис чата принимает гаранта, и вы можете установить гаранта по своему усмотрению.

j.jaffeux · 08.Декабрь.2023 19:54:16

Эта тема была автоматически закрыта через 4 дня после последнего ответа. Новые ответы больше не принимаются.

Тема		Ответов	Просм.
How to make only a group of users can send messages to staff Support	5	947	29.06.2020
Chat message failed to send because of channel membership Bug automation , chat	0	274	26.10.2023
Discourse Chat Plugin official , chat , included-in-core	45	33149	21.05.2026
Introducing Discourse Chat (BETA) Announcements chat , new-feature	149	26936	03.08.2022
Discourse Chat Integrations Plugin official , included-in-core , chat-integration	22	102602	21.04.2026

Разрешить указывать пользователя, отличного от guardian.user, в сервисах чата, таких как Chat::CreateMessage

Связанные темы