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

fix(django): Fix 404 Handler handler being labeled as "generic ASGI request" #1277

Merged
merged 9 commits into from
Jul 11, 2023
Merged

fix(django): Fix 404 Handler handler being labeled as "generic ASGI request" #1277

merged 9 commits into from
Jul 11, 2023

Conversation

BeryJu
Copy link
Contributor

@BeryJu BeryJu commented Dec 12, 2021

No description provided.

@github-actions
Copy link

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

Copy link
Member

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

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

I have one consideration to make the change use less resources.

sentry_sdk/integrations/django/__init__.py Outdated Show resolved Hide resolved
@antonpirker
Copy link
Member

It seems that we changed a lot since this PR was submitted originally. @BeryJu I know it is a lot to ask, but could you move your check for the 404 into the new _set_transaction_name_and_source? (you probably have to rebase your branch on master or merge master into your branch

Copy link
Member

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

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

See comment

sentry_sdk/integrations/django/__init__.py Outdated Show resolved Hide resolved
Copy link
Member

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

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

lgtm

@antonpirker antonpirker merged commit 684c43f into getsentry:master Jul 11, 2023
@BeryJu BeryJu deleted the django-404-handler branch July 11, 2023 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants