Skip to content
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

feat: additional performance tracing #443

Merged
merged 1 commit into from
Jan 23, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel

Unreleased
~~~~~~~~~~
9.8.3 - 2024-01-23
------------------
* Additional NewRelic traces to functions suspected of causing performance issues.

9.8.2 - 2024-01-19
------------------
* Add NewRelic traces to functions suspected of causing performance issues.
Expand Down
2 changes: 1 addition & 1 deletion lti_consumer/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@
from .apps import LTIConsumerApp
from .lti_xblock import LtiConsumerXBlock

__version__ = '9.8.2'
__version__ = '9.8.3'
1 change: 1 addition & 0 deletions lti_consumer/lti_1p3/consumer.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ class LtiConsumer1p3:
LTI 1.3 Consumer Implementation
"""

@function_trace('lti_consumer.lti_1p3.consumer.LtiConsumer1p3.__init__')
def __init__(
self,
iss,
Expand Down
3 changes: 3 additions & 0 deletions lti_consumer/lti_1p3/key_handlers.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
import logging

from Cryptodome.PublicKey import RSA
from edx_django_utils.monitoring import function_trace
from jwkest import BadSignature, BadSyntax, WrongNumberOfParts, jwk
from jwkest.jwk import RSAKey, load_jwks_from_url
from jwkest.jws import JWS, NoSuitableSigningKeys, UnknownAlgorithm
Expand All @@ -32,6 +33,7 @@ class ToolKeyHandler:
in order to validate the JWT Signature of messages
signed with the tools signature.
"""
@function_trace('lti_consumer.key_handlers.ToolKeyHandler.__init__')
def __init__(self, public_key=None, keyset_url=None):
"""
Instance message validator
Expand Down Expand Up @@ -162,6 +164,7 @@ class PlatformKeyHandler:
This class loads the platform key and is responsible for
encoding JWT messages and exporting public keys.
"""
@function_trace('lti_consumer.key_handlers.PlatformKeyHandler.__init__')
def __init__(self, key_pem, kid=None):
"""
Import Key when instancing class if a key is present.
Expand Down
Loading