-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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(stacketrace-link): Fix tagging and tag early adopters #41648
Conversation
Fixes: * `stacktrace_link.found` is containing the wrong value * All code mappings are tried even if `config.stack_root` does not match Changes: * Tag if the organization is an `early_adopter`
Changes: * fix: `stacktrace_link.found` is storing the wrong value * feature: Tag early adopter organizations
scope.set_tag("project.slug", project.slug) | ||
scope.set_tag("organization.slug", project.organization.slug) | ||
try: | ||
scope.set_tag("organization.early_adopter", project.organization.flags.early_adopter) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the new addition.
scope.set_tag("organization.early_adopter", project.organization.flags.early_adopter) | ||
except Exception: | ||
# If errors arise we can then follow up with a fix | ||
logger.exception("We failed to set the early adopter flag") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this was to cause errors I would see it and then fix it.
@@ -95,7 +108,7 @@ class ProjectStacktraceLinkEndpoint(ProjectEndpoint): | |||
users can go from the file in the stack trace to the | |||
provider of their choice. | |||
|
|||
`filepath`: The file path from the stack trace | |||
`file`: The file path from the stack trace |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The parameter the UI uses is file
rather than filepath
.
@@ -166,7 +178,7 @@ def get(self, request: Request, project) -> Response: | |||
break | |||
|
|||
# Post-processing before exiting scope context | |||
found = result.get("sourceUrl") | |||
found: bool = result["sourceUrl"] is not None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We were storing the URL rather than a boolean.
Changes:
stacktrace_link.found
is storing the wrong value