Skip to content

Don't override @opts in create_client_from_token#168

Merged
agrare merged 1 commit intofog:masterfrom
evgeni:ssl
Jan 28, 2026
Merged

Don't override @opts in create_client_from_token#168
agrare merged 1 commit intofog:masterfrom
evgeni:ssl

Conversation

@evgeni
Copy link
Contributor

@evgeni evgeni commented Jan 28, 2026

@opts is already correctly set in initialize and if we override it, we disable SSL validation

https://bugzilla.redhat.com/show_bug.cgi?id=CVE-2026-1530

Fixes: efbeed4
Fixes: CVE-2026-1530

`@opts` is already correctly set in `initialize` and if we override it,
we disable SSL validation

Fixes: efbeed4
Fixes: CVE-2026-1530
@evgeni
Copy link
Contributor Author

evgeni commented Jan 28, 2026

I'd also argue that kubevirt_verify_ssl should default to true

@agrare agrare self-assigned this Jan 28, 2026
@agrare agrare added the bug Something isn't working label Jan 28, 2026
@agrare
Copy link
Collaborator

agrare commented Jan 28, 2026

I'd also argue that kubevirt_verify_ssl should default to true

I agree, I'm going to release a 1.5.1 with just this commit but if you want to make that change I'll do that in a 1.6.0 since it is a change in behavior

@agrare agrare merged commit 363e3da into fog:master Jan 28, 2026
4 checks passed
@evgeni evgeni deleted the ssl branch January 28, 2026 13:52
@evgeni
Copy link
Contributor Author

evgeni commented Jan 28, 2026

Sure, thanks!

Second PR: #169

@agrare
Copy link
Collaborator

agrare commented Jan 28, 2026

Fixed in https://rubygems.org/gems/fog-kubevirt/versions/1.5.1

agrare added a commit that referenced this pull request Jan 28, 2026
Don't override @opts in create_client_from_token

(cherry picked from commit 363e3da)
agrare added a commit that referenced this pull request Jan 28, 2026
Fixed:
- CVE-2026-1530 - Don't override @opts in create_client_from_token (#168)
@agrare
Copy link
Collaborator

agrare commented Feb 2, 2026

Released in [v1.6.0]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants