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

jQuery 1.x is End of Life and does not receive patches #4396

Closed
tonygermano opened this issue Jun 16, 2020 · 5 comments
Closed

jQuery 1.x is End of Life and does not receive patches #4396

tonygermano opened this issue Jun 16, 2020 · 5 comments
Labels
Fix-Commited Issue fixed and will be available in milestone Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-5473 triaged
Milestone

Comments

@tonygermano
Copy link
Collaborator

Recently #4300 upgraded jQuery to version 1.12.4 in Connect version 3.9, however, the 1.x branch of jQuery has already been EoL for about 3 years according to jquery/jquery.com#162

@MichaelLeeHobbs
Copy link

This is showing up on vul scans now. Recommend JQuery 3.5 per https://snyk.io/test/npm/jquery/1.12.4

@jonbartels
Copy link
Contributor

jonbartels commented Jul 27, 2020

A search for "min.js" to find references to the jqeuery.min.js files is: https://github.com/nextgenhealthcare/connect/search?q=%22.min.js%22&unscoped_q=%22.min.js%22

It shows references at

  • the Swagger UI (so either have to ensure swagger works with Jquery 3.5 or update swagger)
  • the MirthTagBrowser which IIRC is used for the fuzzy matching and selection of tags in list pages in the MR UI

I would expect JQuery to be generally backward compatible but those two use cases would have to be evaluated to make sure it still works. Since its UI/UX related I don't see a good way to automate the testing either.

@jonbartels
Copy link
Contributor

The user "naql" in Slack applied the manual workaround. He reported on 2020-07-29 in a thread in #general that:

I first tried swapping out with 1.12.4, but the scan didn't like that either. I put in the most recent 3.5.1 and it seemed to work. Follow the instructions on the ticket: delete the webadmin war, replace the jquery-1.x-min.js file with the 3.5 one in the public_html and the public_api_html . I didn't bother with the other jquery files in the latter. Just left them alone. Edit references in both index pages, reboot mirth. Passed the scan, I can still log into it with mirth admin, button to launch admin console from web page still works, swagger page displays, but I did not test it very thoroughly. Should not affect use of the api.

@baileyglen
Copy link

I am running into this issue as well. Has any progress been made on upgrading the version?

@cturczynskyj
Copy link
Collaborator

Thank you for reporting this. We've added an issue to our backlog to look into updating jQuery (again).

@cturczynskyj cturczynskyj added Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-5473 labels Jan 22, 2021
@pladesma pladesma added the Fix-Commited Issue fixed and will be available in milestone label Apr 15, 2021
@pladesma pladesma added this to the 3.11.0 milestone Apr 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix-Commited Issue fixed and will be available in milestone Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-5473 triaged
Projects
None yet
Development

No branches or pull requests

6 participants