Project

General

Profile

Refactor #29184

Make cdn_ssl_version a Setting

Added by Chris Roberts over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Chris Roberts
Category:
foreman-installer script
Target version:
-
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/puppet-katello/pull/320, https://github.com/theforeman/foreman-installer/pull/477, https://github.com/theforeman/foreman-installer/pull/494
Fixed in Releases:
Foreman - 2.1.0
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 Installer - Refactor #28906: Make cdn_ssl_version a SettingClosed

Associated revisions

Revision f928a41c (diff)
Added by Chris Roberts over 2 years ago

Fixes #29184 - Add cdn-ssl param migration hook

Revision a3a0fe6d (diff)
Added by Eric Helms over 2 years ago

Refs #29184: Check that cdn-ssl answer exists

History

#1 Updated by Chris Roberts over 2 years ago

#2 Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-installer/pull/477 added

#3 Updated by The Foreman Bot over 2 years ago

  • Fixed in Releases 2.1.0 added

#4 Updated by Chris Roberts over 2 years ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot over 2 years ago

  • Pull request https://github.com/theforeman/foreman-installer/pull/494 added

Also available in: Atom PDF