2013-10-21 23:25:53 +02:00
|
|
|
import datetime
|
2017-12-13 01:45:57 +01:00
|
|
|
import logging
|
2020-06-11 00:54:34 +02:00
|
|
|
from collections import defaultdict
|
2020-11-03 20:48:57 +01:00
|
|
|
from typing import Any, Dict, List, Set, Tuple
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2013-11-16 00:54:12 +01:00
|
|
|
from django.conf import settings
|
2020-11-13 17:53:41 +01:00
|
|
|
from django.db import transaction
|
2017-08-27 16:30:48 +02:00
|
|
|
from django.utils.timezone import now as timezone_now
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2018-11-08 22:40:27 +01:00
|
|
|
from confirmation.models import one_click_unsubscribe_link
|
2016-11-08 10:07:47 +01:00
|
|
|
from zerver.context_processors import common_context
|
2020-06-11 00:54:34 +02:00
|
|
|
from zerver.lib.email_notifications import build_message_list
|
2017-12-13 01:45:57 +01:00
|
|
|
from zerver.lib.logging_util import log_to_file
|
2020-11-12 12:11:35 +01:00
|
|
|
from zerver.lib.message import get_last_message_id
|
2020-06-11 00:54:34 +02:00
|
|
|
from zerver.lib.queue import queue_json_publish
|
|
|
|
from zerver.lib.send_email import FromAddress, send_future_email
|
|
|
|
from zerver.lib.url_encoding import encode_stream
|
|
|
|
from zerver.models import (
|
|
|
|
Message,
|
|
|
|
Realm,
|
|
|
|
RealmAuditLog,
|
|
|
|
Recipient,
|
2020-11-13 16:52:13 +01:00
|
|
|
Stream,
|
2020-06-11 00:54:34 +02:00
|
|
|
Subscription,
|
2020-11-12 16:55:32 +01:00
|
|
|
UserActivityInterval,
|
2020-06-11 00:54:34 +02:00
|
|
|
UserProfile,
|
|
|
|
get_active_streams,
|
|
|
|
)
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2017-12-13 01:45:57 +01:00
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
log_to_file(logger, settings.DIGEST_LOG_PATH)
|
2014-01-08 17:33:12 +01:00
|
|
|
|
2017-08-27 16:30:48 +02:00
|
|
|
DIGEST_CUTOFF = 5
|
|
|
|
|
2020-11-03 14:57:11 +01:00
|
|
|
TopicKey = Tuple[int, str]
|
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
class DigestTopic:
|
|
|
|
def __init__(self, topic_key: TopicKey) -> None:
|
|
|
|
self.topic_key = topic_key
|
|
|
|
self.human_senders: Set[str] = set()
|
|
|
|
self.sample_messages: List[Message] = []
|
|
|
|
self.num_human_messages = 0
|
|
|
|
|
2020-11-13 12:07:50 +01:00
|
|
|
def stream_id(self) -> int:
|
|
|
|
# topic_key is (stream_id, topic_name)
|
|
|
|
return self.topic_key[0]
|
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
def add_message(self, message: Message) -> None:
|
|
|
|
if len(self.sample_messages) < 2:
|
|
|
|
self.sample_messages.append(message)
|
|
|
|
|
|
|
|
if message.sent_by_human():
|
|
|
|
self.human_senders.add(message.sender.full_name)
|
|
|
|
self.num_human_messages += 1
|
|
|
|
|
|
|
|
def length(self) -> int:
|
|
|
|
return self.num_human_messages
|
|
|
|
|
|
|
|
def diversity(self) -> int:
|
|
|
|
return len(self.human_senders)
|
|
|
|
|
2020-11-13 16:52:13 +01:00
|
|
|
def teaser_data(self, user: UserProfile, stream_map: Dict[int, Stream]) -> Dict[str, Any]:
|
2020-11-06 15:45:42 +01:00
|
|
|
teaser_count = self.num_human_messages - len(self.sample_messages)
|
|
|
|
first_few_messages = build_message_list(
|
2020-11-13 16:52:13 +01:00
|
|
|
user=user,
|
|
|
|
messages=self.sample_messages,
|
|
|
|
stream_map=stream_map,
|
2020-11-06 15:45:42 +01:00
|
|
|
)
|
|
|
|
return {
|
2020-12-01 06:12:49 +01:00
|
|
|
"participants": sorted(self.human_senders),
|
2020-11-06 15:45:42 +01:00
|
|
|
"count": teaser_count,
|
|
|
|
"first_few_messages": first_few_messages,
|
|
|
|
}
|
2020-11-03 17:13:22 +01:00
|
|
|
|
2019-05-02 05:54:48 +02:00
|
|
|
# Digests accumulate 2 types of interesting traffic for a user:
|
|
|
|
# 1. New streams
|
|
|
|
# 2. Interesting stream traffic, as determined by the longest and most
|
2013-10-21 23:25:53 +02:00
|
|
|
# diversely comment upon topics.
|
|
|
|
|
2017-11-05 11:15:10 +01:00
|
|
|
def should_process_digest(realm_str: str) -> bool:
|
2017-08-25 21:48:19 +02:00
|
|
|
if realm_str in settings.SYSTEM_ONLY_REALMS:
|
|
|
|
# Don't try to send emails to system-only realms
|
|
|
|
return False
|
|
|
|
return True
|
|
|
|
|
2017-08-25 21:52:47 +02:00
|
|
|
# Changes to this should also be reflected in
|
|
|
|
# zerver/worker/queue_processors.py:DigestWorker.consume()
|
2020-11-13 18:13:13 +01:00
|
|
|
def queue_digest_user_ids(user_ids: List[int], cutoff: datetime.datetime) -> None:
|
2017-08-25 21:52:47 +02:00
|
|
|
# Convert cutoff to epoch seconds for transit.
|
2020-11-12 16:34:15 +01:00
|
|
|
event = {
|
2020-11-13 18:13:13 +01:00
|
|
|
"user_profile_id": user_ids,
|
2020-11-12 16:34:15 +01:00
|
|
|
"cutoff": cutoff.strftime('%s')
|
|
|
|
}
|
2017-11-24 13:18:46 +01:00
|
|
|
queue_json_publish("digest_emails", event)
|
2017-08-25 21:52:47 +02:00
|
|
|
|
2017-11-05 11:15:10 +01:00
|
|
|
def enqueue_emails(cutoff: datetime.datetime) -> None:
|
2018-03-09 00:16:44 +01:00
|
|
|
if not settings.SEND_DIGEST_EMAILS:
|
|
|
|
return
|
|
|
|
|
2019-03-28 04:47:03 +01:00
|
|
|
weekday = timezone_now().weekday()
|
|
|
|
for realm in Realm.objects.filter(deactivated=False, digest_emails_enabled=True, digest_weekday=weekday):
|
2020-11-12 13:01:07 +01:00
|
|
|
if should_process_digest(realm.string_id):
|
|
|
|
_enqueue_emails_for_realm(realm, cutoff)
|
|
|
|
|
|
|
|
def _enqueue_emails_for_realm(realm: Realm, cutoff: datetime.datetime) -> None:
|
|
|
|
# This should only be called directly by tests. Use enqueue_emails
|
|
|
|
# to process all realms that are set up for processing on any given day.
|
2020-11-12 16:34:15 +01:00
|
|
|
realm_user_ids = set(UserProfile.objects.filter(
|
2020-11-12 13:01:07 +01:00
|
|
|
realm=realm,
|
|
|
|
is_active=True,
|
|
|
|
is_bot=False,
|
|
|
|
enable_digest_emails=True,
|
2020-11-12 16:34:15 +01:00
|
|
|
).values_list('id', flat=True))
|
2020-11-12 13:01:07 +01:00
|
|
|
|
|
|
|
twelve_hours_ago = timezone_now() - datetime.timedelta(hours=12)
|
|
|
|
|
|
|
|
recent_user_ids = set(RealmAuditLog.objects.filter(
|
|
|
|
realm_id=realm.id,
|
|
|
|
event_type=RealmAuditLog.USER_DIGEST_EMAIL_CREATED,
|
|
|
|
event_time__gt=twelve_hours_ago,
|
|
|
|
).values_list('modified_user_id', flat=True).distinct())
|
|
|
|
|
2020-11-12 16:55:32 +01:00
|
|
|
realm_user_ids -= recent_user_ids
|
|
|
|
|
|
|
|
active_user_ids = set(UserActivityInterval.objects.filter(
|
|
|
|
user_profile_id__in=realm_user_ids,
|
|
|
|
end__gt=cutoff,
|
|
|
|
).values_list('user_profile_id', flat=True).distinct())
|
|
|
|
|
2020-11-13 18:13:13 +01:00
|
|
|
user_ids = list(realm_user_ids - active_user_ids)
|
|
|
|
user_ids.sort()
|
2020-11-12 13:01:07 +01:00
|
|
|
|
2020-11-13 18:13:13 +01:00
|
|
|
# We process batches of 30. We want a big enough batch
|
|
|
|
# to amorize work, but not so big that a single item
|
|
|
|
# from the queue takes too long to process.
|
|
|
|
chunk_size = 30
|
|
|
|
for i in range(0, len(user_ids), chunk_size):
|
|
|
|
chunk_user_ids = user_ids[i:i + chunk_size]
|
|
|
|
queue_digest_user_ids(chunk_user_ids, cutoff)
|
2020-11-12 13:01:07 +01:00
|
|
|
logger.info(
|
2020-11-13 18:13:13 +01:00
|
|
|
"Queuing user_ids for potential digest: %s",
|
|
|
|
chunk_user_ids,
|
2020-11-12 13:01:07 +01:00
|
|
|
)
|
2017-08-27 16:30:48 +02:00
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
def get_recent_topics(
|
2020-11-03 13:51:59 +01:00
|
|
|
stream_ids: List[int],
|
|
|
|
cutoff_date: datetime.datetime,
|
2020-11-06 15:45:42 +01:00
|
|
|
) -> List[DigestTopic]:
|
2020-11-03 17:13:22 +01:00
|
|
|
# Gather information about topic conversations, then
|
|
|
|
# classify by:
|
|
|
|
# * topic length
|
|
|
|
# * number of senders
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-03 13:51:59 +01:00
|
|
|
messages = Message.objects.filter(
|
|
|
|
recipient__type=Recipient.STREAM,
|
|
|
|
recipient__type_id__in=stream_ids,
|
2020-11-06 17:17:01 +01:00
|
|
|
date_sent__gt=cutoff_date,
|
|
|
|
).order_by(
|
|
|
|
'id', # we will sample the first few messages
|
|
|
|
).select_related(
|
|
|
|
'recipient', # we need stream_id
|
|
|
|
'sender', # we need the sender's full name
|
|
|
|
'sending_client' # for Message.sent_by_human
|
|
|
|
)
|
2020-11-03 13:51:59 +01:00
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
digest_topic_map: Dict[TopicKey, DigestTopic] = {}
|
2018-11-11 19:06:59 +01:00
|
|
|
for message in messages:
|
2020-11-06 15:45:42 +01:00
|
|
|
topic_key = (message.recipient.type_id, message.topic_name())
|
2018-11-11 19:06:59 +01:00
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
if topic_key not in digest_topic_map:
|
|
|
|
digest_topic_map[topic_key] = DigestTopic(topic_key)
|
2018-11-11 18:46:21 +01:00
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
digest_topic_map[topic_key].add_message(message)
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
topics = list(digest_topic_map.values())
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
return topics
|
2020-11-03 17:13:22 +01:00
|
|
|
|
2020-11-13 12:07:50 +01:00
|
|
|
def get_hot_topics(
|
|
|
|
all_topics: List[DigestTopic],
|
|
|
|
stream_ids: Set[int],
|
|
|
|
) -> List[DigestTopic]:
|
|
|
|
topics = [
|
|
|
|
topic for topic in all_topics
|
|
|
|
if topic.stream_id() in stream_ids
|
|
|
|
]
|
2020-11-06 15:45:42 +01:00
|
|
|
topics_by_diversity = sorted(topics, key=lambda dt: dt.diversity())
|
|
|
|
topics_by_length = sorted(topics, key=lambda dt: dt.length())
|
2020-11-06 12:44:48 +01:00
|
|
|
|
|
|
|
# Start with the two most diverse topics.
|
2020-11-03 16:54:45 +01:00
|
|
|
hot_topics = topics_by_diversity[:2]
|
2020-11-06 12:44:48 +01:00
|
|
|
|
|
|
|
# Pad out our list up to 4 items, using the topics' length (aka message
|
|
|
|
# count) as the secondary filter.
|
2020-11-06 15:45:42 +01:00
|
|
|
for topic in topics_by_length:
|
|
|
|
if topic not in hot_topics:
|
|
|
|
hot_topics.append(topic)
|
2020-11-03 16:54:45 +01:00
|
|
|
if len(hot_topics) >= 4:
|
2013-10-21 23:25:53 +02:00
|
|
|
break
|
|
|
|
|
2020-11-03 18:03:42 +01:00
|
|
|
return hot_topics
|
|
|
|
|
2020-11-13 17:25:52 +01:00
|
|
|
def get_recent_streams(realm: Realm, threshold: datetime.datetime) -> List[Stream]:
|
|
|
|
fields = ["id", "name", "is_web_public", "invite_only"]
|
|
|
|
return list(get_active_streams(realm).filter(date_created__gt=threshold).only(*fields))
|
|
|
|
|
|
|
|
def gather_new_streams(
|
|
|
|
realm: Realm,
|
|
|
|
recent_streams: List[Stream], # streams only need id and name
|
|
|
|
can_access_public: bool,
|
|
|
|
) -> Tuple[int, Dict[str, List[str]]]:
|
|
|
|
if can_access_public:
|
|
|
|
new_streams = [stream for stream in recent_streams if not stream.invite_only]
|
|
|
|
else:
|
|
|
|
new_streams = [stream for stream in recent_streams if stream.is_web_public]
|
|
|
|
|
|
|
|
base_url = f"{realm.uri}/#narrow/stream/"
|
2013-10-25 18:53:35 +02:00
|
|
|
|
|
|
|
streams_html = []
|
|
|
|
streams_plain = []
|
|
|
|
|
2013-10-21 23:25:53 +02:00
|
|
|
for stream in new_streams:
|
2018-02-15 21:02:47 +01:00
|
|
|
narrow_url = base_url + encode_stream(stream.id, stream.name)
|
2020-06-10 06:41:04 +02:00
|
|
|
stream_link = f"<a href='{narrow_url}'>{stream.name}</a>"
|
2013-10-25 18:53:35 +02:00
|
|
|
streams_html.append(stream_link)
|
|
|
|
streams_plain.append(stream.name)
|
2013-10-21 23:25:53 +02:00
|
|
|
|
|
|
|
return len(new_streams), {"html": streams_html, "plain": streams_plain}
|
|
|
|
|
2019-05-02 05:54:48 +02:00
|
|
|
def enough_traffic(hot_conversations: str, new_streams: int) -> bool:
|
|
|
|
return bool(hot_conversations or new_streams)
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-13 16:53:50 +01:00
|
|
|
def get_user_stream_map(user_ids: List[int]) -> Dict[int, Set[int]]:
|
2020-11-13 12:30:59 +01:00
|
|
|
rows = Subscription.objects.filter(
|
|
|
|
user_profile_id__in=user_ids,
|
|
|
|
recipient__type=Recipient.STREAM,
|
|
|
|
active=True,
|
|
|
|
is_muted=False,
|
|
|
|
).values('user_profile_id', 'recipient__type_id')
|
|
|
|
|
|
|
|
# maps user_id -> {stream_id, stream_id, ...}
|
|
|
|
dct: Dict[int, Set[int]] = defaultdict(set)
|
|
|
|
for row in rows:
|
|
|
|
dct[row['user_profile_id']].add(row['recipient__type_id'])
|
|
|
|
|
|
|
|
return dct
|
|
|
|
|
2020-11-13 16:52:13 +01:00
|
|
|
def get_slim_stream_map(stream_ids: Set[int]) -> Dict[int, Stream]:
|
|
|
|
# This can be passed to build_message_list.
|
|
|
|
streams = Stream.objects.filter(
|
|
|
|
id__in=stream_ids,
|
|
|
|
).only('id', 'name')
|
|
|
|
|
|
|
|
return {stream.id: stream for stream in streams}
|
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
def bulk_get_digest_context(users: List[UserProfile], cutoff: float) -> Dict[int, Dict[str, Any]]:
|
2020-11-13 17:25:52 +01:00
|
|
|
# We expect a non-empty list of users all from the same realm.
|
|
|
|
assert(users)
|
|
|
|
realm = users[0].realm
|
|
|
|
for user in users:
|
|
|
|
assert user.realm_id == realm.id
|
|
|
|
|
2013-10-21 23:25:53 +02:00
|
|
|
# Convert from epoch seconds to a datetime object.
|
2020-06-05 06:55:20 +02:00
|
|
|
cutoff_date = datetime.datetime.fromtimestamp(int(cutoff), tz=datetime.timezone.utc)
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
result: Dict[int, Dict[str, Any]] = {}
|
2016-11-08 10:07:47 +01:00
|
|
|
|
2020-11-05 11:48:48 +01:00
|
|
|
user_ids = [user.id for user in users]
|
|
|
|
|
2020-11-13 16:53:50 +01:00
|
|
|
user_stream_map = get_user_stream_map(user_ids)
|
2020-11-05 11:48:48 +01:00
|
|
|
|
2020-11-17 13:31:08 +01:00
|
|
|
recently_modified_streams = get_modified_streams(user_ids, cutoff_date)
|
2020-11-13 12:27:39 +01:00
|
|
|
|
2020-11-13 12:07:50 +01:00
|
|
|
all_stream_ids = set()
|
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
for user in users:
|
2020-11-13 16:53:50 +01:00
|
|
|
stream_ids = user_stream_map[user.id]
|
2020-11-17 13:31:08 +01:00
|
|
|
stream_ids -= recently_modified_streams.get(user.id, set())
|
2020-11-13 12:07:50 +01:00
|
|
|
all_stream_ids |= stream_ids
|
|
|
|
|
|
|
|
# Get all the recent topics for all the users. This does the heavy
|
|
|
|
# lifting of making an expensive query to the Message table. Then
|
|
|
|
# for each user, we filter to just the streams they care about.
|
|
|
|
recent_topics = get_recent_topics(sorted(list(all_stream_ids)), cutoff_date)
|
|
|
|
|
2020-11-13 16:52:13 +01:00
|
|
|
stream_map = get_slim_stream_map(all_stream_ids)
|
|
|
|
|
2020-11-13 17:25:52 +01:00
|
|
|
recent_streams = get_recent_streams(realm, cutoff_date)
|
|
|
|
|
2020-11-13 12:07:50 +01:00
|
|
|
for user in users:
|
2020-11-13 16:53:50 +01:00
|
|
|
stream_ids = user_stream_map[user.id]
|
2020-11-13 12:07:50 +01:00
|
|
|
|
|
|
|
hot_topics = get_hot_topics(recent_topics, stream_ids)
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-13 11:20:38 +01:00
|
|
|
context = common_context(user)
|
|
|
|
|
|
|
|
# Start building email template data.
|
|
|
|
unsubscribe_link = one_click_unsubscribe_link(user, "digest")
|
|
|
|
context.update(unsubscribe_link=unsubscribe_link)
|
|
|
|
|
2020-11-06 15:45:42 +01:00
|
|
|
# Get context data for hot conversations.
|
|
|
|
context["hot_conversations"] = [
|
2020-11-13 16:52:13 +01:00
|
|
|
hot_topic.teaser_data(user, stream_map)
|
2020-11-06 15:45:42 +01:00
|
|
|
for hot_topic in hot_topics
|
|
|
|
]
|
2013-10-21 23:25:53 +02:00
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
# Gather new streams.
|
2020-11-13 17:25:52 +01:00
|
|
|
new_streams_count, new_streams = gather_new_streams(
|
|
|
|
realm=realm,
|
|
|
|
recent_streams=recent_streams,
|
|
|
|
can_access_public=user.can_access_public_streams()
|
|
|
|
)
|
2020-11-04 23:40:29 +01:00
|
|
|
context["new_streams"] = new_streams
|
|
|
|
context["new_streams_count"] = new_streams_count
|
2019-08-17 11:04:48 +02:00
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
result[user.id] = context
|
|
|
|
|
|
|
|
return result
|
2020-11-03 20:48:57 +01:00
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
def get_digest_context(user: UserProfile, cutoff: float) -> Dict[str, Any]:
|
|
|
|
return bulk_get_digest_context([user], cutoff)[user.id]
|
|
|
|
|
2020-11-13 17:53:41 +01:00
|
|
|
@transaction.atomic
|
2020-11-04 23:40:29 +01:00
|
|
|
def bulk_handle_digest_email(user_ids: List[int], cutoff: float) -> None:
|
2020-11-12 16:00:39 +01:00
|
|
|
# We go directly to the database to get user objects,
|
|
|
|
# since inactive users are likely to not be in the cache.
|
2020-11-13 13:21:12 +01:00
|
|
|
users = UserProfile.objects.filter(id__in=user_ids).order_by("id").select_related("realm")
|
2020-11-04 23:40:29 +01:00
|
|
|
context_map = bulk_get_digest_context(users, cutoff)
|
|
|
|
|
2020-11-12 12:11:35 +01:00
|
|
|
digest_users = []
|
|
|
|
|
2020-11-04 23:40:29 +01:00
|
|
|
for user in users:
|
|
|
|
context = context_map[user.id]
|
|
|
|
|
|
|
|
# We don't want to send emails containing almost no information.
|
|
|
|
if enough_traffic(context["hot_conversations"], context["new_streams_count"]):
|
2020-11-12 12:11:35 +01:00
|
|
|
digest_users.append(user)
|
2020-11-04 23:40:29 +01:00
|
|
|
logger.info("Sending digest email for user %s", user.id)
|
|
|
|
# Send now, as a ScheduledEmail
|
|
|
|
send_future_email(
|
|
|
|
'zerver/emails/digest',
|
|
|
|
user.realm,
|
|
|
|
to_user_ids=[user.id],
|
|
|
|
from_name="Zulip Digest",
|
|
|
|
from_address=FromAddress.no_reply_placeholder,
|
|
|
|
context=context,
|
|
|
|
)
|
|
|
|
|
2020-11-12 12:11:35 +01:00
|
|
|
bulk_write_realm_audit_logs(digest_users)
|
|
|
|
|
|
|
|
def bulk_write_realm_audit_logs(users: List[UserProfile]) -> None:
|
|
|
|
if not users:
|
|
|
|
return
|
|
|
|
|
|
|
|
# We write RealmAuditLog rows for auditing, and we will also
|
|
|
|
# use these rows during the next run to possibly exclude the
|
|
|
|
# users (if insufficient time has passed).
|
|
|
|
last_message_id = get_last_message_id()
|
|
|
|
now = timezone_now()
|
|
|
|
|
|
|
|
log_rows = [
|
|
|
|
RealmAuditLog(
|
|
|
|
realm_id=user.realm_id,
|
|
|
|
modified_user_id=user.id,
|
|
|
|
event_last_message_id=last_message_id,
|
|
|
|
event_time=now,
|
|
|
|
event_type=RealmAuditLog.USER_DIGEST_EMAIL_CREATED,
|
|
|
|
)
|
|
|
|
for user in users
|
|
|
|
]
|
|
|
|
|
|
|
|
RealmAuditLog.objects.bulk_create(log_rows)
|
|
|
|
|
2020-11-13 12:27:39 +01:00
|
|
|
def get_modified_streams(user_ids: List[int], cutoff_date: datetime.datetime) -> Dict[int, Set[int]]:
|
2021-01-17 20:37:59 +01:00
|
|
|
"""Skipping streams where the user's subscription status has changed
|
|
|
|
when constructing digests is critical to ensure correctness for
|
|
|
|
streams without shared history, guest users, and long-term idle
|
|
|
|
users, because it means that every user has the same view of the
|
|
|
|
history of a given stream whose message history is being included
|
|
|
|
(and thus we can share a lot of work).
|
|
|
|
|
|
|
|
The downside is that newly created streams are never included in
|
|
|
|
the first digest email after their creation. Should we wish to
|
|
|
|
change that, we will need to be very careful to avoid creating
|
|
|
|
bugs for any of those classes of users.
|
|
|
|
"""
|
2019-03-03 07:14:58 +01:00
|
|
|
events = [
|
|
|
|
RealmAuditLog.SUBSCRIPTION_CREATED,
|
|
|
|
RealmAuditLog.SUBSCRIPTION_ACTIVATED,
|
python: Use trailing commas consistently.
Automatically generated by the following script, based on the output
of lint with flake8-comma:
import re
import sys
last_filename = None
last_row = None
lines = []
for msg in sys.stdin:
m = re.match(
r"\x1b\[35mflake8 \|\x1b\[0m \x1b\[1;31m(.+):(\d+):(\d+): (\w+)", msg
)
if m:
filename, row_str, col_str, err = m.groups()
row, col = int(row_str), int(col_str)
if filename == last_filename:
assert last_row != row
else:
if last_filename is not None:
with open(last_filename, "w") as f:
f.writelines(lines)
with open(filename) as f:
lines = f.readlines()
last_filename = filename
last_row = row
line = lines[row - 1]
if err in ["C812", "C815"]:
lines[row - 1] = line[: col - 1] + "," + line[col - 1 :]
elif err in ["C819"]:
assert line[col - 2] == ","
lines[row - 1] = line[: col - 2] + line[col - 1 :].lstrip(" ")
if last_filename is not None:
with open(last_filename, "w") as f:
f.writelines(lines)
Signed-off-by: Anders Kaseorg <anders@zulipchat.com>
2020-04-10 05:23:40 +02:00
|
|
|
RealmAuditLog.SUBSCRIPTION_DEACTIVATED,
|
2019-03-03 07:14:58 +01:00
|
|
|
]
|
|
|
|
|
2020-11-13 12:27:39 +01:00
|
|
|
# Get rows where the users' subscriptions have changed.
|
|
|
|
rows = RealmAuditLog.objects.filter(
|
|
|
|
modified_user_id__in=user_ids,
|
2019-03-03 07:14:58 +01:00
|
|
|
event_time__gt=cutoff_date,
|
2020-11-13 12:27:39 +01:00
|
|
|
event_type__in=events,
|
|
|
|
).values("modified_user_id", "modified_stream_id").distinct()
|
|
|
|
|
|
|
|
result: Dict[int, Set[int]] = defaultdict(set)
|
2019-03-03 07:14:58 +01:00
|
|
|
|
2020-11-13 12:27:39 +01:00
|
|
|
for row in rows:
|
|
|
|
user_id = row["modified_user_id"]
|
|
|
|
stream_id = row["modified_stream_id"]
|
|
|
|
result[user_id].add(stream_id)
|
|
|
|
|
|
|
|
return result
|