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.

Sign up for GitHub

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

fix(india): 401 & 403 client error while generating IRN (backport #30757) #30773

Merged
merged 4 commits into from
Apr 22, 2022
Merged

fix(india): 401 & 403 client error while generating IRN (backport #30757) #30773

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
198bdcf
fix(india): 401 & 403 client error while generating IRN
nextchamp-saqib Apr 20, 2022
712443a
chore: Resolve conflicts
deepeshgarg007 Apr 22, 2022
a074d12
Merge branch 'version-13-hotfix' into mergify/bp/version-13-hotfix/pr…
deepeshgarg007 Apr 22, 2022
6dddbb9
fix: linter
nextchamp-saqib Apr 22, 2022
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
20 changes: 16 additions & 4 deletions erpnext/regional/india/e_invoice/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@

import frappe
import jwt
import requests
import six
from frappe import _, bold
from frappe.core.page.background_jobs.background_jobs import get_info
Expand Down Expand Up @@ -828,14 +829,25 @@ def get_auth_token(self):
return self.e_invoice_settings.auth_token

def make_request(self, request_type, url, headers=None, data=None):
if request_type == "post":
res = make_post_request(url, headers=headers, data=data)
else:
res = make_get_request(url, headers=headers, data=data)
try:
if request_type == "post":
res = make_post_request(url, headers=headers, data=data)
else:
res = make_get_request(url, headers=headers, data=data)

except requests.exceptions.HTTPError as e:
if e.response.status_code in [401, 403] and not hasattr(self, "token_auto_refreshed"):
self.auto_refresh_token()
headers = self.get_headers()
return self.make_request(request_type, url, headers, data)

self.log_request(url, headers, data, res)
return res

def auto_refresh_token(self):
self.fetch_auth_token()
self.token_auto_refreshed = True

def log_request(self, url, headers, data, res):
headers.update({"password": self.credentials.password})
request_log = frappe.get_doc(
Expand Down