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

OpenVPN Instance : Client Specific Overrides not applied #6784

Closed
2 tasks done
pfoo opened this issue Aug 26, 2023 · 3 comments
Closed
2 tasks done

OpenVPN Instance : Client Specific Overrides not applied #6784

pfoo opened this issue Aug 26, 2023 · 3 comments
Assignees
Labels
bug Production bug

Comments

@pfoo
Copy link

pfoo commented Aug 26, 2023

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Describe the bug

On OPNsense 23.7.2 Client Specific Overrides are not applied with an OpenVPN (new) instance (no mention to CSC in log, and /var/etc/openvpn-csc/3/ stays empty) despite checking the instance in the CSO "Servers" field.

After applying commit d3af50a and manually creating directory /var/etc/openvpn-csc/961988-0461-4-8933-779744/ CSO are now created in /var/etc/openvpn-csc/961988-0461-4-8933-779744/ but instance configuration file still mention client-config-dir /var/etc/openvpn-csc/3 instead of /var/etc/openvpn-csc/961988-0461-4-8933-779744/

Followup forum thread : https://forum.opnsense.org/index.php?topic=35447.0

To Reproduce

Steps to reproduce the behavior: Try to use CSO in 23.7.2 with or without patch d3af50a

Environment

OPNsense 23.7.2 + patch d3af50a

@AdSchellevis AdSchellevis self-assigned this Aug 26, 2023
@AdSchellevis AdSchellevis added the bug Production bug label Aug 26, 2023
@AdSchellevis
Copy link
Member

In my attempt to server both new (mvc) id's and legacy id's (which are used for the device naming), I missed a spot it seems. Can you try 78d49d8 on top? It should preserve the usual numbering in the /var/etc/openvpn-csc/ path.

@pfoo
Copy link
Author

pfoo commented Aug 27, 2023

It is working with both d3af50a and 78d49d8 commit applied ! Thanks !

@AdSchellevis
Copy link
Member

@pfoo thanks for confirming, we'll aim to include this in our next minor release.

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

No branches or pull requests

2 participants