Project

General

Profile

Refactor #28906

Make cdn_ssl_version a Setting

Added by Chris Roberts 7 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman modules
Target version:
-
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Currently the SSL version used to talk to the CDN can be set via cdn_ssl_version. This exists for proxies that don't support modern standards like TLS 1.2. In an ideal world these proxies wouldn't exist.

It's used here:

https://github.com/Katello/katello/blob/5f7f32ebf1f4a221b6c370ea2a15b733e18527b8/app/lib/katello/resources/cdn.rb#L64-L69

The installer has a parameter for this:

https://github.com/theforeman/puppet-katello/blob/1de6fedb0b8ccbeab024d754b5c678840676f677/manifests/init.pp#L131

Since this is not critical to be set at installation time and can actually be a runtime setting, a database Setting would be a better place.


Related issues

Copied from Katello - Refactor #28476: Make cdn_ssl_version a SettingClosed
Copied to Installer - Refactor #29184: Make cdn_ssl_version a SettingClosed

Associated revisions

Revision dc90390a (diff)
Added by Chris Roberts 5 months ago

Fixes #28906 - Remove cdn_ssl_version from module

History

#1 Updated by Chris Roberts 7 months ago

#2 Updated by The Foreman Bot 7 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/puppet-katello/pull/320 added

#3 Updated by Chris Roberts 7 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/8552)

#4 Updated by Chris Roberts 6 months ago

#5 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 2.1.0 added

#6 Updated by Chris Roberts 5 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF