Project

General

Profile

Actions

Refactor #28906

closed

Make cdn_ssl_version a Setting

Added by Chris Roberts about 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman modules
Target version:
-
Difficulty:
easy
Triaged:
Yes
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 2 (0 open2 closed)

Copied from Katello - Refactor #28476: Make cdn_ssl_version a SettingClosedChris RobertsActions
Copied to Installer - Refactor #29184: Make cdn_ssl_version a SettingClosedChris RobertsActions
Actions #1

Updated by Chris Roberts about 4 years ago

Actions #2

Updated by The Foreman Bot about 4 years ago

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

Updated by Chris Roberts about 4 years ago

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

Updated by Chris Roberts about 4 years ago

Actions #5

Updated by The Foreman Bot almost 4 years ago

  • Fixed in Releases 2.1.0 added
Actions #6

Updated by Chris Roberts almost 4 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF