Skip to content

Commit b0f14cd

Browse files
committed
add logging
1 parent 9a375e5 commit b0f14cd

File tree

2 files changed

+1439
-3
lines changed

2 files changed

+1439
-3
lines changed

src/sentry/mediators/sentry_apps/creator.py

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
from __future__ import absolute_import
22

33
import six
4+
import logging
45

56
from collections import Iterable
7+
from django.db import IntegrityError
68

79
from sentry import analytics
810
from sentry.mediators import Mediator, Param
@@ -15,6 +17,8 @@
1517
User,
1618
)
1719

20+
logger = logging.getLogger('sentry.mediators.sentry-apps')
21+
1822

1923
class Creator(Mediator):
2024
name = Param(six.string_types)
@@ -80,9 +84,18 @@ def _create_ui_components(self):
8084
def _create_integration_feature(self):
8185
# sentry apps must have at least one feature
8286
# defaults to 'integrations-api'
83-
IntegrationFeature.objects.create(
84-
sentry_app=self.sentry_app,
85-
)
87+
try:
88+
IntegrationFeature.objects.create(
89+
sentry_app=self.sentry_app,
90+
)
91+
except IntegrityError as e:
92+
logger.info(
93+
'creator.error',
94+
extra={
95+
'sentry_app': self.sentry_app.slug,
96+
'error_message': e.message,
97+
}
98+
)
8699

87100
def audit(self):
88101
from sentry.utils.audit import create_audit_entry

0 commit comments

Comments
 (0)