Project

General

Profile

Actions

Bug #18936

closed

Download of OS specific kernel and initrd files should verify certificates

Added by Dominic Schlegel about 7 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
TFTP
Target version:
-
Difficulty:
trivial
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

The documentation https://theforeman.org/manuals/1.14/index.html#4.3.9TFTP mentions that the Download of OS specific kernel and initrd files does not verify certificates. In particular step 5 of the work flow mentions the exact wget command:

wget --no-check-certificate -nv -c <src> -O "<destination>" 

I can not imagine any reason why the option --no-check-certificate should be used. Therefore I suggest to remove it.
If there is a use case which is valid I suggest that the GUI offers a checkbox in the installation Media detail page that turns on/off the certificate check.

Actions #1

Updated by Dominic Cleal about 7 years ago

  • Project changed from Foreman to Smart Proxy
  • Category set to TFTP
Actions #2

Updated by Lukas Zapletal almost 3 years ago

  • Difficulty set to trivial
  • Triaged changed from No to Yes
Actions #3

Updated by Lukas Zapletal almost 3 years ago

Let's add a new tftp.yaml setting named "verify_server_cert", when true the "--no-check-certificate" argument will be used.

Actions #5

Updated by Dominic Schlegel almost 3 years ago

Lukas Zapletal wrote:

Let's add a new tftp.yaml setting named "verify_server_cert", when true the "--no-check-certificate" argument will be used.

That naming is confusing me. Shouldn't it be when "verify_server_cert" is set to false that the "--no-check-certificate" should be added?

Actions #6

Updated by Lukas Zapletal almost 3 years ago

Correct, typing on a call is hard :-)

Actions #7

Updated by The Foreman Bot almost 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/792 added
Actions #8

Updated by Anna Vítová almost 3 years ago

  • Assignee set to Anna Vítová
Actions #9

Updated by The Foreman Bot over 2 years ago

  • Fixed in Releases 3.0.0 added
Actions #10

Updated by Anonymous over 2 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF