Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: use later aws sdk version in configTools #407

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Conversation

tjsilver
Copy link
Contributor

@tjsilver tjsilver commented Feb 7, 2024

What is the purpose of this change?

Makes the configTools project use the same version of aws-sdk as the main app.

What is the value of this change and how do we measure success?

This is an attempt to mitigate a high severity vulnerability discovered by Dependabot, whereby configTools' reliance on aws-scala which has been deprecated, and which in turn relies on an old version of aws-java-sdk-s3. Once this is merged, if successful we should see the issue disappear.

I have tested this locally and the dependency tree for configTools shows that the vulnerable version has been replaced a safe version:

Before:
image

After:
image

@tjsilver tjsilver changed the title use later aws sdk version in configTools fix: use later aws sdk version in configTools Feb 7, 2024
@tjsilver tjsilver force-pushed the ts/remove-high-vuln branch from 1e6ff10 to 1099403 Compare February 7, 2024 15:44
@tjsilver tjsilver requested review from a team February 7, 2024 15:47
@tjsilver
Copy link
Contributor Author

tjsilver commented Feb 7, 2024

Closing for now to see if Dependabot raises a PR

@tjsilver tjsilver closed this Feb 7, 2024
@tjsilver tjsilver reopened this Feb 7, 2024
@tjsilver
Copy link
Contributor Author

tjsilver commented Feb 7, 2024

Re-opening as Dependabot didn't raise a PR to fix this.

@tjsilver tjsilver merged commit c11d278 into main Feb 8, 2024
7 checks passed
@tjsilver tjsilver deleted the ts/remove-high-vuln branch February 8, 2024 09:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants