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

upgrade flyway v9.17.0 #5436

Merged
merged 1 commit into from
May 8, 2023
Merged

Conversation

tmpayton
Copy link
Contributor

@tmpayton tmpayton commented May 5, 2023

Summary (required)

This ticket upgrades flyway to the latest version to remove security vulnerabilities.

Required reviewers - 1 Developer

Impacted areas of the application

General components of the application that this PR will affect:

  • flyway migrations
  • readme

How to test

  1. Checkout this branch
  2. Uninstall flyway brew uninstall flyway
  3. Install the latest version of flyway:
    Method One: brew install flyway
    Method Two: Follow the readme to install flyway version 9.17.0 manually
  4. Confirm version 9.17.0 was downloaded flyway -v
  5. Find your flyway.conf file in your Cellar and add URL and locations
  6. dropdb cfdm_test
  7. createdb cfdm_test
  8. invoke create_sample_db
  9. pytest
  10. flask run (check endpoints)
  11. Runsnyk test --file=data/flyway/build.gradle and confirm vulnerabilities are gone

@codecov
Copy link

codecov bot commented May 5, 2023

Codecov Report

Merging #5436 (45a94ee) into develop (0a4029d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #5436   +/-   ##
========================================
  Coverage    85.44%   85.44%           
========================================
  Files           81       81           
  Lines         8157     8157           
========================================
  Hits          6970     6970           
  Misses        1187     1187           

@fec-jli
Copy link
Contributor

fec-jli commented May 8, 2023

Screen Shot 2023-05-08 at 10 45 37 AM

Copy link
Contributor

@fec-jli fec-jli left a comment

Choose a reason for hiding this comment

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

Work as expected. Thanks

@fec-jli fec-jli merged commit 194fcb0 into develop May 8, 2023
@tmpayton tmpayton deleted the feature/5417-upgrade-flyway-v9.17.0 branch July 20, 2023 13:57
@tmpayton tmpayton restored the feature/5417-upgrade-flyway-v9.17.0 branch September 18, 2023 19:23
@tmpayton tmpayton deleted the feature/5417-upgrade-flyway-v9.17.0 branch July 22, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Snyk:High] net.minidev:json-smart Denial of Service (DoS)(due by 05/31/2023)
2 participants