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.

Sign up for GitHub

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

Jump to bottom

Remove IP address from https config #300

Closed
nynymike opened this issue May 23, 2017 · 1 comment
Closed

Remove IP address from https config #300

nynymike opened this issue May 23, 2017 · 1 comment
Assignees
@tecoholic
Labels
enhancement Update to existing feature or functionality
Milestone
CE 3.0.2

Comments

@nynymike
Copy link
Contributor

nynymike commented May 23, 2017 •
edited
Loading

Replace %{ip}s:443 with *:443 in apache config.

We should also remove the /etc/hosts entry, and see if anything breaks, in which case we could probably just remove the IP address prompt all together.
@nynymike nynymike added the enhancement Update to existing feature or functionality label May 23, 2017
@nynymike nynymike added this to the CE 3.0.2 milestone May 23, 2017
@tecoholic
Copy link
Contributor

IP from Apache configs have been removed. I have left the /etc/hosts file after @ganesh-at-wiw's recommendataion that

  • It's required to be in /etc/hosts for adverse situations. As a standard practice, it's used, so every name resolution doesn't go through DNS
  • The other reason is: nsswitch refers to files first and then DNS

tecoholic added a commit that referenced this issue Jun 8, 2017
@tecoholic
closes #300: removed ip specified in apache confs
382426b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tecoholic tecoholic

Labels
enhancement Update to existing feature or functionality
Projects
None yet
Milestone
CE 3.0.2
Development

No branches or pull requests
2 participants
@tecoholic @nynymike