From fee8e17f950997ebe2aeff159656e469139b36d6 Mon Sep 17 00:00:00 2001 From: Christie Rice <8483753+crice100@users.noreply.github.com> Date: Fri, 19 Jul 2019 10:46:46 -0400 Subject: [PATCH] REVMI-341 Stop sending lms_client_id in jwt sent to ecommerce (#21167) --- lms/djangoapps/commerce/tests/__init__.py | 4 +--- openedx/core/djangoapps/commerce/utils.py | 1 - 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/lms/djangoapps/commerce/tests/__init__.py b/lms/djangoapps/commerce/tests/__init__.py index c7b92328ab..f842cdf53c 100644 --- a/lms/djangoapps/commerce/tests/__init__.py +++ b/lms/djangoapps/commerce/tests/__init__.py @@ -28,7 +28,6 @@ TEST_PAYMENT_DATA = { class EdxRestApiClientTest(TestCase): """ Tests to ensure the client is initialized properly. """ - TEST_CLIENT_ID = 'test-client-id' SCOPES = [ 'user_id', 'email', @@ -55,7 +54,7 @@ class EdxRestApiClientTest(TestCase): ) mock_tracker = mock.Mock() - mock_tracker.resolve_context = mock.Mock(return_value={'client_id': self.TEST_CLIENT_ID, 'ip': '127.0.0.1'}) + mock_tracker.resolve_context = mock.Mock(return_value={'ip': '127.0.0.1'}) with mock.patch('openedx.core.djangoapps.commerce.utils.tracker.get_tracker', return_value=mock_tracker): ecommerce_api_client(self.user).baskets(1).post() @@ -65,7 +64,6 @@ class EdxRestApiClientTest(TestCase): claims = { 'tracking_context': { 'lms_user_id': self.user.id, - 'lms_client_id': self.TEST_CLIENT_ID, 'lms_ip': '127.0.0.1', } } diff --git a/openedx/core/djangoapps/commerce/utils.py b/openedx/core/djangoapps/commerce/utils.py index 2249657dbd..3bde30d588 100644 --- a/openedx/core/djangoapps/commerce/utils.py +++ b/openedx/core/djangoapps/commerce/utils.py @@ -18,7 +18,6 @@ def create_tracking_context(user): return { 'lms_user_id': user.id, - 'lms_client_id': context_tracker.get('client_id'), 'lms_ip': context_tracker.get('ip'), }