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

Snyk May Review #7702

Merged
merged 16 commits into from
Jun 5, 2024
Merged

Snyk May Review #7702

merged 16 commits into from
Jun 5, 2024

Conversation

shanice-skylight
Copy link
Collaborator

@shanice-skylight shanice-skylight commented May 13, 2024

BACKEND PULL REQUEST

Changes Proposed

  • Update packages in build.gradle (backend).

Testing

A smoke test of basic functionality should be accomplished before merging.
This PR has already been verified in dev4.

@shanice-skylight shanice-skylight marked this pull request as ready for review May 17, 2024 14:39
@alismx alismx requested review from a team, mehansen, mpbrown and fzhao99 and removed request for a team June 4, 2024 15:37
Copy link

sonarcloud bot commented Jun 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor

@fzhao99 fzhao99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested on dev4 and lgtm!

Copy link
Collaborator

@emyl3 emyl3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Smoke tested on dev4 -- LGTM! Thank you for all your work on this @shanice-skylight

Copy link
Collaborator

@mpbrown mpbrown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks Shanice!

@shanice-skylight shanice-skylight added this pull request to the merge queue Jun 5, 2024
Merged via the queue into main with commit 5fdc224 Jun 5, 2024
40 checks passed
@shanice-skylight shanice-skylight deleted the shanice/7679-snyk branch June 5, 2024 19:19
shanice-skylight added a commit that referenced this pull request Jun 26, 2024
* fixed 89 & 284

* fixed 287 & 770

* testing modified package back to original

* fixed backages error

* updated all high and critical

* modified runtime class

* modified liquibase version to 4.20.0 to fix error

* updated liquibase version

* updated liquibase to 4.27.0

* upgrade libraries

* added to build.gradle

* modified source Compatibility to fix Deprecated Gradle features error

* modified to use the java { }

* change source version back to original

* changed org.postgresql:postgresql back to version 42.6.1 to fix vulnerability
This was referenced Jul 1, 2024
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.

4 participants