Skip to content

Fix for another apache configuration Listen special case #2093

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

Merged
merged 1 commit into from
Jan 5, 2016

Conversation

watercrossing
Copy link
Contributor

The configurator currently fails on a setup of:

Listen 443
Listen 127.0.0.1:8080

This fixes this shortcoming.

@watercrossing
Copy link
Contributor Author

Fixes #1994 .

@SwartzCr
Copy link
Contributor

SwartzCr commented Jan 5, 2016

@pde I'd like to see this go through the letstest test farm, but the code and test look good to me

@pde pde added this to the 0.2.0 milestone Jan 5, 2016
@pde
Copy link
Member

pde commented Jan 5, 2016

Test farm test_apache2.sh:

0 ubuntu15.10 pass
1 ubuntu15.04LTS pass
2 ubuntu14.04LTS pass
3 ubuntu14.04LTS_32bit pass
4 ubuntu12.04LTS fail
5 debian8.1 pass
6 debian7.8.aws.1 fail
7 debian7.8.aws.1_32bit fail
8 amazonlinux-2015.09.1 fail
9 amazonlinux-2015.03.1 fail
10 RHEL7 pass
11 fedora23 pass
12 centos7 pass
13 centos6 fail
Cleaning up...
Logs in  letest-1452024755

@pde
Copy link
Member

pde commented Jan 5, 2016

That's a non-regression, so merging. Let's also figure out whether this closes #1699 for us.

pde added a commit that referenced this pull request Jan 5, 2016
Fix for another apache configuration `Listen` special case
@pde pde merged commit b1de1cd into certbot:master Jan 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants