Chat DM throwing back-end exception appearing on client as Not Found

merefield · January 22, 2024, 9:01am

Hey guys, I believe I’m getting an intermittent issue on Chat since updating this weekend:

This happens sometimes when entering a DM, e.g. by hitting the Chat button on a Users Card:

`PUT https://mysite.com/chat/api/channels/10/read/458 404 (Not Found)

f @ ajax.js:173`

I did a little investigation and I believe the back end is throwing this:

discourse/discourse/blob/b3904eab456f1574431761d18a5e27fc648b2c10/plugins/chat/app/controllers/chat/api/reads_controller.rb#L12


      
          
          class Chat::Api::ReadsController < Chat::ApiController
            def update
              params.require(%i[channel_id message_id])
          
              with_service(Chat::UpdateUserLastRead) do
                on_failed_policy(:ensure_message_id_recency) do
                  raise Discourse::InvalidParameters.new(:message_id)
                end
                on_model_not_found(:message) { raise Discourse::NotFound }
                on_model_not_found(:active_membership) { raise Discourse::NotFound }
                on_model_not_found(:channel) { raise Discourse::NotFound }
              end
            end
          
            def update_all
              with_service(Chat::MarkAllUserChannelsRead) do
                on_success do
                  render(json: success_json.merge(updated_memberships: result.updated_memberships))
                end
              end

merefield · January 22, 2024, 9:09am

FYI I can reproduce on Meta:

merefield · January 22, 2024, 12:46pm

STR reliably:

Using anyone’s User Card, hit “Chat”
Send them a message
Close the Chat Window completely
Hit the Chat icon and open the same DM
Exception shows in Browser.

Observation: a browser refresh initially prevents the exception.

Topic		Replies	Views
Errors after updating chat plugin support chat	13	338	August 31, 2022
Personal chat currently unavailable? bug chat	11	466	December 1, 2022
Get /users/by-external/ throwing URL Not Found error after update sso rest-api	3	487	January 20, 2023
Discourse_api URL not found error dev rest-api	5	1131	November 22, 2020
API endpoint users/by-external/ failing with authorization headers, works without bug rest-api	3	436	June 26, 2023

Chat DM throwing back-end exception appearing on client as Not Found

Related Topics