Skip to content

Fix dependencies with security vulnerabilities#13255

Merged
vrajmohan merged 2 commits into
mainfrom
vm-security-audit
Jul 9, 2026
Merged

Fix dependencies with security vulnerabilities#13255
vrajmohan merged 2 commits into
mainfrom
vm-security-audit

Conversation

@vrajmohan

@vrajmohan vrajmohan commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

changelog: Internal, Security, Fix dependencies with security vulnerabilities

  1. Update json
  2. Mark webrick CVE in bundle audit as it is used only for testing:
    • in capybara_mock
    • in prometheus_export (to be verified)

changelog: Internal, Security, Fix dependencies with security vulnerabilities

1. Update json
2. Mark webrick CVE in bundle audit as it is used only for testing:
    - in capybara_mock
    - in prometheus_export (to be verified)
@vrajmohan
vrajmohan requested a review from jgrevich July 8, 2026 18:46
@jgrevich

jgrevich commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

We don't use the prometheus exporter anywhere in production at this time. Is there not a patch for it yet? I think it's fine to add the ignore for now, but we should remove it ASAP.

@vrajmohan

Copy link
Copy Markdown
Contributor Author

The issue is really with webrick. There is no fix right now.

@vrajmohan
vrajmohan merged commit e9e3ab4 into main Jul 9, 2026
1 check passed
@vrajmohan
vrajmohan deleted the vm-security-audit branch July 9, 2026 20:40
@jgrevich

Copy link
Copy Markdown
Contributor

Yep, I get that. Unfortunately, prometheus_exporter does use the webrick ruby server for 'production' use. At least it's never meant to be public facing, but still seems like a poor choice given the other ruby web server options: ruby/webrick#199 (comment). ruby/webrick#199 (comment) explains why even non-user-facing use of webrick should be discouraged.

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.

3 participants