gdpelican / babble

Discourse Shoutbox plugin
MIT License
107 stars 63 forks source link

PostAlerter.notify_users was called outside of sidekiq on user '@'taging #256

Closed MakaryGo closed 6 years ago

MakaryGo commented 6 years ago

Hi. Using newest Discourse and fabulous pms branch of Babble

When summoning some @user warning(s) show up:

image

hostname | d<#>.org
-- | --
process_id | 190
application_version | 9b18ee4bba634bf239d6cb83e78e7167b2eca8d9
HTTP_HOST | d<#>.org
REQUEST_URI | /babble/topics/907/posts
REQUEST_METHOD | POST
HTTP_USER_AGENT | Mozilla/5.0 (Linux; Android 6.0.1; SM-G920F Build/MMB29K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.183 Mobile Safari/537.36 Vivaldi/1.96.1147.55
HTTP_ACCEPT | */*
HTTP_REFERER | https://d<#>.org/
HTTP_X_FORWARDED_FOR | 89.64.27.10, unix:
HTTP_X_REAL_IP | unix:
username | xavery
  | paramsraw@MFLisewicz a dziękuję, jak to mawia dziś młodzież _"szału nie ma"_ , ale cóż... Lepiej martiwć się n | params | raw@MFLisewicz a dziękuję, jak to mawia dziś młodzież _"szału nie ma"_ , ale cóż... Lepiej martiwć się n | raw | @MFLisewicz a dziękuję, jak to mawia dziś młodzież _"szału nie ma"_ , ale cóż... Lepiej martiwć się n
params | raw@MFLisewicz a dziękuję, jak to mawia dziś młodzież _"szału nie ma"_ , ale cóż... Lepiej martiwć się n | raw | @MFLisewicz a dziękuję, jak to mawia dziś młodzież _"szału nie ma"_ , ale cóż... Lepiej martiwć się n
raw | @MFLisewicz a dziękuję, jak to mawia dziś młodzież _"szału nie ma"_ , ale cóż... Lepiej martiwć się n

//EDIT: Olso, notification never reached the other guy

gdpelican commented 6 years ago

Alright, I believe I've fixed this in latest by moving babble post processing into a sidekiq job. Let me know if it crops up for you again?