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

Fix OpenVPN client additional options typo #2781

Merged
merged 1 commit into from Mar 23, 2016

Conversation

Projects
None yet
3 participants
@jlduran
Member

jlduran commented Mar 23, 2016

Should read client instead of server. Also remove the push example.

Fix OpenVPN client additional options typo
Should read client instead of server.  Also remove the `push` example.
@NOYB

This comment has been minimized.

Show comment
Hide comment
@NOYB

NOYB Mar 23, 2016

Contributor

Why remove the push example? Example seem to be appropriate.
Also not so sure that this should say client instead of server. The server configuration is what is being specified here.

Contributor

NOYB commented Mar 23, 2016

Why remove the push example? Example seem to be appropriate.
Also not so sure that this should say client instead of server. The server configuration is what is being specified here.

@jlduran

This comment has been minimized.

Show comment
Hide comment
@jlduran

jlduran Mar 23, 2016

Member

This page (OpenVPN client) is supposed to configure the /var/etc/openvpn/client<number>.conf file, not the server.conffile. I guess the push example is more appropriate for the server config.

Member

jlduran commented Mar 23, 2016

This page (OpenVPN client) is supposed to configure the /var/etc/openvpn/client<number>.conf file, not the server.conffile. I guess the push example is more appropriate for the server config.

@NOYB

This comment has been minimized.

Show comment
Hide comment
@NOYB

NOYB Mar 23, 2016

Contributor

Ah. I missed that this was on *client.php

Contributor

NOYB commented Mar 23, 2016

Ah. I missed that this was on *client.php

@pfSense-git-updates pfSense-git-updates merged commit a7a430b into pfsense:master Mar 23, 2016

@jlduran jlduran deleted the jlduran:fix-openvpn-client-typo branch Mar 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment