Bug #4529: Discovery not working well with foreman.server=https://abc - Discovery - Foreman

Actions

Copy link

Bug #4529

closed

Discovery not working well with foreman.server=https://abc

Added by Lukas Zapletal about 10 years ago. Updated almost 6 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Lukas Zapletal

Category:

Target version:

1.9.0

Difficulty:

easy

Triaged:

Bugzilla link:

Pull request:

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

17:07      gwmngilfen | lzap: ah hang on
17:07      gwmngilfen | discover_by_dns looks wrong
17:08      gwmngilfen | it's figuring out server_name from foreman.server=.... but then it does a direct DNS lookup on that name without stripping https://
17:08      gwmngilfen | line 42
17:09      gwmngilfen | you probably want to do TCPSocket.gethostbyname(server_name.gsub('https://',''))
17:09            lzap | oh ok so I found a bug, will fix
17:09      gwmngilfen | or do it properly with URI.parse and then you the host part