Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(replays): allow replay requests to query across projects #48493

Merged
merged 2 commits into from
May 5, 2023

Conversation

JoshFerge
Copy link
Member

@JoshFerge JoshFerge commented May 3, 2023

  • On the Replay Details page, since replays can be associated with backend errors and traces, we need to be able to query across projects to get the associated data associated with the replay for issues and events.
  • We'll do so by adding a header to those specific requests that the backend will then look for, and allow those queries to get multi project results.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 3, 2023
@JoshFerge JoshFerge marked this pull request as ready for review May 3, 2023 23:32
@JoshFerge JoshFerge requested review from a team May 3, 2023 23:32
@codecov
Copy link

codecov bot commented May 3, 2023

Codecov Report

Merging #48493 (05ced3e) into master (c1b2bff) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #48493    +/-   ##
========================================
  Coverage   80.93%   80.94%            
========================================
  Files        4775     4780     +5     
  Lines      201658   201912   +254     
  Branches    11503    11503            
========================================
+ Hits       163221   163432   +211     
- Misses      38182    38225    +43     
  Partials      255      255            
Impacted Files Coverage Δ
src/sentry/api/bases/organization_events.py 97.45% <100.00%> (+0.04%) ⬆️
...c/sentry/api/endpoints/organization_group_index.py 91.92% <100.00%> (+0.15%) ⬆️
.../sentry/api/endpoints/organization_issues_count.py 87.93% <100.00%> (+0.21%) ⬆️

... and 66 files with indirect coverage changes

Copy link
Member

@markstory markstory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should there be a test covering this?

@JoshFerge JoshFerge requested a review from a team May 5, 2023 02:09
"organizations:global-views", organization, actor=request.user
is_fetching_replay_data = request.headers.get("X-Sentry-Replay-Request") == "1"

if (
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are there no tests for this endpoint? 😨

@eliashussary eliashussary merged commit ad0300f into master May 5, 2023
@eliashussary eliashussary deleted the jferg/add-replay-header-query-data branch May 5, 2023 14:57
eliashussary pushed a commit that referenced this pull request May 5, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2023
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