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

(maint) Pin bundler to 2.4.22, which supports Ruby 2.7.4 #138

Merged
merged 3 commits into from
Feb 5, 2024
Merged

Conversation

Magisus
Copy link
Contributor

@Magisus Magisus commented Feb 5, 2024

We currently pin our Ruby version in this Dockerfile to 2.7.4, so we also need to pin bundler to a version compatible with that. Updating the Ruby version is a bigger project that would need to be looked into separately.

We currently pin our Ruby version in this Dockerfile to 2.7.4, so we
also need to pin bundler to a version compatible with that. Updating the
Ruby version is a bigger project that would need to be looked into
separately.
@Magisus Magisus requested a review from a team as a code owner February 5, 2024 20:59
This deps file contains an RSA key that was used to for GPG signing of
one of the deps. Trivy is incorrectly flagging it as a leaked AWS key.
All future versions of nokogiri contain this key in their deps file, so
we are just skipping the file when doing the Trivy scan, since it is not
a real security issue.
One of the files in the aws-sdk-core gem contains some comments with
tokens in them. This is causing our Trivy scan to fail. This commit
skips scanning that file, since these tokens aren't a real security
issue.
@Magisus Magisus merged commit 10993e4 into main Feb 5, 2024
2 checks passed
@Magisus Magisus deleted the pin-bundler branch February 5, 2024 21:55
@Magisus Magisus restored the pin-bundler branch February 5, 2024 22:12
@Magisus Magisus deleted the pin-bundler branch February 5, 2024 22:13
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.

2 participants