Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need to add http_port to app.py #63

Open
wanghaiqing2015 opened this issue Jul 14, 2020 · 3 comments
Open

Need to add http_port to app.py #63

wanghaiqing2015 opened this issue Jul 14, 2020 · 3 comments

Comments

@wanghaiqing2015
Copy link

Port missing -> http://%s/cblr/svc/op/ks/profile

            if not self.system:
                profile_data["autoinst"] = "http://%s/cblr/svc/op/ks/profile/%s" % (
                    profile_data['http_server'], profile_data['name'])
            else:
                profile_data["autoinst"] = "http://%s/cblr/svc/op/ks/system/%s" % (
                    profile_data['http_server'], profile_data['name'])
@wanghaiqing2015
Copy link
Author

@opoplawski
Copy link
Contributor

We really need to be able to support https: as well

@mpdehaan
Copy link

While that sounds good for servers that don't offer things up over http (cobbler already did), it's important to understand the greatest security implications of a PXE network.

TFTP is already a wide open protocol and it is easy to spoof and access information for other systems. At this point, you really have to trust the network because the system could already be offering you up a fake installer, or the client could be lying about a MAC.

In short, there's no way to authenticate the https:// client so no "secret" content can be put in the kickstart, because there's no way to put a username/password in there safely.

At this point, https:// only defends against MITM, but you can already compromise the earlier stage of the install process.

I'm not sure of the level of certificate checking done by the installer these days - it might have some value for external systems - but can't be used with authentication.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Feature Requests
Development

No branches or pull requests

3 participants