-
Notifications
You must be signed in to change notification settings - Fork 0
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
Vulnerabilities and dependencies #374
Comments
Note: Python 3.13 is now out too but is not yet supported by Wagtail, Django, etc. In fact, neither is Python 3.12. We should stay with Python 3.11.
|
Since I first began investigating these vulnerabilities, a new critical one appeared:
But we're not even using |
Okay, I've fixed what I can; the remaining vulnerabilities are:
For these I have the following mitigation statements:
|
The remaining upgrades are "nice-to-haves":
|
@hoodriverheather @tloubrieu-jpl okay, I've made a ton of changes to support all of the above ↑ These security updates and modernization changes are now live on the dev portal. Because this affected lots of the core software (Django, Wagtail, etc.), there could be unexpected effects on various parts. I ran the automated tests and got all 258 passing. But of course, since I came up with the automated tests, they have my bias. If you could "kick the tires" and make sure your favorite parts of the portal are still working the way you expect, I'd appreciate it (@hoodriverheather you might especially look at the metadata forms and other forms, as the form machinery got updated). |
Hi folks, I'm going to go ahead and merge this into the The dev portal will be down for about 15 minutes |
@nutjob4life The LabCAS Documentation Page under Data and Resources/Informatics/ is not displaying the icons and giving me Not found error when I try to click on all links except - LabCAS EDRN Cancer Biomarker Data Commons (LabCAS) Data Link to page: |
@hoodriverheather that's to be expected; it's because that page is raw HTML and won't work anywhere except at NCI. We can fix this, but it's a lower priority item. |
@nutjob4life Sounds good. Just trying to click on most things. :) |
@nutjob4life Looks good to me. Nice job! |
@hoodriverheather thanks for being so thorough :relieved |
Reopening this because the fix to #380 was to undo a lot of these changes. |
The P5 portal uses a number of packages (and Python) itself which may be out-of-date or may have security vulnerabilities. A security scan revealed the following:
We don't have any critical vulnerabilities thankfully, but 11 "high" vulnerabilities have slipped by us and we need to address them ASAP.
It would also be nice to check to see if we can upgrade any of the following:
Postponing this for now:
The text was updated successfully, but these errors were encountered: