fix(weekly_report): Switch to uuidv4

switch to uuidv4 like the other alerts

Author: scttcper

Diff for: src/sentry/notifications/utils/__init__.py

@@ -1,8 +1,6 @@
 from __future__ import annotations
 
 import logging
-import random
-import string
 import time
 from collections import defaultdict
 from dataclasses import dataclass
@@ -443,13 +441,6 @@ def get_replay_id(event: Event | GroupEvent) -> str | None:
     return replay_id
 
 
-def generate_notification_uuid() -> str:
-    """
-    Generates a random string of 16 characters to be used as a notification uuid
-    """
-    return "".join(random.choices(string.ascii_letters + string.digits, k=16))
-
-
 @dataclass
 class PerformanceProblemContext:
     problem: PerformanceProblem

Diff for: src/sentry/tasks/weekly_reports.py

@@ -2,6 +2,7 @@
 
 import heapq
 import logging
+import uuid
 from datetime import timedelta
 from functools import partial, reduce
 from typing import Tuple
@@ -32,7 +33,6 @@
     OrganizationMember,
     OrganizationStatus,
 )
-from sentry.notifications.utils import generate_notification_uuid
 from sentry.services.hybrid_cloud.user_option import user_option_service
 from sentry.silo import SiloMode
 from sentry.snuba.dataset import Dataset
@@ -743,7 +743,7 @@ def render_template_context(ctx, user_id):
         "organizations:session-replay", ctx.organization
     ) and features.has("organizations:session-replay-weekly-email", ctx.organization)
 
-    notification_uuid = generate_notification_uuid()
+    notification_uuid = str(uuid.uuid4())
 
     # Render the first section of the email where we had the table showing the
     # number of accepted/dropped errors/transactions for each project.