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

Jump to bottom

fix: e-invoice jwt verification error #37818

Merged
merged 1 commit into from
Nov 1, 2023
Merged

fix: e-invoice jwt verification error #37818

merged 1 commit into from
Nov 1, 2023

Conversation

rtdany10
Copy link
Contributor

@rtdany10 rtdany10 commented Nov 1, 2023 •
edited
Loading 

Exception:
Traceback (most recent call last):
  File "/home/frappe/frappe-bench/apps/erpnext/erpnext/regional/india/e_invoice/utils.py", line 1020, in generate_irn
    self.set_einvoice_data(irn_details)
  File "/home/frappe/frappe-bench/apps/erpnext/erpnext/regional/india/e_invoice/utils.py", line 1381, in set_einvoice_data
    dec_signed_invoice = jwt.decode(enc_signed_invoice, verify=False)["data"]
  File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/jwt/api_jwt.py", line 129, in decode
    decoded = self.decode_complete(jwt, key, algorithms, options, **kwargs)
  File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/jwt/api_jwt.py", line 97, in decode_complete
    'It is required that you pass in a value for the "algorithms" argument when calling decode().'
jwt.exceptions.DecodeError: It is required that you pass in a value for the "algorithms" argument when calling decode()

@ankush
Copy link
Member

ankush commented Nov 1, 2023

@Mergifyio backport version-13

@mergify Mergify
Copy link
Contributor

mergify bot commented Nov 1, 2023 •
edited
Loading

backport version-13

✅ Backports have been created

@ankush ankush merged commit 8f4ded6 into frappe:version-13-hotfix Nov 1, 2023
8 checks passed
mergify bot pushed a commit that referenced this pull request Nov 1, 2023
@rtdany10 @mergify
fix: e-invoice jwt verification error (#37818)
79e631e 
(cherry picked from commit 8f4ded6)
@mergify mergify bot mentioned this pull request Nov 1, 2023
Merged
ankush pushed a commit that referenced this pull request Nov 1, 2023
@mergify @rtdany10
fix: e-invoice jwt verification error (backport #37818) (#37820)
428f7a6 
fix: e-invoice jwt verification error (#37818)

(cherry picked from commit 8f4ded6)

Co-authored-by: Dany Robert <[email protected]>
frappe-pr-bot pushed a commit that referenced this pull request Nov 1, 2023
@frappe-bot
chore(release): Bumped to Version 13.54.9
c84384a 
## [13.54.9](v13.54.8...v13.54.9) (2023-11-01)

### Bug Fixes

* e-invoice jwt verification error (backport [#37818](#37818)) ([#37820](#37820)) ([428f7a6](428f7a6))
frappe-pr-bot pushed a commit that referenced this pull request Nov 8, 2023
@frappe-bot
chore(release): Bumped to Version 13.54.10
6a53e82 
## [13.54.10](v13.54.9...v13.54.10) (2023-11-08)

### Bug Fixes

* add missing disbursement account in update_old_loans patch ([37b1a0e](37b1a0e))
* bump pygithub requirement to handle conflict ([#37800](#37800)) ([13d5eec](13d5eec))
* e-invoice jwt verification error ([#37818](#37818)) ([8f4ded6](8f4ded6))
* permission error while creating Supplier Quotation from Portal (backport [#37864](#37864)) ([#37872](#37872)) ([6952f0f](6952f0f))
* Quality Inspection Parameter migration - DuplicateEntryError due to case sensitivity (backport [#37499](#37499)) ([#37918](#37918)) ([4789eca](4789eca))
* remove voucher type and no for Item and Warehouse based reposting (backport [#37849](#37849)) (backport [#37850](#37850)) ([#37938](#37938)) ([8cb0f69](8cb0f69))
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 13.54.10 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ankush ankush ankush approved these changes

@deepeshgarg007 deepeshgarg007 Awaiting requested review from deepeshgarg007 deepeshgarg007 is a code owner

@ruthra-kumar ruthra-kumar Awaiting requested review from ruthra-kumar ruthra-kumar is a code owner

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rtdany10 @ankush @frappe-pr-bot