Skip to content

Conversation

@armenzg
Copy link
Member

@armenzg armenzg commented Feb 23, 2023

We have events tagged with organization.slug even when it is not possible.

This change reports instances of this and resets the values for the Github webhook. The resetting would prove that we're understanding the problem and we can fix it in some instances. Finding the root problem will take a bit more work.

You can trigger this code with:

curl http://localhost:8000/extensions/github/webhook/ -d '{}'

A sample event can be seen here (private link)

A query showing transactions that should not have the event can be seen here. There may be more since I'm narrowing it down to just error.value:"Integration matching query does not exist."

It helps investigate: WOR-2464

We have events tagged with organization.slug even when it is not possible.

Fixes [WOR-2464](https://getsentry.atlassian.net/browse/WOR-2464)
@armenzg armenzg self-assigned this Feb 23, 2023
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 23, 2023
@armenzg armenzg marked this pull request as ready for review February 23, 2023 17:45
@armenzg armenzg requested a review from a team as a code owner February 23, 2023 17:45
@armenzg armenzg requested review from a team February 23, 2023 17:45
@armenzg armenzg enabled auto-merge (squash) February 23, 2023 19:39
@armenzg armenzg merged commit 5fd8242 into master Feb 24, 2023
@armenzg armenzg deleted the armenzg/integrations/org_slug branch February 24, 2023 17:45
@armenzg armenzg added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Feb 24, 2023
@getsentry-bot
Copy link
Contributor

PR reverted: 1f4d5f9

getsentry-bot added a commit that referenced this pull request Feb 24, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Mar 12, 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 Trigger: Revert Add to a merged PR to revert it (skips CI)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants