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

Add verifyhost parameter to balancermember resource #268

Merged
merged 1 commit into from
Nov 22, 2016

Conversation

JAORMX
Copy link

@JAORMX JAORMX commented Nov 18, 2016

The verifyhost parameter from haproxy [1] will attempt to match
the server's certificate CN or SubjectAltName, and will fail if
there is no match.

In the balancermember resource, it was added as a boolean, since it's
somewhat difficult to add the parameter to the options list, since
each hostname in the verifyhost will be different for each of the
servers in the server_names list. So, to address this, we now can
specify a boolean, and if it's set to true, it will use the host of
the specific server line and use it for the verifyhost option.

Note that this configuration only works if we are using HAProxy with
OpenSSL, and if we set up the 'ssl' and 'verify required' options in
the options of the servers.

[1] https://cbonte.github.io/haproxy-dconv/configuration-1.5.html#5.2-verifyhost

@JAORMX JAORMX force-pushed the haproxy-verifyhost branch 2 times, most recently from 6c72af3 to 27099de Compare November 22, 2016 07:48
The verifyhost parameter from haproxy [1] will attempt to match
the server's certificate CN or SubjectAltName, and will fail if
there is no match.

In the balancermember resource, it was added as a boolean, since it's
somewhat difficult to add the parameter to the options list, since
each hostname in the verifyhost will be different for each of the
servers in the server_names list. So, to address this, we now can
specify a boolean, and if it's set to true, it will use the host of
the specific server line and use it for the verifyhost option.

Note that this configuration only works if we are using HAProxy with
OpenSSL, and if we set up the 'ssl' and 'verify required' options in
the options of the servers.

[1] https://cbonte.github.io/haproxy-dconv/configuration-1.5.html#5.2-verifyhost
@JAORMX JAORMX force-pushed the haproxy-verifyhost branch from 27099de to 2030a99 Compare November 22, 2016 08:50
@eputnam eputnam merged commit cb0e282 into puppetlabs:master Nov 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants