Project

General

Profile

Actions
Copy link

Bug #4529

closed

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

Added by Lukas Zapletal over 10 years ago. Updated over 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
Actions
Copy link

Also available in: Atom PDF