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.

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

updated asn1-1.0.13 to 1.0.28 from jsrsasign. #307

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

omkarDev0
Copy link

removed yahoo.js as it contains yui 2.9.0 which has Vulnerabilities. added base64x-1.1.js file from jsrsasign with functions which are necessary to use for replacing yahoo.js.

removed yahoo.js as it contains yui 2.9.0 which has Vulnerabilities.
added base64x-1.1.js file from jsrsasign with functions which are necessary to use for replacing yahoo.js.
@travist
Copy link
Owner

travist commented Apr 7, 2024

This looks like great work. If possible I would like to see some automation tests to ensure the validity of the replacement of yahoo.js. If you provide that, then I would be happy to merge.

@omkarDev0
Copy link
Author

@travist I have updated test cases for extendClass function to ensure the validity of the replacement of yahoo.js
kindly review it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants