Project

General

Profile

Bug #27654

Expose http(s) port number setting via httpboot plugin

Added by Lukas Zapletal 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Category:
HTTPBoot
Target version:
-

Description

Foreman core needs this to setup boot URL correctly.


Related issues

Related to Foreman - Bug #26338: HTTP UEFI boot PXE loader use wrong smart proxy portClosed

Associated revisions

Revision e344672b (diff)
Added by Lukas Zapletal 5 months ago

Fixes #27654 - expose http(s) port setting

Revision f88b53d6 (diff)
Added by Ewoud Kohl van Wijngaarden 5 months ago

Refs #27654 - Move HTTP enabled logic to plugin settings

This introduces simple helpers to be used elsewhere. The Plugin Settings
class makes more sense for this since it's specific to a plugin setting.

Revision 79ce65f7 (diff)
Added by Ewoud Kohl van Wijngaarden 5 months ago

Refs #27654 - Account for module enabled over http or not

The module can be enabled for http-only while the proxy is https-only
(or other combinations). This patch takes the enabled setting into
account and guarantees that the http_port is only set when the module is
actually available over http. The same goes for https_port.

History

#1 Updated by Lukas Zapletal 5 months ago

  • Related to Bug #26338: HTTP UEFI boot PXE loader use wrong smart proxy port added

#2 Updated by The Foreman Bot 5 months ago

  • Assignee set to Lukas Zapletal
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/676 added

#3 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 1.24.0 added

#4 Updated by Anonymous 5 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/theforeman/smart-proxy/pull/677 added

#6 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/theforeman/smart-proxy/pull/679 added

Also available in: Atom PDF