From fdfbe5b73455185b960279eaa7bbdcfbc83019ae Mon Sep 17 00:00:00 2001
From: Zachary Hancock <zhancock@edx.org>
Date: Mon, 22 Jan 2024 09:53:51 -0500
Subject: [PATCH] feat: add newrelic tracing (#439)

---
 CHANGELOG.rst                    | 3 +++
 lti_consumer/__init__.py         | 2 +-
 lti_consumer/lti_1p3/consumer.py | 2 ++
 lti_consumer/models.py           | 3 +++
 4 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
index 57c0e1bc..f6fe13f0 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -15,6 +15,9 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
 
 Unreleased
 ~~~~~~~~~~
+9.8.2 - 2024-01-19
+------------------
+* Add NewRelic traces to functions suspected of causing performance issues.
 
 9.8.1 - 2023-11-17
 ------------------
diff --git a/lti_consumer/__init__.py b/lti_consumer/__init__.py
index 2d5f2787..02aa0744 100644
--- a/lti_consumer/__init__.py
+++ b/lti_consumer/__init__.py
@@ -4,4 +4,4 @@
 from .apps import LTIConsumerApp
 from .lti_xblock import LtiConsumerXBlock
 
-__version__ = '9.8.1'
+__version__ = '9.8.2'
diff --git a/lti_consumer/lti_1p3/consumer.py b/lti_consumer/lti_1p3/consumer.py
index cdb55dea..9d4a099e 100644
--- a/lti_consumer/lti_1p3/consumer.py
+++ b/lti_consumer/lti_1p3/consumer.py
@@ -5,6 +5,7 @@
 from urllib.parse import urlencode
 import uuid
 
+from edx_django_utils.monitoring import function_trace
 from django.conf import settings
 
 from lti_consumer.lti_1p3.exceptions import InvalidClaimValue
@@ -103,6 +104,7 @@ def _get_user_roles(role):
 
         return list(lti_user_roles)
 
+    @function_trace('lti_consumer.lti_1p3.consumer.prepare_preflight_url')
     def prepare_preflight_url(
             self,
             launch_data,
diff --git a/lti_consumer/models.py b/lti_consumer/models.py
index 0f39e212..6cb35f3c 100644
--- a/lti_consumer/models.py
+++ b/lti_consumer/models.py
@@ -17,6 +17,7 @@
 from config_models.models import ConfigurationModel
 from django.utils.functional import cached_property
 from django.utils.translation import gettext_lazy as _
+from edx_django_utils.monitoring import function_trace
 from lti_consumer.filters import get_external_config_from_filter
 
 # LTI 1.1
@@ -280,6 +281,7 @@ def clean(self):
         if consumer is None:
             raise ValidationError(_("Invalid LTI configuration."))
 
+    @function_trace('lti_consumer.models.LtiConfiguration.sync_configurations')
     def sync_configurations(self):
         """Syncronize main/children configurations.
 
@@ -611,6 +613,7 @@ def _get_lti_1p3_consumer(self):
 
         return consumer
 
+    @function_trace('lti_consumer.models.LtiConfiguration.get_lti_consumer')
     def get_lti_consumer(self):
         """
         Returns an instanced class of LTI 1.1 or 1.3 consumer.