Skip to content
Snippets Groups Projects
Commit 0f48c033 authored by Bastien Abadie's avatar Bastien Abadie
Browse files

Merge branch 'bump-sentry' into 'master'

Bump sentry to 0.17.3 & activate django integration

Closes #393

See merge request !898
parents f47dca8b 4b723f6a
No related branches found
No related tags found
1 merge request!898Bump sentry to 0.17.3 & activate django integration
......@@ -34,7 +34,7 @@ def requires_s3_object(func):
"""
def wrapper(self, *args, **kwargs):
if not self.s3_object:
logger.warning(
logger.info(
'Function {} requires file to have s3_object property, skipping'
.format(func.__name__)
)
......
......@@ -530,11 +530,13 @@ except ImportError:
if SENTRY_DSN:
import sentry_sdk
from sentry_sdk.integrations.django import DjangoIntegration
sentry_sdk.init(
dsn=SENTRY_DSN,
environment=ARKINDEX_ENV,
release=f'arkindex-backend@{VERSION}',
debug=DEBUG,
send_default_pii=True
send_default_pii=True,
integrations=[DjangoIntegration()],
)
......@@ -35,7 +35,7 @@ class TestAWSMixin(TestCase):
self.assertEqual(s3_file.get_s3_object(), None)
self.assertEqual(s3_file.s3_put_url, None)
self.assertEqual(s3_file.s3_url, None)
self.assertEqual(logger_mock.warning.call_count, 2)
self.assertEqual(logger_mock.info.call_count, 2)
@patch('arkindex.project.aws.S3FileMixin.s3_object')
def test_check_hash(self, s3_object_mock):
......
......@@ -25,7 +25,7 @@ pytz>=2018.7
PyYAML==5.1
requests==2.22
responses==0.10.7
sentry-sdk==0.14.3
sentry-sdk==0.17.3
tenacity==6.2
transkribus-client>=0.1.1
git+https://gitlab.com/arkindex/transkribus.git#egg=transkribus-client
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment