ref: skip event pre-normalize through CanonicalKeyDict (#70853)

keys get normalized through the relay normalization

this gets the rest of the tests passing when changing CanonicalKeyDict
to raise when legacy keys are passed -- the testsuite passes lots of
`{"message": ...}` similar to current sdks!

<!-- Describe your PR here. -->

src/sentry/event_manager.py

@@ -338,7 +338,7 @@ def __init__(
         project_config: Any | None = None,
         sent_at: datetime | None = None,
     ):
-        self._data = CanonicalKeyDict(data)
+        self._data: MutableMapping[str, Any] = data
         self.version = version
         self._project = project
         # if not explicitly specified try to get the grouping from project_config
@@ -401,7 +401,7 @@ def _normalize_impl(self, project_id: int | None = None) -> None:
         if pre_normalize_type in ("generic", "feedback"):
             self._data["type"] = pre_normalize_type
 
-    def get_data(self) -> CanonicalKeyDict:
+    def get_data(self) -> MutableMapping[str, Any]:
         return self._data
 
     @sentry_sdk.tracing.trace

src/sentry/web/frontend/debug/mail.py

@@ -431,7 +431,7 @@ def get(self, request: AuthenticatedHttpRequest) -> HttpResponse:
         event_type = get_event_type(data)
 
         event = eventstore.backend.create_event(
-            event_id="a" * 32, group_id=group.id, project_id=project.id, data=data.data
+            event_id="a" * 32, group_id=group.id, project_id=project.id, data=data
         )
 
         group.message = event.search_message
@@ -532,7 +532,7 @@ def digest(request):
             )
 
             event = eventstore.backend.create_event(
-                event_id=uuid.uuid4().hex, group_id=group.id, project_id=project.id, data=data.data
+                event_id=uuid.uuid4().hex, group_id=group.id, project_id=project.id, data=data
             )
             records.append(
                 Record(

tests/sentry/event_manager/test_normalization.py

@@ -199,13 +199,13 @@ def test_deprecated_attrs(key):
     assert not data.get("errors")
 
 
-def test_returns_canonical_dict():
+def test_returns_canonical_dict_after_normalization():
     from sentry.utils.canonical import CanonicalKeyDict
 
     event = make_event()
 
     manager = EventManager(event)
-    assert isinstance(manager.get_data(), CanonicalKeyDict)
+    assert not isinstance(manager.get_data(), CanonicalKeyDict)
     manager.normalize()
     assert isinstance(manager.get_data(), CanonicalKeyDict)