Environment SaaS ( http

Assigning to @getsentry/support for <a href="https://open.sentry.io/triage/#2-route" r

Routing to @getsentry/product-owners-settings-integrations for <a href="https://develo

Webhook event data is generated here <div

OpsGenie Webhook missing attached tags about sentry HOT 3 OPEN

dachakra commented on August 24, 2024

OpsGenie Webhook missing attached tags

from sentry.

Comments (3)

getsantry commented on August 24, 2024

Assigning to @getsentry/support for routing ⏲️

from sentry.

getsantry commented on August 24, 2024

Routing to @getsentry/product-owners-settings-integrations for triage ⏲️

from sentry.

cathteng commented on August 24, 2024

Webhook event data is generated here

sentry/src/sentry/tasks/sentry_apps.py

Lines 60 to 83 in fdbcd3a

 def _webhook_event_data(event, group_id, project_id): 

 project = Project.objects.get_from_cache(id=project_id) 

 organization = Organization.objects.get_from_cache(id=project.organization_id) 

 event_context = event.as_dict() 

 event_context["url"] = absolute_uri( 

 reverse( 

 "sentry-api-0-project-event-details", 

 args=[project.organization.slug, project.slug, event.event_id], 

 ) 

 ) 

 event_context["web_url"] = absolute_uri( 

 reverse( 

 "sentry-organization-event-detail", args=[organization.slug, group_id, event.event_id] 

 ) 

 ) 

 # The URL has a regex OR in it ("|") which means `reverse` cannot generate 

 # a valid URL (it can't know which option to pick). We have to manually 

 # create this URL for, that reason. 

 event_context["issue_url"] = absolute_uri(f"/api/0/issues/{group_id}/") 

 event_context["issue_id"] = str(group_id) 

 return event_context

We call event.as_dict() to get the event information

sentry/src/sentry/eventstore/models.py

Lines 541 to 568 in fdbcd3a

 def as_dict(self) -> Mapping[str, Any]: 

 """Returns the data in normalized form for external consumers.""" 

 data: MutableMapping[str, Any] = {} 

 data["event_id"] = self.event_id 

 data["project"] = self.project_id 

 data["release"] = self.release 

 data["dist"] = self.dist 

 data["platform"] = self.platform 

 data["message"] = self.message 

 data["datetime"] = self.datetime 

 data["tags"] = [(k.split("sentry:", 1)[-1], v) for (k, v) in self.tags] 

 for k, v in sorted(self.data.items()): 

 if k in data: 

 continue 

 if k == "sdk": 

 v = {v_k: v_v for v_k, v_v in v.items() if v_k != "client_ip"} 

 data[k] = v 

 # for a long time culprit was not persisted. In those cases put 

 # the culprit in from the group. 

 if data.get("culprit") is None and self.group_id and self.group: 

 data["culprit"] = self.group.culprit 

 # Override title and location with dynamically generated data 

 data["title"] = self.title 

 data["location"] = self.location 

 return data

, which includes all of the tags.

This fetches tags from Snuba with a fallback on Nodestore

sentry/src/sentry/eventstore/models.py

Lines 130 to 157 in fdbcd3a

 @property 

 def tags(self) -> Sequence[tuple[str, str]]: 

 """ 

  Tags property uses tags from snuba if loaded otherwise falls back to 

  nodestore. 

  """ 

 tags_key_column = self._get_column_name(Columns.TAGS_KEY) 

 tags_value_column = self._get_column_name(Columns.TAGS_VALUE) 

 if tags_key_column in self._snuba_data and tags_value_column in self._snuba_data: 

 keys = self._snuba_data[tags_key_column] 

 values = self._snuba_data[tags_value_column] 

 if keys and values and len(keys) == len(values): 

 return sorted(zip(keys, values)) 

 else: 

 return [] 

 # Nodestore implementation 

 try: 

 rv = sorted( 

 (t, v) 

 for t, v in get_path(self.data, "tags", filter=True) or () 

 if t is not None and v is not None 

 ) 

 return rv 

 except ValueError: 

 # at one point Sentry allowed invalid tag sets such as (foo, bar) 

 # vs ((tag, foo), (tag, bar)) 

 return []

If tags are not appearing in the webhook data, that would appear to be some problem with Snuba or Nodestore as the integration does not manipulate (remove) the tags in any way besides formatting them.

from sentry.

OpsGenie Webhook missing attached tags about sentry HOT 3 OPEN

Comments (3)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	def _webhook_event_data(event, group_id, project_id):
	project = Project.objects.get_from_cache(id=project_id)
	organization = Organization.objects.get_from_cache(id=project.organization_id)

	event_context = event.as_dict()
	event_context["url"] = absolute_uri(
	reverse(
	"sentry-api-0-project-event-details",
	args=[project.organization.slug, project.slug, event.event_id],
	)
	)

	event_context["web_url"] = absolute_uri(
	reverse(
	"sentry-organization-event-detail", args=[organization.slug, group_id, event.event_id]
	)
	)

	# The URL has a regex OR in it ("\|") which means `reverse` cannot generate
	# a valid URL (it can't know which option to pick). We have to manually
	# create this URL for, that reason.
	event_context["issue_url"] = absolute_uri(f"/api/0/issues/{group_id}/")
	event_context["issue_id"] = str(group_id)
	return event_context

	def as_dict(self) -> Mapping[str, Any]:
	"""Returns the data in normalized form for external consumers."""
	data: MutableMapping[str, Any] = {}
	data["event_id"] = self.event_id
	data["project"] = self.project_id
	data["release"] = self.release
	data["dist"] = self.dist
	data["platform"] = self.platform
	data["message"] = self.message
	data["datetime"] = self.datetime
	data["tags"] = [(k.split("sentry:", 1)[-1], v) for (k, v) in self.tags]
	for k, v in sorted(self.data.items()):
	if k in data:
	continue
	if k == "sdk":
	v = {v_k: v_v for v_k, v_v in v.items() if v_k != "client_ip"}
	data[k] = v

	# for a long time culprit was not persisted. In those cases put
	# the culprit in from the group.
	if data.get("culprit") is None and self.group_id and self.group:
	data["culprit"] = self.group.culprit

	# Override title and location with dynamically generated data
	data["title"] = self.title
	data["location"] = self.location

	return data

	@property
	def tags(self) -> Sequence[tuple[str, str]]:
	"""
	Tags property uses tags from snuba if loaded otherwise falls back to
	nodestore.
	"""
	tags_key_column = self._get_column_name(Columns.TAGS_KEY)
	tags_value_column = self._get_column_name(Columns.TAGS_VALUE)

	if tags_key_column in self._snuba_data and tags_value_column in self._snuba_data:
	keys = self._snuba_data[tags_key_column]
	values = self._snuba_data[tags_value_column]
	if keys and values and len(keys) == len(values):
	return sorted(zip(keys, values))
	else:
	return []
	# Nodestore implementation
	try:
	rv = sorted(
	(t, v)
	for t, v in get_path(self.data, "tags", filter=True) or ()
	if t is not None and v is not None
	)
	return rv
	except ValueError:
	# at one point Sentry allowed invalid tag sets such as (foo, bar)
	# vs ((tag, foo), (tag, bar))
	return []