Merge pull request #2957 from hughrun/redisfix

fix illegal values in redis jobs
This commit is contained in:
Mouse Reeve 2023-08-19 16:41:50 -07:00 committed by GitHub
commit 0f8da5b738
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 2 deletions

View file

@ -112,7 +112,7 @@ class ActivityStream(RedisStore):
trace.get_current_span().set_attribute("status_privacy", status.privacy)
trace.get_current_span().set_attribute(
"status_reply_parent_privacy",
status.reply_parent.privacy if status.reply_parent else None,
status.reply_parent.privacy if status.reply_parent else status.privacy,
)
# direct messages don't appear in feeds, direct comments/reviews/etc do
if status.privacy == "direct" and status.status_type == "Note":

View file

@ -254,7 +254,8 @@ def rerank_suggestions_task(user_id):
def rerank_user_task(user_id, update_only=False):
"""do the hard work in celery"""
user = models.User.objects.get(id=user_id)
suggested_users.rerank_obj(user, update_only=update_only)
if user:
suggested_users.rerank_obj(user, update_only=update_only)
@app.task(queue=SUGGESTED_USERS)