From 4aa20c72c19e197f322a176e9571f72282088cbc Mon Sep 17 00:00:00 2001
From: Ankush Menat <ankush@frappe.io>
Date: Sat, 24 Jun 2023 11:54:20 +0530
Subject: [PATCH] refactor: Simplify FrappeOAuth2Client

No need to override anything, use request.session in way it is intended.
---
 frappe/frappeclient.py | 39 +++++----------------------------------
 1 file changed, 5 insertions(+), 34 deletions(-)

diff --git a/frappe/frappeclient.py b/frappe/frappeclient.py
index 3f7577fac6..7ad016828a 100644
--- a/frappe/frappeclient.py
+++ b/frappe/frappeclient.py
@@ -4,8 +4,6 @@ FrappeClient is a library that helps you connect with other frappe systems
 import base64
 import json
 
-import requests
-
 import frappe
 from frappe.utils.data import cstr
 
@@ -37,6 +35,8 @@ class FrappeClient:
 		api_secret=None,
 		frappe_authorization_source=None,
 	):
+		import requests
+
 		self.headers = {
 			"Accept": "application/json",
 			"content-type": "application/x-www-form-urlencoded",
@@ -390,42 +390,13 @@ class FrappeClient:
 
 class FrappeOAuth2Client(FrappeClient):
 	def __init__(self, url, access_token, verify=True):
+		import requests
+
 		self.access_token = access_token
 		self.headers = {
 			"Authorization": "Bearer " + access_token,
 			"content-type": "application/x-www-form-urlencoded",
 		}
 		self.verify = verify
-		self.session = OAuth2Session(self.headers)
+		self.session = requests.session()
 		self.url = url
-
-	def get_request(self, params):
-		res = requests.get(
-			self.url, params=self.preprocess(params), headers=self.headers, verify=self.verify
-		)
-		res = self.post_process(res)
-		return res
-
-	def post_request(self, data):
-		res = requests.post(
-			self.url, data=self.preprocess(data), headers=self.headers, verify=self.verify
-		)
-		res = self.post_process(res)
-		return res
-
-
-class OAuth2Session:
-	def __init__(self, headers):
-		self.headers = headers
-
-	def get(self, url, params, verify):
-		res = requests.get(url, params=params, headers=self.headers, verify=verify)
-		return res
-
-	def post(self, url, data, verify):
-		res = requests.post(url, data=data, headers=self.headers, verify=verify)
-		return res
-
-	def put(self, url, data, verify):
-		res = requests.put(url, data=data, headers=self.headers, verify=verify)
-		return res