Skip to content

Conversation

@cliffordxing
Copy link
Contributor

Currently, the replay EAP trace items do not include user data in the attributes dictionary. As a result, when opening a replay under the eap flag, the replay details page displays "Anonymous User". This PR writes the user data to the attributes dictionary, and asserts it with respective unit tests.

@cliffordxing cliffordxing requested a review from a team as a code owner December 18, 2025 22:14
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 18, 2025
@cliffordxing cliffordxing merged commit f78da39 into master Dec 18, 2025
67 checks passed
@cliffordxing cliffordxing deleted the add_user_metadata_to_trace_item branch December 18, 2025 22:49
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

❌ Patch coverage is 67.21311% with 20 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/replays/usecases/ingest/__init__.py 31.03% 20 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #105289      +/-   ##
===========================================
+ Coverage   79.84%    80.53%   +0.68%     
===========================================
  Files        9369      9399      +30     
  Lines      402194    404194    +2000     
  Branches    25835     25835              
===========================================
+ Hits       321145    325529    +4384     
+ Misses      80601     78217    -2384     
  Partials      448       448              

@github-actions github-actions bot locked and limited conversation to collaborators Jan 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants