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

T1259: Fix quotation with --push parameters #7

Merged
merged 1 commit into from Feb 21, 2019

Conversation

varesa
Copy link
Contributor

@varesa varesa commented Feb 21, 2019

Using some of the server-options on recently rolling release builds fails with:
Options error: Unrecognized option or missing or extra parameter(s) in [CMD-LINE]:1: push (2.4.0)

This seems to be caused by missing quotes around the parameters to --push.

From openvpn man-page:

   --push option
          Push a config file option back to the client for remote execution.  Note that option must be enclosed in double quotes ("").

This PR adds double-quotes around the parameters that didn't yet have any.

@varesa
Copy link
Contributor Author

varesa commented Feb 21, 2019

This fixes https://phabricator.vyos.net/T1259 (confirmed by trae32566)

@dmbaturin dmbaturin merged commit 9166dde into vyos-legacy:current Feb 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants