select_related added to introspect view (#834)

* select_related add to introspect

* flake8 fixes

* flake8 fix

* added author and change log

* test for select related

* test for select related

Author: anveshagarwal

AUTHORS

@@ -10,6 +10,7 @@ Contributors
 Abhishek Patel
 Alessandro De Angelis
 Alan Crosswell
+Anvesh Agarwal
 Asif Saif Uddin
 Ash Christopher
 Aristóbulo Meneses

CHANGELOG.md

@@ -14,6 +14,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Security
   -->
 
+## [unreleased] 
+
+### added
+* added `select_related` in intospect view for better query performance
+
+
 ## [1.3.2] 2020-03-24
 
 ### Fixed

oauth2_provider/views/introspect.py

@@ -24,7 +24,9 @@ class IntrospectTokenView(ClientProtectedScopedResourceView):
     @staticmethod
     def get_token_response(token_value=None):
         try:
-            token = get_access_token_model().objects.get(token=token_value)
+            token = get_access_token_model().objects.select_related(
+                "user", "application"
+                ).get(token=token_value)
         except ObjectDoesNotExist:
             return HttpResponse(
                 content=json.dumps({"active": False}),

tests/test_introspection_view.py

@@ -321,3 +321,7 @@ def test_view_post_invalid_client_creds_plaintext(self):
              "client_id": self.application.client_id,
              "client_secret": self.application.client_secret + "_so_wrong"})
         self.assertEqual(response.status_code, 403)
+
+    def test_select_related_in_view_for_less_db_queries(self):
+        with self.assertNumQueries(1):
+            self.client.post(reverse("oauth2_provider:introspect"))